Merge pull request #21981 from nextcloud/backport/21972/stable19

MorrisJobke · web-flow · commit de70dcf70843 · 2020-07-30T11:03:54.000+02:00
[stable19] Fix PHPUnit deprecation warnings
diff --git a/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php b/apps/dav/tests/unit/CalDAV/CalDavBackendTest.php
@@ -279,7 +279,7 @@ public function testCalendarObjectsOperations() {
 		$this->assertCount(0, $calendarObjects);
 	}
 
-	
+
 	public function testMultipleCalendarObjectsWithSameUID() {
 		$this->expectException(\Sabre\DAV\Exception\BadRequest::class);
 		$this->expectExceptionMessage('Calendar object with uid already exists in this calendar collection.');
@@ -441,7 +441,7 @@ public function testCalendarQuery($expectedEventsInResult, $propFilters, $compFi
 		$expectedEventsInResult = array_map(function ($index) use ($events) {
 			return $events[$index];
 		}, $expectedEventsInResult);
-		$this->assertEquals($expectedEventsInResult, $result, '', 0.0, 10, true);
+		$this->assertEqualsCanonicalizing($expectedEventsInResult, $result);
 	}
 
 	public function testGetCalendarObjectByUID() {
diff --git a/apps/dav/tests/unit/Command/ListCalendarsTest.php b/apps/dav/tests/unit/Command/ListCalendarsTest.php
@@ -63,7 +63,6 @@ protected function setUp(): void {
 		);
 	}
 
-	
 	public function testWithBadUser() {
 		$this->expectException(\InvalidArgumentException::class);
 
@@ -76,7 +75,7 @@ public function testWithBadUser() {
 		$commandTester->execute([
 			'uid' => self::USERNAME,
 		]);
-		$this->assertContains("User <" . self::USERNAME . "> in unknown", $commandTester->getDisplay());
+		$this->assertStringContainsString("User <" . self::USERNAME . "> in unknown", $commandTester->getDisplay());
 	}
 
 	public function testWithCorrectUserWithNoCalendars() {
@@ -94,7 +93,7 @@ public function testWithCorrectUserWithNoCalendars() {
 		$commandTester->execute([
 			'uid' => self::USERNAME,
 		]);
-		$this->assertContains("User <" . self::USERNAME . "> has no calendars\n", $commandTester->getDisplay());
+		$this->assertStringContainsString("User <" . self::USERNAME . "> has no calendars\n", $commandTester->getDisplay());
 	}
 
 	public function dataExecute() {
@@ -133,7 +132,7 @@ public function testWithCorrectUser(bool $readOnly, string $output) {
 		$commandTester->execute([
 			'uid' => self::USERNAME,
 		]);
-		$this->assertContains($output, $commandTester->getDisplay());
-		$this->assertNotContains(BirthdayService::BIRTHDAY_CALENDAR_URI, $commandTester->getDisplay());
+		$this->assertStringContainsString($output, $commandTester->getDisplay());
+		$this->assertStringNotContainsString(BirthdayService::BIRTHDAY_CALENDAR_URI, $commandTester->getDisplay());
 	}
 }
diff --git a/apps/dav/tests/unit/Command/MoveCalendarTest.php b/apps/dav/tests/unit/Command/MoveCalendarTest.php
@@ -121,7 +121,7 @@ public function testWithBadUserOrigin($userOriginExists, $userDestinationExists)
 		]);
 	}
 
-	
+
 	public function testMoveWithInexistantCalendar() {
 		$this->expectException(\InvalidArgumentException::class);
 		$this->expectExceptionMessage('User <user> has no calendar named <personal>. You can run occ dav:list-calendars to list calendars URIs for this user.');
@@ -148,7 +148,7 @@ public function testMoveWithInexistantCalendar() {
 		]);
 	}
 
-	
+
 	public function testMoveWithExistingDestinationCalendar() {
 		$this->expectException(\InvalidArgumentException::class);
 		$this->expectExceptionMessage('User <user2> already has a calendar named <personal>.');
@@ -313,7 +313,7 @@ public function testMoveWithDestinationPartOfGroup() {
 			'destinationuid' => 'user2',
 		]);
 
-		$this->assertContains("[OK] Calendar <personal> was moved from user <user> to <user2>", $commandTester->getDisplay());
+		$this->assertStringContainsString("[OK] Calendar <personal> was moved from user <user> to <user2>", $commandTester->getDisplay());
 	}
 
 	public function testMoveWithDestinationNotPartOfGroupAndForce() {
@@ -360,7 +360,7 @@ public function testMoveWithDestinationNotPartOfGroupAndForce() {
 			'--force' => true
 		]);
 
-		$this->assertContains("[OK] Calendar <personal> was moved from user <user> to <user2>", $commandTester->getDisplay());
+		$this->assertStringContainsString("[OK] Calendar <personal> was moved from user <user> to <user2>", $commandTester->getDisplay());
 	}
 
 	public function dataTestMoveWithCalendarAlreadySharedToDestination(): array {
diff --git a/apps/federatedfilesharing/tests/FederatedShareProviderTest.php b/apps/federatedfilesharing/tests/FederatedShareProviderTest.php
@@ -197,7 +197,9 @@ public function testCreate() {
 			'accepted' => 0,
 			'token' => 'token',
 		];
-		$this->assertArraySubset($expected, $data);
+		foreach (array_keys($expected) as $key) {
+			$this->assertEquals($expected[$key], $data[$key], "Assert that value for key '$key' is the same");
+		}
 
 		$this->assertEquals($data['id'], $share->getId());
 		$this->assertEquals(\OCP\Share::SHARE_TYPE_REMOTE, $share->getShareType());
diff --git a/tests/Core/Command/TwoFactorAuth/CleanupTest.php b/tests/Core/Command/TwoFactorAuth/CleanupTest.php
@@ -60,6 +60,6 @@ public function testCleanup() {
 
 		$this->assertEquals(0, $rc);
 		$output = $this->cmd->getDisplay();
-		$this->assertContains("All user-provider associations for provider u2f have been removed", $output);
+		$this->assertStringContainsString("All user-provider associations for provider u2f have been removed", $output);
 	}
 }
diff --git a/tests/Core/Command/TwoFactorAuth/DisableTest.php b/tests/Core/Command/TwoFactorAuth/DisableTest.php
@@ -67,7 +67,7 @@ public function testInvalidUID() {
 		]);
 
 		$this->assertEquals(1, $rc);
-		$this->assertContains("Invalid UID", $this->command->getDisplay());
+		$this->assertStringContainsString("Invalid UID", $this->command->getDisplay());
 	}
 
 	public function testEnableNotSupported() {
@@ -87,7 +87,7 @@ public function testEnableNotSupported() {
 		]);
 
 		$this->assertEquals(2, $rc);
-		$this->assertContains("The provider does not support this operation", $this->command->getDisplay());
+		$this->assertStringContainsString("The provider does not support this operation", $this->command->getDisplay());
 	}
 
 	public function testEnabled() {
@@ -107,6 +107,6 @@ public function testEnabled() {
 		]);
 
 		$this->assertEquals(0, $rc);
-		$this->assertContains("Two-factor provider totp disabled for user ricky", $this->command->getDisplay());
+		$this->assertStringContainsString("Two-factor provider totp disabled for user ricky", $this->command->getDisplay());
 	}
 }
diff --git a/tests/Core/Command/TwoFactorAuth/EnableTest.php b/tests/Core/Command/TwoFactorAuth/EnableTest.php
@@ -67,7 +67,7 @@ public function testInvalidUID() {
 		]);
 
 		$this->assertEquals(1, $rc);
-		$this->assertContains("Invalid UID", $this->command->getDisplay());
+		$this->assertStringContainsString("Invalid UID", $this->command->getDisplay());
 	}
 
 	public function testEnableNotSupported() {
@@ -87,7 +87,7 @@ public function testEnableNotSupported() {
 		]);
 
 		$this->assertEquals(2, $rc);
-		$this->assertContains("The provider does not support this operation", $this->command->getDisplay());
+		$this->assertStringContainsString("The provider does not support this operation", $this->command->getDisplay());
 	}
 
 	public function testEnabled() {
@@ -107,6 +107,6 @@ public function testEnabled() {
 		]);
 
 		$this->assertEquals(0, $rc);
-		$this->assertContains("Two-factor provider totp enabled for user belle", $this->command->getDisplay());
+		$this->assertStringContainsString("Two-factor provider totp enabled for user belle", $this->command->getDisplay());
 	}
 }
diff --git a/tests/Core/Command/TwoFactorAuth/EnforceTest.php b/tests/Core/Command/TwoFactorAuth/EnforceTest.php
@@ -64,7 +64,7 @@ public function testEnforce() {
 
 		$this->assertEquals(0, $rc);
 		$display = $this->command->getDisplay();
-		$this->assertContains("Two-factor authentication is enforced for all users", $display);
+		$this->assertStringContainsString("Two-factor authentication is enforced for all users", $display);
 	}
 
 	public function testEnforceForOneGroup() {
@@ -82,7 +82,7 @@ public function testEnforceForOneGroup() {
 
 		$this->assertEquals(0, $rc);
 		$display = $this->command->getDisplay();
-		$this->assertContains("Two-factor authentication is enforced for members of the group(s) twofactorers", $display);
+		$this->assertStringContainsString("Two-factor authentication is enforced for members of the group(s) twofactorers", $display);
 	}
 
 	public function testEnforceForAllExceptOneGroup() {
@@ -100,7 +100,7 @@ public function testEnforceForAllExceptOneGroup() {
 
 		$this->assertEquals(0, $rc);
 		$display = $this->command->getDisplay();
-		$this->assertContains("Two-factor authentication is enforced for all users, except members of yoloers", $display);
+		$this->assertStringContainsString("Two-factor authentication is enforced for all users, except members of yoloers", $display);
 	}
 
 	public function testDisableEnforced() {
@@ -117,7 +117,7 @@ public function testDisableEnforced() {
 
 		$this->assertEquals(0, $rc);
 		$display = $this->command->getDisplay();
-		$this->assertContains("Two-factor authentication is not enforced", $display);
+		$this->assertStringContainsString("Two-factor authentication is not enforced", $display);
 	}
 
 	public function testCurrentStateEnabled() {
@@ -129,7 +129,7 @@ public function testCurrentStateEnabled() {
 
 		$this->assertEquals(0, $rc);
 		$display = $this->command->getDisplay();
-		$this->assertContains("Two-factor authentication is enforced for all users", $display);
+		$this->assertStringContainsString("Two-factor authentication is enforced for all users", $display);
 	}
 
 	public function testCurrentStateDisabled() {
@@ -141,6 +141,6 @@ public function testCurrentStateDisabled() {
 
 		$this->assertEquals(0, $rc);
 		$display = $this->command->getDisplay();
-		$this->assertContains("Two-factor authentication is not enforced", $display);
+		$this->assertStringContainsString("Two-factor authentication is not enforced", $display);
 	}
 }
diff --git a/tests/Core/Command/TwoFactorAuth/StateTest.php b/tests/Core/Command/TwoFactorAuth/StateTest.php
@@ -61,7 +61,7 @@ public function testWrongUID() {
 		]);
 
 		$output = $this->cmd->getDisplay();
-		$this->assertContains("Invalid UID", $output);
+		$this->assertStringContainsString("Invalid UID", $output);
 	}
 
 	public function testStateNoProvidersActive() {
@@ -84,7 +84,7 @@ public function testStateNoProvidersActive() {
 		]);
 
 		$output = $this->cmd->getDisplay();
-		$this->assertContains("Two-factor authentication is not enabled for user eldora", $output);
+		$this->assertStringContainsString("Two-factor authentication is not enabled for user eldora", $output);
 	}
 
 	public function testStateOneProviderActive() {
@@ -107,6 +107,6 @@ public function testStateOneProviderActive() {
 		]);
 
 		$output = $this->cmd->getDisplay();
-		$this->assertContains("Two-factor authentication is enabled for user mohamed", $output);
+		$this->assertStringContainsString("Two-factor authentication is enabled for user mohamed", $output);
 	}
 }
diff --git a/tests/lib/AppFramework/Http/DownloadResponseTest.php b/tests/lib/AppFramework/Http/DownloadResponseTest.php
@@ -45,7 +45,7 @@ protected function setUp(): void {
 	public function testHeaders() {
 		$headers = $this->response->getHeaders();
 
-		$this->assertContains('attachment; filename="file"', $headers['Content-Disposition']);
-		$this->assertContains('content', $headers['Content-Type']);
+		$this->assertStringContainsString('attachment; filename="file"', $headers['Content-Disposition']);
+		$this->assertStringContainsString('content', $headers['Content-Type']);
 	}
 }
diff --git a/tests/lib/ConfigTest.php b/tests/lib/ConfigTest.php
@@ -71,9 +71,7 @@ public function testGetValueReturnsEnvironmentValueIfSetToFalse() {
 
 	public function testSetValue() {
 		$this->config->setValue('foo', 'moo');
-		$expectedConfig = $this->initialConfig;
-		$expectedConfig['foo'] = 'moo';
-		$this->assertAttributeEquals($expectedConfig, 'cache', $this->config);
+		$this->assertSame('moo', $this->config->getValue('foo'));
 
 		$content = file_get_contents($this->configFile);
 		$expected = "<?php\n\$CONFIG = array (\n  'foo' => 'moo',\n  'beers' => \n  array (\n    0 => 'Appenzeller',\n  " .
@@ -82,9 +80,8 @@ public function testSetValue() {
 
 		$this->config->setValue('bar', 'red');
 		$this->config->setValue('apps', ['files', 'gallery']);
-		$expectedConfig['bar'] = 'red';
-		$expectedConfig['apps'] = ['files', 'gallery'];
-		$this->assertAttributeEquals($expectedConfig, 'cache', $this->config);
+		$this->assertSame('red', $this->config->getValue('bar'));
+		$this->assertSame(['files', 'gallery'], $this->config->getValue('apps'));
 
 		$content = file_get_contents($this->configFile);
 
@@ -105,18 +102,17 @@ public function testSetValues() {
 			'not_exists'	=> null,
 		]);
 
-		$this->assertAttributeEquals($this->initialConfig, 'cache', $this->config);
+		$this->assertSame('bar', $this->config->getValue('foo'));
+		$this->assertSame(null, $this->config->getValue('not_exists'));
 		$content = file_get_contents($this->configFile);
 		$this->assertEquals(self::TESTCONTENT, $content);
 
 		$this->config->setValues([
 			'foo'			=> 'moo',
 			'alcohol_free'	=> null,
 		]);
-		$expectedConfig = $this->initialConfig;
-		$expectedConfig['foo'] = 'moo';
-		unset($expectedConfig['alcohol_free']);
-		$this->assertAttributeEquals($expectedConfig, 'cache', $this->config);
+		$this->assertSame('moo', $this->config->getValue('foo'));
+		$this->assertSame(null, $this->config->getValue('not_exists'));
 
 		$content = file_get_contents($this->configFile);
 		$expected = "<?php\n\$CONFIG = array (\n  'foo' => 'moo',\n  'beers' => \n  array (\n    0 => 'Appenzeller',\n  " .
@@ -126,9 +122,7 @@ public function testSetValues() {
 
 	public function testDeleteKey() {
 		$this->config->deleteKey('foo');
-		$expectedConfig = $this->initialConfig;
-		unset($expectedConfig['foo']);
-		$this->assertAttributeEquals($expectedConfig, 'cache', $this->config);
+		$this->assertSame('this_was_clearly_not_set_before', $this->config->getValue('foo', 'this_was_clearly_not_set_before'));
 		$content = file_get_contents($this->configFile);
 
 		$expected = "<?php\n\$CONFIG = array (\n  'beers' => \n  array (\n    0 => 'Appenzeller',\n  " .
diff --git a/tests/lib/DB/DBSchemaTest.php b/tests/lib/DB/DBSchemaTest.php
@@ -83,8 +83,8 @@ public function doTestSchemaDumping() {
 		$outfile = $this->tempManager->getTemporaryFile();
 		OC_DB::getDbStructure($outfile);
 		$content = file_get_contents($outfile);
-		$this->assertContains($this->table1, $content);
-		$this->assertContains($this->table2, $content);
+		$this->assertStringContainsString($this->table1, $content);
+		$this->assertStringContainsString($this->table2, $content);
 	}
 
 	public function doTestSchemaRemoving() {
diff --git a/tests/lib/L10N/FactoryTest.php b/tests/lib/L10N/FactoryTest.php
@@ -329,7 +329,7 @@ public function testFindAvailableLanguages($app) {
 			->with($app)
 			->willReturn(\OC::$SERVERROOT . '/tests/data/l10n/');
 
-		$this->assertEquals(['cs', 'de', 'en', 'ru'], $factory->findAvailableLanguages($app), '', 0.0, 10, true);
+		$this->assertEqualsCanonicalizing(['cs', 'de', 'en', 'ru'], $factory->findAvailableLanguages($app));
 	}
 
 	public function dataLanguageExists() {
@@ -360,7 +360,7 @@ public function testFindAvailableLanguagesWithThemes() {
 			->with('theme')
 			->willReturn('abc');
 
-		$this->assertEquals(['en', 'zz'], $factory->findAvailableLanguages($app), '', 0.0, 10, true);
+		$this->assertEqualsCanonicalizing(['en', 'zz'], $factory->findAvailableLanguages($app));
 	}
 
 	/**
diff --git a/tests/lib/Template/CSSResourceLocatorTest.php b/tests/lib/Template/CSSResourceLocatorTest.php
@@ -107,16 +107,6 @@ private function randomString() {
 		return sha1(uniqid(mt_rand(), true));
 	}
 
-	public function testConstructor() {
-		$locator = $this->cssResourceLocator();
-		$this->assertAttributeEquals('theme', 'theme', $locator);
-		$this->assertAttributeEquals('core', 'serverroot', $locator);
-		$this->assertAttributeEquals(['core'=>'map','3rd'=>'party'], 'mapping', $locator);
-		$this->assertAttributeEquals('3rd', 'thirdpartyroot', $locator);
-		$this->assertAttributeEquals('map', 'webroot', $locator);
-		$this->assertAttributeEquals([], 'resources', $locator);
-	}
-
 	public function testFindWithAppPathSymlink() {
 		// First create new apps path, and a symlink to it
 		$apps_dirname = $this->randomString();
diff --git a/tests/lib/Template/JSResourceLocatorTest.php b/tests/lib/Template/JSResourceLocatorTest.php
@@ -86,17 +86,6 @@ private function randomString() {
 		return sha1(uniqid(mt_rand(), true));
 	}
 
-
-	public function testConstructor() {
-		$locator = $this->jsResourceLocator();
-		$this->assertAttributeEquals('theme', 'theme', $locator);
-		$this->assertAttributeEquals('core', 'serverroot', $locator);
-		$this->assertAttributeEquals(['core'=>'map','3rd'=>'party'], 'mapping', $locator);
-		$this->assertAttributeEquals('3rd', 'thirdpartyroot', $locator);
-		$this->assertAttributeEquals('map', 'webroot', $locator);
-		$this->assertAttributeEquals([], 'resources', $locator);
-	}
-
 	public function testFindWithAppPathSymlink() {
 		// First create new apps path, and a symlink to it
 		$apps_dirname = $this->randomString();
diff --git a/tests/lib/Template/ResourceLocatorTest.php b/tests/lib/Template/ResourceLocatorTest.php
@@ -33,17 +33,6 @@ public function getResourceLocator($theme, $core_map, $party_map, $appsRoots) {
 			'', true, true, true, []);
 	}
 
-	public function testConstructor() {
-		$locator = $this->getResourceLocator('theme',
-			['core'=>'map'], ['3rd'=>'party'], ['foo'=>'bar']);
-		$this->assertAttributeEquals('theme', 'theme', $locator);
-		$this->assertAttributeEquals('core', 'serverroot', $locator);
-		$this->assertAttributeEquals(['core'=>'map','3rd'=>'party'], 'mapping', $locator);
-		$this->assertAttributeEquals('3rd', 'thirdpartyroot', $locator);
-		$this->assertAttributeEquals('map', 'webroot', $locator);
-		$this->assertAttributeEquals([], 'resources', $locator);
-	}
-
 	public function testFind() {
 		$locator = $this->getResourceLocator('theme',
 			['core' => 'map'], ['3rd' => 'party'], ['foo' => 'bar']);

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,6 @@ protected function setUp(): void {`
`63`	`63`	`);`
`64`	`64`	`}`
`65`	`65`
`66`		`-`
`67`	`66`	`public function testWithBadUser() {`
`68`	`67`	`$this->expectException(\InvalidArgumentException::class);`
`69`	`68`
`@@ -76,7 +75,7 @@ public function testWithBadUser() {`
`76`	`75`	`$commandTester->execute([`
`77`	`76`	`'uid' => self::USERNAME,`
`78`	`77`	`]);`
`79`		`- $this->assertContains("User <" . self::USERNAME . "> in unknown", $commandTester->getDisplay());`
	`78`	`+ $this->assertStringContainsString("User <" . self::USERNAME . "> in unknown", $commandTester->getDisplay());`
`80`	`79`	`}`
`81`	`80`
`82`	`81`	`public function testWithCorrectUserWithNoCalendars() {`
`@@ -94,7 +93,7 @@ public function testWithCorrectUserWithNoCalendars() {`
`94`	`93`	`$commandTester->execute([`
`95`	`94`	`'uid' => self::USERNAME,`
`96`	`95`	`]);`
`97`		`- $this->assertContains("User <" . self::USERNAME . "> has no calendars\n", $commandTester->getDisplay());`
	`96`	`+ $this->assertStringContainsString("User <" . self::USERNAME . "> has no calendars\n", $commandTester->getDisplay());`
`98`	`97`	`}`
`99`	`98`
`100`	`99`	`public function dataExecute() {`
`@@ -133,7 +132,7 @@ public function testWithCorrectUser(bool $readOnly, string $output) {`
`133`	`132`	`$commandTester->execute([`
`134`	`133`	`'uid' => self::USERNAME,`
`135`	`134`	`]);`
`136`		`- $this->assertContains($output, $commandTester->getDisplay());`
`137`		`- $this->assertNotContains(BirthdayService::BIRTHDAY_CALENDAR_URI, $commandTester->getDisplay());`
	`135`	`+ $this->assertStringContainsString($output, $commandTester->getDisplay());`
	`136`	`+ $this->assertStringNotContainsString(BirthdayService::BIRTHDAY_CALENDAR_URI, $commandTester->getDisplay());`
`138`	`137`	`}`
`139`	`138`	`}`
Original file line number	Diff line number	Diff line change
`@@ -60,6 +60,6 @@ public function testCleanup() {`
`60`	`60`
`61`	`61`	`$this->assertEquals(0, $rc);`
`62`	`62`	`$output = $this->cmd->getDisplay();`
`63`		`- $this->assertContains("All user-provider associations for provider u2f have been removed", $output);`
	`63`	`+ $this->assertStringContainsString("All user-provider associations for provider u2f have been removed", $output);`
`64`	`64`	`}`
`65`	`65`	`}`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ public function testInvalidUID() {`
`67`	`67`	`]);`
`68`	`68`
`69`	`69`	`$this->assertEquals(1, $rc);`
`70`		`- $this->assertContains("Invalid UID", $this->command->getDisplay());`
	`70`	`+ $this->assertStringContainsString("Invalid UID", $this->command->getDisplay());`
`71`	`71`	`}`
`72`	`72`
`73`	`73`	`public function testEnableNotSupported() {`
`@@ -87,7 +87,7 @@ public function testEnableNotSupported() {`
`87`	`87`	`]);`
`88`	`88`
`89`	`89`	`$this->assertEquals(2, $rc);`
`90`		`- $this->assertContains("The provider does not support this operation", $this->command->getDisplay());`
	`90`	`+ $this->assertStringContainsString("The provider does not support this operation", $this->command->getDisplay());`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`public function testEnabled() {`
`@@ -107,6 +107,6 @@ public function testEnabled() {`
`107`	`107`	`]);`
`108`	`108`
`109`	`109`	`$this->assertEquals(0, $rc);`
`110`		`- $this->assertContains("Two-factor provider totp disabled for user ricky", $this->command->getDisplay());`
	`110`	`+ $this->assertStringContainsString("Two-factor provider totp disabled for user ricky", $this->command->getDisplay());`
`111`	`111`	`}`
`112`	`112`	`}`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public function testEnforce() {`
`64`	`64`
`65`	`65`	`$this->assertEquals(0, $rc);`
`66`	`66`	`$display = $this->command->getDisplay();`
`67`		`- $this->assertContains("Two-factor authentication is enforced for all users", $display);`
	`67`	`+ $this->assertStringContainsString("Two-factor authentication is enforced for all users", $display);`
`68`	`68`	`}`
`69`	`69`
`70`	`70`	`public function testEnforceForOneGroup() {`
`@@ -82,7 +82,7 @@ public function testEnforceForOneGroup() {`
`82`	`82`
`83`	`83`	`$this->assertEquals(0, $rc);`
`84`	`84`	`$display = $this->command->getDisplay();`
`85`		`- $this->assertContains("Two-factor authentication is enforced for members of the group(s) twofactorers", $display);`
	`85`	`+ $this->assertStringContainsString("Two-factor authentication is enforced for members of the group(s) twofactorers", $display);`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`public function testEnforceForAllExceptOneGroup() {`
`@@ -100,7 +100,7 @@ public function testEnforceForAllExceptOneGroup() {`
`100`	`100`
`101`	`101`	`$this->assertEquals(0, $rc);`
`102`	`102`	`$display = $this->command->getDisplay();`
`103`		`- $this->assertContains("Two-factor authentication is enforced for all users, except members of yoloers", $display);`
	`103`	`+ $this->assertStringContainsString("Two-factor authentication is enforced for all users, except members of yoloers", $display);`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`public function testDisableEnforced() {`
`@@ -117,7 +117,7 @@ public function testDisableEnforced() {`
`117`	`117`
`118`	`118`	`$this->assertEquals(0, $rc);`
`119`	`119`	`$display = $this->command->getDisplay();`
`120`		`- $this->assertContains("Two-factor authentication is not enforced", $display);`
	`120`	`+ $this->assertStringContainsString("Two-factor authentication is not enforced", $display);`
`121`	`121`	`}`
`122`	`122`
`123`	`123`	`public function testCurrentStateEnabled() {`
`@@ -129,7 +129,7 @@ public function testCurrentStateEnabled() {`
`129`	`129`
`130`	`130`	`$this->assertEquals(0, $rc);`
`131`	`131`	`$display = $this->command->getDisplay();`
`132`		`- $this->assertContains("Two-factor authentication is enforced for all users", $display);`
	`132`	`+ $this->assertStringContainsString("Two-factor authentication is enforced for all users", $display);`
`133`	`133`	`}`
`134`	`134`
`135`	`135`	`public function testCurrentStateDisabled() {`
`@@ -141,6 +141,6 @@ public function testCurrentStateDisabled() {`
`141`	`141`
`142`	`142`	`$this->assertEquals(0, $rc);`
`143`	`143`	`$display = $this->command->getDisplay();`
`144`		`- $this->assertContains("Two-factor authentication is not enforced", $display);`
	`144`	`+ $this->assertStringContainsString("Two-factor authentication is not enforced", $display);`
`145`	`145`	`}`
`146`	`146`	`}`
Original file line number	Diff line number	Diff line change
`@@ -61,7 +61,7 @@ public function testWrongUID() {`
`61`	`61`	`]);`
`62`	`62`
`63`	`63`	`$output = $this->cmd->getDisplay();`
`64`		`- $this->assertContains("Invalid UID", $output);`
	`64`	`+ $this->assertStringContainsString("Invalid UID", $output);`
`65`	`65`	`}`
`66`	`66`
`67`	`67`	`public function testStateNoProvidersActive() {`
`@@ -84,7 +84,7 @@ public function testStateNoProvidersActive() {`
`84`	`84`	`]);`
`85`	`85`
`86`	`86`	`$output = $this->cmd->getDisplay();`
`87`		`- $this->assertContains("Two-factor authentication is not enabled for user eldora", $output);`
	`87`	`+ $this->assertStringContainsString("Two-factor authentication is not enabled for user eldora", $output);`
`88`	`88`	`}`
`89`	`89`
`90`	`90`	`public function testStateOneProviderActive() {`
`@@ -107,6 +107,6 @@ public function testStateOneProviderActive() {`
`107`	`107`	`]);`
`108`	`108`
`109`	`109`	`$output = $this->cmd->getDisplay();`
`110`		`- $this->assertContains("Two-factor authentication is enabled for user mohamed", $output);`
	`110`	`+ $this->assertStringContainsString("Two-factor authentication is enabled for user mohamed", $output);`
`111`	`111`	`}`
`112`	`112`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ protected function setUp(): void {`
`45`	`45`	`public function testHeaders() {`
`46`	`46`	`$headers = $this->response->getHeaders();`
`47`	`47`
`48`		`- $this->assertContains('attachment; filename="file"', $headers['Content-Disposition']);`
`49`		`- $this->assertContains('content', $headers['Content-Type']);`
	`48`	`+ $this->assertStringContainsString('attachment; filename="file"', $headers['Content-Disposition']);`
	`49`	`+ $this->assertStringContainsString('content', $headers['Content-Type']);`
`50`	`50`	`}`
`51`	`51`	`}`