Skip to content

Commit

Permalink
fix test since we use array_is_list
Browse files Browse the repository at this point in the history
  • Loading branch information
chriskapp committed Apr 6, 2024
1 parent 66ef26e commit 671f84e
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions tests/Util/CurveArrayTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -112,11 +112,7 @@ public function testIsAssoc()
$this->assertFalse(CurveArray::isAssoc(['foo']));
$this->assertTrue(CurveArray::isAssoc(['foo' => null, 'bar' => null]));
$this->assertTrue(CurveArray::isAssoc([0 => 'foo', 'foo' => null, 'bar' => null]));

// it is a know problem that this should return false. Since we use
// isset for performance reasons it returns true. If it becomes
// important we can switch to array_key_exists
$this->assertTrue(CurveArray::isAssoc([null]));
$this->assertFalse(CurveArray::isAssoc([null]));
}

public function testObjectify()
Expand Down

0 comments on commit 671f84e

Please sign in to comment.