Skip to content

Commit 9f61749

Browse files
authored
Merge pull request #2417 from nextcloud/chore/noid/fix-phpunit-12-issues
chore: fix phpunit 12 issues
2 parents f924b31 + 69029cd commit 9f61749

2 files changed

Lines changed: 9 additions & 2 deletions

File tree

tests/ConsumerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ public static function receiveData(): array {
119119
}
120120

121121
#[DataProvider('receiveData')]
122-
public function testReceiveStream(string $type, string $author, string $affectedUser, string $subject): void {
122+
public function testReceiveStream(string $type, string $author, string $affectedUser, string $subject, $expected): void {
123123
$consumer = new Consumer($this->data, $this->activityManager, $this->userSettings, $this->notificationGenerator);
124124
$event = Server::get(IManager::class)->generateEvent();
125125
$event->setApp('test')

tests/FilesHooksTest.php

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -185,7 +185,14 @@ public function testFileCreate(string $currentUser, string $selfSubject, string
185185
$filesHooks->fileCreate('path');
186186
}
187187

188-
#[DataProvider('dataFileCreate')]
188+
public static function dataFileCreateUser(): array {
189+
return [
190+
['user'],
191+
[''],
192+
];
193+
}
194+
195+
#[DataProvider('dataFileCreateUser')]
189196
public function testFileCreateRoot(string $currentUser): void {
190197
$filesHooks = $this->getFilesHooks([
191198
'addNotificationsForFileAction',

0 commit comments

Comments
 (0)