API phpunit 9 support

silverstripe · Oct 13, 2021 · 2f0837c · 2f0837c
1 parent 42ee290
commit 2f0837c
Show file tree

Hide file tree

Showing 36 changed files with 138 additions and 155 deletions.
diff --git a/composer.json b/composer.json
@@ -26,9 +26,8 @@
         "league/flysystem": "^1.0.70"
     },
     "require-dev": {
+        "silverstripe/recipe-testing": "dev-two",
         "silverstripe/versioned": "^1@dev",
-        "sminnee/phpunit-mock-objects": "^3.4.5",
-        "sminnee/phpunit": "^5.7.29",
         "squizlabs/php_codesniffer": "^3",
         "mikey179/vfsstream": "^1.6"
     },

diff --git a/tests/php/AssetControlExtensionTest.php b/tests/php/AssetControlExtensionTest.php
@@ -25,7 +25,7 @@ class AssetControlExtensionTest extends SapphireTest
         TestObject::class
     ];
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -55,7 +55,7 @@ public function setUp()
         $object3->publishSingle();
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         parent::tearDown();
@@ -224,7 +224,7 @@ public function testReplaceFile()
         $this->assertEquals(AssetStore::VISIBILITY_PUBLIC, $object3->Header->getVisibility());
     }
 
-    
+
     public function testReplaceWithVariant()
     {
         $store = $this->getAssetStore();
@@ -250,7 +250,7 @@ public function testReplaceWithVariant()
             $store->exists($v1->getFilename(), $v1->getHash(), 'boom'),
             sprintf('A variant of %s has been created', $v1->getFilename())
         );
-        
+
         // Let's replace the content of the main file and publish it
         $download->setFromString($v2Content, $v2->getFilename());
         $object1->write();

diff --git a/tests/php/Dev/Tasks/FileMigrationHelperTest.php b/tests/php/Dev/Tasks/FileMigrationHelperTest.php
@@ -50,7 +50,7 @@ protected function getBasePath()
     }
 
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -164,7 +164,7 @@ function ($args) {
         )->execute();
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         Filesystem::removeFolder($this->getBasePath());
@@ -283,7 +283,7 @@ private function individualImage(Image $file)
         $fullFilename = TestAssetStore::base_path() . '/' . $file->getFilename();
         $dir = dirname($fullFilename);
         $baseFilename = basename($fullFilename);
-        $this->assertFileNotExists($dir . '/_resampled');
+        $this->assertFileDoesNotExist($dir . '/_resampled');
         $this->assertFileExists($dir . '/' . $baseFilename);
 
         // Test that SS3.3 variants have been migrated

diff --git a/tests/php/Dev/Tasks/LegacyThumbnailMigrationHelperTest.php b/tests/php/Dev/Tasks/LegacyThumbnailMigrationHelperTest.php
@@ -41,7 +41,7 @@ protected function getBasePath()
     }
 
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -61,7 +61,7 @@ public function setUp()
         }
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         Filesystem::removeFolder($this->getBasePath());
@@ -90,7 +90,7 @@ public function testMigratesWithExistingThumbnailInNewLocation($coreVersion)
         // Moved contains store relative paths
         $base = TestAssetStore::base_path();
 
-        $this->assertFileNotExists(
+        $this->assertFileDoesNotExist(
             $this->joinPaths($base, $expectedLegacyPath),
             'Legacy file has been removed'
         );
@@ -131,7 +131,7 @@ public function testMigratesMultipleFilesInSameFolder($coreVersion)
         $base = TestAssetStore::base_path();
 
         foreach ($expected as $expectedLegacyPath => $expectedNewPath) {
-            $this->assertFileNotExists(
+            $this->assertFileDoesNotExist(
                 $this->joinPaths($base, $expectedLegacyPath),
                 'Legacy file has been removed'
             );
@@ -176,7 +176,7 @@ public function testMigrate($fixtureId, $formats, $coreVersion)
             $expectedNewPath,
             'New file is mapped as expected'
         );
-        $this->assertFileNotExists(
+        $this->assertFileDoesNotExist(
             $this->joinPaths($base, $expectedLegacyPath),
             'Legacy file has been removed'
         );

diff --git a/tests/php/Dev/Tasks/NormaliseAccessMigrationHelperTest.php b/tests/php/Dev/Tasks/NormaliseAccessMigrationHelperTest.php
@@ -41,7 +41,7 @@ protected function getBasePath()
     }
 
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -62,7 +62,7 @@ protected function setUpAssetStore()
         TestAssetStore::activate('NormaliseAccessMigrationHelperTest/assets');
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         Filesystem::removeFolder($this->getBasePath());
@@ -112,11 +112,9 @@ private function getHelper()
         return new Helper('/assets/NormaliseAccessMigrationHelperTest/');
     }
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
     public function testNeedToMoveWithFolder()
     {
+        $this->expectException(\InvalidArgumentException::class);
         /** @var File $file */
         $folder = Folder::find('Uploads');
         $helper = $this->getHelper();
@@ -630,11 +628,9 @@ public function testFindBadFilesWithMultiPublishedVersions()
         );
     }
 
-    /**
-     * @expectedException LogicException
-     */
     public function testNeedToMoveNonExistentFile()
     {
+        $this->expectException(\LogicException::class);
         /** @var File $file */
         $file = $this->objFromFixture(File::class, 'file1');
         $file->CanViewType = InheritedPermissions::LOGGED_IN_USERS;

diff --git a/tests/php/Dev/Tasks/SS4FileMigrationHelperTest.php b/tests/php/Dev/Tasks/SS4FileMigrationHelperTest.php
@@ -45,7 +45,7 @@ class SS4FileMigrationHelperTest extends SapphireTest
         ]
     ];
 
-    public function setUp()
+    protected function setUp(): void
     {
         Config::nest(); // additional nesting here necessary
         Config::modify()->merge(File::class, 'migrate_legacy_file', false);
@@ -181,7 +181,7 @@ protected function defineDestinationStrategy()
         $store->setPublicResolutionStrategy($public);
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         parent::tearDown();

diff --git a/tests/php/Dev/Tasks/SecureAssetsMigrationHelperTest.php b/tests/php/Dev/Tasks/SecureAssetsMigrationHelperTest.php
@@ -28,7 +28,7 @@ protected function getBasePath()
     }
 
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
 
@@ -52,7 +52,7 @@ public function setUp()
         }
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         Filesystem::removeFolder($this->getBasePath());

diff --git a/tests/php/Dev/Tasks/TagsToShortcodeHelperTest.php b/tests/php/Dev/Tasks/TagsToShortcodeHelperTest.php
@@ -44,7 +44,7 @@ protected function getBasePath()
         return ASSETS_PATH . '/TagsToShortcodeHelperTest';
     }
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->setupAssetStore();
@@ -67,7 +67,7 @@ private function setupAssetStore()
         }
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         Filesystem::removeFolder($this->getBasePath());
@@ -84,17 +84,17 @@ public function testRewrite()
         $documentID = $this->idFromFixture(File::class, 'document');
         $imageID = $this->idFromFixture(Image::class, 'image1');
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             sprintf('<p id="filelink">file link <a href="[file_link,id=%d]">link to file</a></p>', $documentID),
             $newPage->Content
         );
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             sprintf('<p id="image">[image src="/assets/33be1b95cb/myimage.jpg" id="%d"]</p>', $imageID),
             $newPage->Content
         );
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             sprintf(
                 '<p id="variant">[image src="/assets/33be1b95cb/myimage.jpg" width="64" height="64" id="%d"]</p>',
                 $imageID
@@ -137,17 +137,17 @@ public function testLivePageRewrite()
         $documentID = $this->idFromFixture(File::class, 'document');
         $imageID = $this->idFromFixture(Image::class, 'image1');
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             sprintf('<p id="filelink">file link <a href="[file_link,id=%d]">link to file</a></p>', $documentID),
             $newPage->Content
         );
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             sprintf('<p id="image">[image src="/assets/33be1b95cb/myimage.jpg" id="%d"]</p>', $imageID),
             $newPage->Content
         );
 
-        $this->assertContains(
+        $this->assertStringContainsString(
             sprintf(
                 '<p id="variant">[image src="/assets/33be1b95cb/myimage.jpg" width="64" height="64" id="%d"]</p>',
                 $imageID

diff --git a/tests/php/FileFinderTest.php b/tests/php/FileFinderTest.php
@@ -35,11 +35,9 @@ public function testBasicOperation()
         );
     }
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
     public function testInvalidOptionThrowsException()
     {
+        $this->expectException(\InvalidArgumentException::class);
         $finder = new FileFinder();
         $finder->setOption('this_doesnt_exist', 'ok');
     }

diff --git a/tests/php/FileNameFilterTest.php b/tests/php/FileNameFilterTest.php
@@ -8,7 +8,7 @@
 
 class FileNameFilterTest extends SapphireTest
 {
-    protected function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
 

diff --git a/tests/php/FileTest.php b/tests/php/FileTest.php
@@ -37,7 +37,7 @@ class FileTest extends SapphireTest
         MyCustomFile::class
     ];
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->logInWithPermission('ADMIN');
@@ -70,7 +70,7 @@ public function setUp()
         }
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
         parent::tearDown();

diff --git a/tests/php/FilenameParsing/FileIDHelperResolutionStrategyTest.php b/tests/php/FilenameParsing/FileIDHelperResolutionStrategyTest.php
@@ -30,7 +30,7 @@ class FileIDHelperResolutionStrategyTest extends SapphireTest
     /** @var Filesystem */
     private $fs;
 
-    public function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         Sha1FileHashingService::flush();
@@ -49,7 +49,7 @@ public function setUp()
         Injector::inst()->registerService($this->fs, Filesystem::class . '.public');
     }
 
-    public function tearDown()
+    protected function tearDown(): void
     {
         TestAssetStore::reset();
 

diff --git a/tests/php/FilenameParsing/HashFileIDHelperTest.php b/tests/php/FilenameParsing/HashFileIDHelperTest.php
@@ -168,11 +168,9 @@ public function variantIn()
         ];
     }
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
     public function testHashlessBuildFileID()
     {
+        $this->expectException(\InvalidArgumentException::class);
         $this->getHelper()->buildFileID('Filename.txt', '');
     }
 }
diff --git a/tests/php/FilesystemTest.php b/tests/php/FilesystemTest.php
@@ -24,7 +24,7 @@ class FilesystemTest extends SapphireTest
      */
     protected $root;
 
-    protected function setUp()
+    protected function setUp(): void
     {
         parent::setUp();
         $this->root = vfsStream::setup(
@@ -74,7 +74,7 @@ public function testRemoveFolderWithFiles()
 
         Filesystem::removeFolder($folderPath);
         $this->assertDirectoryNotExists($folderPath);
-        $this->assertFileNotExists($filePath);
+        $this->assertFileDoesNotExist($filePath);
     }
 
     public function testRemoveFolderWithSubFolder()
@@ -90,7 +90,7 @@ public function testRemoveFolderWithSubFolder()
         Filesystem::removeFolder($folderPath);
         $this->assertDirectoryNotExists($folderPath);
         $this->assertDirectoryNotExists($emptySubfolder);
-        $this->assertFileNotExists($filePath);
+        $this->assertFileDoesNotExist($filePath);
     }
 
     public function testRemoveFolderWithFalsyFile()
@@ -102,7 +102,7 @@ public function testRemoveFolderWithFalsyFile()
 
         Filesystem::removeFolder($folderPath);
         $this->assertDirectoryNotExists($folderPath);
-        $this->assertFileNotExists($filePath);
+        $this->assertFileDoesNotExist($filePath);
     }
 
     public function testRemovedFolderContentsOnly()
@@ -121,6 +121,6 @@ public function testRemovedFolderContentsOnly()
         $this->assertDirectoryExists($folderPath);
         $this->assertDirectoryNotExists($emptySubfolder);
         $this->assertDirectoryNotExists($notEmptySubfolder);
-        $this->assertFileNotExists($filePath);
+        $this->assertFileDoesNotExist($filePath);
     }
 }