a

AnaelMobilia · Oct 7, 2024 · e7d12ae · e7d12ae
1 parent 1b6d6e4
commit e7d12ae
Show file tree

Hide file tree

Showing 6 changed files with 260 additions and 208 deletions.
diff --git a/__tests/AbuseTest.php b/__tests/AbuseTest.php
@@ -50,15 +50,20 @@ public function testAbuse(): void
 
         $imageBloquee = new ImageObject('image_15.png');
         $imageMemeMd5 = new ImageObject('image_16.png');
-        $this->assertTrue($imageBloquee->isSignalee(), 'Image signalée doit l\'être');
-        $this->assertTrue(
-            $imageMemeMd5->isSignalee(),
-            'Image avec même MD5 qu\'une image signalée doit l\'être également'
-        );
+
+
         $this->assertEmpty(
             $msgErreur,
             __FUNCTION__ . ' ne devrait pas lever de message d\'erreur - Erreur : ' . $msgErreur
         );
+        $this->assertTrue(
+            $imageBloquee->isSignalee(),
+            'Image signalée doit l\'être'
+        );
+        $this->assertTrue(
+            $imageMemeMd5->isSignalee(),
+            'Image avec même MD5 qu\'une image signalée doit l\'être également'
+        );
     }
 
     /**
@@ -79,10 +84,6 @@ public function testAbuseRenvoiImage(): void
         require 'upload.php';
         ob_end_clean();
 
-        $this->assertTrue(
-            $monImage->isBloquee(),
-            'Renvoi image déjà bloquée doit être isBloquée en BDD'
-        );
         $this->assertEmpty(
             $msgErreur,
             __FUNCTION__ . ' ne devrait pas lever de message d\'erreur - Erreur : ' . $msgErreur
@@ -91,6 +92,10 @@ public function testAbuseRenvoiImage(): void
             $msgWarning,
             __FUNCTION__ . ' ne devrait pas lever de message de warning - Warning : ' . $msgWarning
         );
+        $this->assertTrue(
+            $monImage->isBloquee(),
+            'Renvoi image déjà bloquée doit être isBloquée en BDD'
+        );
     }
 
     /**
@@ -111,14 +116,14 @@ public function testAbuseImageApprouvee(): void
         ob_end_clean();
 
         $imageSignalee = new ImageObject('_image_404.png');
-        $this->assertFalse(
-            $imageSignalee->isSignalee(),
-            'Image approuvée qui est signalée ne doit pas être bloquée'
-        );
         $this->assertEmpty(
             $msgErreur,
             __FUNCTION__ . ' ne devrait pas lever de message d\'erreur - Erreur : ' . $msgErreur
         );
+        $this->assertFalse(
+            $imageSignalee->isSignalee(),
+            'Image approuvée qui est signalée ne doit pas être bloquée'
+        );
     }
 
     /**
@@ -177,11 +182,6 @@ public function testAbuseImageRenvoiImageBloqueeDepuisReseauMalveillant(): void
 
         $imagesApresEnvoi = HelperAdmin::getImagesPotentiellementIndesirables();
 
-        $this->assertEquals(
-            $imagesAvantEnvoi->count(),
-            $imagesApresEnvoi->count(),
-            'Le renvoi d\'une image bloquée doit être bloqué'
-        );
         $this->assertEmpty(
             $msgErreur,
             __FUNCTION__ . ' ne devrait pas lever de message d\'erreur - Erreur : ' . $msgErreur
@@ -190,6 +190,11 @@ public function testAbuseImageRenvoiImageBloqueeDepuisReseauMalveillant(): void
             $msgWarning,
             __FUNCTION__ . ' ne devrait pas lever de message de warning - Warning : ' . $msgWarning
         );
+        $this->assertEquals(
+            $imagesAvantEnvoi->count(),
+            $imagesApresEnvoi->count(),
+            'Le renvoi d\'une image bloquée doit être bloqué'
+        );
     }
 
     /**
@@ -215,11 +220,6 @@ public function testAbuseImageEnvoiDepuisReseauMalveillant(): void
 
         $imagesApresEnvoi = HelperAdmin::getImagesPotentiellementIndesirables();
 
-        $this->assertEquals(
-            ($imagesAvantEnvoi->count() + 1),
-            $imagesApresEnvoi->count(),
-            'L\'image envoyée devrait être considérée comme potentiellement indésirable : ' . var_export($imagesApresEnvoi, true)
-        );
         $this->assertEmpty(
             $msgErreur,
             __FUNCTION__ . ' ne devrait pas lever de message d\'erreur - Erreur : ' . $msgErreur
@@ -228,6 +228,11 @@ public function testAbuseImageEnvoiDepuisReseauMalveillant(): void
             $msgWarning,
             __FUNCTION__ . ' ne devrait pas lever de message de warning - Warning : ' . $msgWarning
         );
+        $this->assertEquals(
+            ($imagesAvantEnvoi->count() + 1),
+            $imagesApresEnvoi->count(),
+            'L\'image envoyée devrait être considérée comme potentiellement indésirable : ' . var_export($imagesApresEnvoi, true)
+        );
     }
 
     /**

diff --git a/__tests/DisplayPicsTest.php b/__tests/DisplayPicsTest.php
@@ -91,7 +91,7 @@ public function testRepertoireInexistant(): void
     public function testImageBloquee(): void
     {
         $_SERVER['REMOTE_ADDR'] = '127.0.0.1';
-        $_SERVER['REQUEST_URI'] = 'files/image_10.jpg';
+        $_SERVER['REQUEST_URI'] = 'files/image_10.png';
         ob_start();
         require 'displayPics.php';
         ob_end_clean();