Add tests for contains to certify contains() works on the same objec…

…t, and not only the value of it's properties
steevanb · May 15, 2024 · c84b8f6 · c84b8f6
1 parent 17b6342
commit c84b8f6
Show file tree

Hide file tree

Showing 3 changed files with 67 additions and 5 deletions.
diff --git a/tests/Unit/ObjectCollection/AbstractObjectCollectionTest.php b/tests/Unit/ObjectCollection/AbstractObjectCollectionTest.php
@@ -52,4 +52,33 @@ public function testAssertValueTypeInvalidValueNull(): void
         /** @phpstan-ignore-next-line Parameter #1 $values ... constructor expects ... array<int, null> given. */
         new TestObjectCollection([null]);
     }
+
+    public function testNotContains(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $object3 = new TestObject('value 3');
+        $collection = new TestObjectCollection([$object1, $object2]);
+
+        static::assertFalse($collection->contains($object3));
+    }
+
+    public function testContainsEquals(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $object3 = new TestObject('value 1');
+        $collection = new TestObjectCollection([$object1, $object2]);
+
+        static::assertFalse($collection->contains($object3));
+    }
+
+    public function testContainsSame(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $collection = new TestObjectCollection([$object1, $object2]);
+
+        static::assertTrue($collection->contains($object2));
+    }
 }
diff --git a/tests/Unit/ObjectCollection/AbstractObjectNullableCollectionTest.php b/tests/Unit/ObjectCollection/AbstractObjectNullableCollectionTest.php
@@ -50,4 +50,42 @@ public function testAssertValueTypeNull(): void
 
         static::assertNull($collection->get(0));
     }
+
+    public function testNotContains(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $object3 = new TestObject('value 3');
+        $collection = new TestObjectNullableCollection([$object1, $object2]);
+
+        static::assertFalse($collection->contains($object3));
+    }
+
+    public function testContainsEquals(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $object3 = new TestObject('value 1');
+        $collection = new TestObjectNullableCollection([$object1, $object2]);
+
+        static::assertFalse($collection->contains($object3));
+    }
+
+    public function testContainsSame(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $collection = new TestObjectNullableCollection([$object1, $object2]);
+
+        static::assertTrue($collection->contains($object2));
+    }
+
+    public function testContainsNull(): void
+    {
+        $object1 = new TestObject('value 1');
+        $object2 = new TestObject('value 2');
+        $collection = new TestObjectNullableCollection([$object1, $object2, null]);
+
+        static::assertTrue($collection->contains(null));
+    }
 }
diff --git a/tests/Unit/ObjectCollection/TestObject.php b/tests/Unit/ObjectCollection/TestObject.php
@@ -14,9 +14,4 @@ public function getValue(): string
     {
         return $this->value;
     }
-
-    public function __toString(): string
-    {
-        return $this->value;
-    }
 }