Skip to content

Commit 2ef58b2

Browse files
committed
Add regression tests for ReferenceList
1 parent 988f427 commit 2ef58b2

1 file changed

Lines changed: 13 additions & 6 deletions

File tree

tests/unit/ReferenceListTest.php

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -192,14 +192,11 @@ public function testAddReferenceAtIndexZero() {
192192
}
193193

194194
public function testAddReferenceAtNegativeIndex() {
195-
$reference1 = new Reference( array( new PropertyNoValueSnak( 1 ) ) );
196-
$reference2 = new Reference( array( new PropertyNoValueSnak( 2 ) ) );
197-
$reference3 = new Reference( array( new PropertyNoValueSnak( 3 ) ) );
198-
199-
$references = new ReferenceList( array( $reference1, $reference2 ) );
195+
$reference = new Reference( array( new PropertyNoValueSnak( 1 ) ) );
196+
$referenceList = new ReferenceList();
200197

201198
$this->setExpectedException( 'InvalidArgumentException' );
202-
$references->addReference( $reference3, -1 );
199+
$referenceList->addReference( $reference, -1 );
203200
}
204201

205202
public function testGivenEmptyReference_addReferenceDoesNotAdd() {
@@ -239,6 +236,16 @@ public function testIndexOf( ReferenceList $array ) {
239236
}
240237
}
241238

239+
public function testIndexOf_falseForMissingReferences() {
240+
$referenceList = new ReferenceList();
241+
$reference = new Reference( array( new PropertyNoValueSnak( 1 ) ) );
242+
243+
$referenceList->addNewReference( new PropertyNoValueSnak( 1 ) );
244+
245+
$this->assertFalse( $referenceList->indexOf( new Reference() ) );
246+
$this->assertFalse( $referenceList->indexOf( $reference ) );
247+
}
248+
242249
/**
243250
* @dataProvider instanceProvider
244251
* @param ReferenceList $array

0 commit comments

Comments
 (0)