Skip to content

Commit 6edd506

Browse files
Copilotsamdark
andcommitted
Rename test method and use assertSame directly in tests
Co-authored-by: samdark <47294+samdark@users.noreply.github.com>
1 parent 74e8ab3 commit 6edd506

File tree

2 files changed

+8
-3
lines changed

2 files changed

+8
-3
lines changed

tests/Common/Reader/ReaderWithFilter/BaseReaderWithLikeTestCase.php

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,12 @@ public function testWithReaderAndMode(
6666
$reader = $this->getReader()->withFilter(new Like($field, $value, $caseSensitive, $mode));
6767
$actualData = $reader->read();
6868

69-
// Assert that we get the expected fixtures based on the filter criteria
70-
$this->assertFixtures($expectedFixtureIndexes, $actualData);
69+
// Build expected fixtures array directly
70+
$expectedFixtures = [];
71+
foreach ($expectedFixtureIndexes as $index) {
72+
$expectedFixtures[$index] = $this->getFixture($index);
73+
}
74+
75+
$this->assertSame($expectedFixtures, $actualData);
7176
}
7277
}

tests/Reader/Iterable/FilterHandler/LikeHandlerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ public function testMatchWithMode(
109109
);
110110
}
111111

112-
public function testBackwardCompatibility(): void
112+
public function testConstructorDefaultMode(): void
113113
{
114114
$filterHandler = new LikeHandler();
115115
$context = new Context([], new FlatValueReader());

0 commit comments

Comments
 (0)