Skip to content

Commit

Permalink
Remove redundant assertion
Browse files Browse the repository at this point in the history
It is not necessary to test against both the 1st parsing result and the
reference array, since we already know they are equal.
  • Loading branch information
dregad committed Apr 30, 2016
1 parent b942dff commit db05b48
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions tests/Mantis/ConfigParserTest.php
Expand Up @@ -84,12 +84,10 @@ public function testArrays() {
$t_parsed_1 = $t_parser->parse();
$this->assertEquals( $t_parsed_1, $t_reference_result, $this->errorMessage( $t_string ) );

# Export the converted array, and parse again.
# The result should match both the model and the previously parsed array
# Export converted array and parse again: result should match the model
$t_parser = new ConfigParser( var_export( $t_parsed_1 , true ) );
$t_parsed_2 = $t_parser->parse();
$this->assertEquals( $t_parsed_2, $t_reference_result, $this->errorMessage( $t_string ) );
$this->assertEquals( $t_parsed_2, $t_parsed_1, $this->errorMessage( $t_string ) );
}
}

Expand Down

0 comments on commit db05b48

Please sign in to comment.