Skip to content

Commit

Permalink
Fix mock class naming mismatch
Browse files Browse the repository at this point in the history
  • Loading branch information
rosstuck committed Mar 14, 2014
1 parent 498dd80 commit 827dd21
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions Tests/ConfigFormatConverterTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
use Tuck\ConverterBundle\Dumper\StandardDumperFactory;
use Tuck\ConverterBundle\File\SysTempFileFactory;
use Tuck\ConverterBundle\Loader\StandardLoaderFactory;
use Tuck\ConverterBundle\Tests\File\MockSysTempFileFactory;
use Tuck\ConverterBundle\Tests\File\MockTempFileFactory;

class ConfigFormatConverterTest extends \PHPUnit_Framework_TestCase
{
Expand Down Expand Up @@ -80,7 +80,7 @@ public function testCanConvertString()
public function testTempFileIsCleanedUpAfterConversion()
{
// Same setup but with a mock temp file factory that shows us the temp file name used
$mockTempFileFactory = new MockSysTempFileFactory();
$mockTempFileFactory = new MockTempFileFactory();
$converter = new ConfigFormatConverter(new StandardLoaderFactory(), new StandardDumperFactory(), $mockTempFileFactory);

// Make sure the file gets removed, even if the test fails so it doesn't jam the next run
Expand Down
2 changes: 1 addition & 1 deletion Tests/File/TempFileFactoryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ public function testAddingAnExtensionWorks()
public function testThrowsExceptionWhenTempFileAlreadyExists()
{
// This factory always creates the same file name...
$factory = new MockSysTempFileFactory();
$factory = new MockTempFileFactory();

// ...so running it twice here should thrown an exception
$this->tempFile = $factory->createFile();
Expand Down

0 comments on commit 827dd21

Please sign in to comment.