Merge pull request #747 from wmde/expectedFirst

Flip mixed $expected and $actual parameters
wmde · Jun 15, 2017 · 7737e66 · 7737e66
2 parents 497d6b6 + 756c013
commit 7737e66
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/tests/unit/Entity/BasicEntityIdParserTest.php b/tests/unit/Entity/BasicEntityIdParserTest.php
@@ -24,7 +24,7 @@ public function testCanParseEntityId( $idString, EntityId $expected ) {
 		$parser = new BasicEntityIdParser();
 		$actual = $parser->parse( $idString );
 
-		$this->assertEquals( $actual, $expected );
+		$this->assertEquals( $expected, $actual );
 	}
 
 	public function entityIdProvider() {

diff --git a/tests/unit/Entity/DispatchingEntityIdParserTest.php b/tests/unit/Entity/DispatchingEntityIdParserTest.php
@@ -29,7 +29,7 @@ public function testCanParseEntityId( $idString, EntityId $expected ) {
 		$parser = $this->getBasicParser();
 		$actual = $parser->parse( $idString );
 
-		$this->assertEquals( $actual, $expected );
+		$this->assertEquals( $expected, $actual );
 	}
 
 	public function entityIdProvider() {

diff --git a/tests/unit/Entity/ItemIdParserTest.php b/tests/unit/Entity/ItemIdParserTest.php
@@ -22,7 +22,7 @@ public function testCanParseEntityId( $idString, ItemId $expected ) {
 		$parser = new ItemIdParser();
 		$actual = $parser->parse( $idString );
 
-		$this->assertEquals( $actual, $expected );
+		$this->assertEquals( $expected, $actual );
 	}
 
 	public function entityIdProvider() {

diff --git a/tests/unit/LegacyIdInterpreterTest.php b/tests/unit/LegacyIdInterpreterTest.php
@@ -25,7 +25,7 @@ class LegacyIdInterpreterTest extends \PHPUnit_Framework_TestCase {
 	public function testNewIdFromTypeAndNumber( EntityId $expected, $type, $number ) {
 		$actual = LegacyIdInterpreter::newIdFromTypeAndNumber( $type, $number );
 
-		$this->assertEquals( $actual, $expected );
+		$this->assertEquals( $expected, $actual );
 	}
 
 	public function idProvider() {