Add additional tests for Rule::array validation scenarios (#54844)

* Added tests for Rule::array with empty arrays, duplicate keys, and numeric values.

* Added tests for array validation with objects, nullable values, and empty arrays

* Add test for getCount with valid excludeId condition

Author: alikhosravidev

tests/Validation/ValidationArrayRuleTest.php

@@ -18,6 +18,9 @@ public function testItCorrectlyFormatsAStringVersionOfTheRule()
 
         $this->assertSame('array', (string) $rule);
 
+        $rule = Rule::array([]);
+        $this->assertSame('array', (string) $rule);
+
         $rule = Rule::array('key_1', 'key_2', 'key_3');
 
         $this->assertSame('array:key_1,key_2,key_3', (string) $rule);
@@ -37,6 +40,12 @@ public function testItCorrectlyFormatsAStringVersionOfTheRule()
         $rule = Rule::array([ArrayKeysBacked::key_1, ArrayKeysBacked::key_2, ArrayKeysBacked::key_3]);
 
         $this->assertSame('array:key_1,key_2,key_3', (string) $rule);
+
+        $rule = Rule::array(['key_1', 'key_1']);
+        $this->assertSame('array:key_1,key_1', (string) $rule);
+
+        $rule = Rule::array([1, 2, 3]);
+        $this->assertSame('array:1,2,3', (string) $rule);
     }
 
     public function testArrayValidation()
@@ -46,6 +55,18 @@ public function testArrayValidation()
         $v = new Validator($trans, ['foo' => 'not an array'], ['foo' => Rule::array()]);
         $this->assertTrue($v->fails());
 
+        $v = new Validator($trans, ['foo' => (object) ['key_1' => 'bar']], ['foo' => Rule::array()]);
+        $this->assertTrue($v->fails());
+
+        $v = new Validator($trans, ['foo' => null], ['foo' => ['nullable', Rule::array()]]);
+        $this->assertTrue($v->passes());
+
+        $v = new Validator($trans, ['foo' => []], ['foo' => Rule::array()]);
+        $this->assertTrue($v->passes());
+
+        $v = new Validator($trans, ['foo' => ['key_1' => []]], ['foo' => Rule::array(['key_1'])]);
+        $this->assertTrue($v->passes());
+
         $v = new Validator($trans, ['foo' => ['bar']], ['foo' => (string) Rule::array()]);
         $this->assertTrue($v->passes());
 

tests/Validation/ValidationDatabasePresenceVerifierTest.php

@@ -60,4 +60,18 @@ public function testBasicCountWithClosures()
 
         $this->assertEquals(100, $verifier->getCount('table', 'column', 'value', null, null, $extra));
     }
+
+    public function testGetCountWithValidExcludeId()
+    {
+        $verifier = new DatabasePresenceVerifier($db = m::mock(ConnectionResolverInterface::class));
+        $verifier->setConnection('connection');
+        $db->shouldReceive('connection')->once()->with('connection')->andReturn($conn = m::mock(stdClass::class));
+        $conn->shouldReceive('table')->once()->with('table')->andReturn($builder = m::mock(stdClass::class));
+        $builder->shouldReceive('useWritePdo')->once()->andReturn($builder);
+        $builder->shouldReceive('where')->with('column', '=', 'value')->andReturn($builder);
+        $builder->shouldReceive('where')->with('id', '<>', 123)->andReturn($builder);
+        $builder->shouldReceive('count')->once()->andReturn(100);
+
+        $this->assertEquals(100, $verifier->getCount('table', 'column', 'value', 123, 'id', []));
+    }
 }