Skip to content
Permalink
Browse files

Merge pull request #26 from ergebnis/fix/move

Fix: Move exceptions into Exception namespace
  • Loading branch information
localheinz committed Mar 10, 2020
2 parents c8d5caf + 0cb471c commit 1fc172900f6d712207ff480fca72fad40ebba766
@@ -56,9 +56,9 @@ parameters:
path: src/EntityDef.php

-
message: "#^Class \"Ergebnis\\\\FactoryBot\\\\EntityDefinitionUnavailable\" is not allowed to extend \"OutOfRangeException\"\\.$#"
message: "#^Class \"Ergebnis\\\\FactoryBot\\\\Exception\\\\EntityDefinitionUnavailable\" is not allowed to extend \"OutOfRangeException\"\\.$#"
count: 1
path: src/EntityDefinitionUnavailable.php
path: src/Exception/EntityDefinitionUnavailable.php

-
message: "#^Class Ergebnis\\\\FactoryBot\\\\FieldDef is neither abstract nor final\\.$#"
@@ -346,9 +346,9 @@ parameters:
path: test/Integration/FixtureFactoryTest.php

-
message: "#^Call to static method PHPUnit\\\\Framework\\\\Assert\\:\\:assertInstanceOf\\(\\) with 'OutOfRangeException' and Ergebnis\\\\FactoryBot\\\\EntityDefinitionUnavailable will always evaluate to true\\.$#"
message: "#^Call to static method PHPUnit\\\\Framework\\\\Assert\\:\\:assertInstanceOf\\(\\) with 'OutOfRangeException' and Ergebnis\\\\FactoryBot\\\\Exception\\\\EntityDefinitionUnavailable will always evaluate to true\\.$#"
count: 1
path: test/Unit/EntityDefinitionUnavailableTest.php
path: test/Unit/Exception/EntityDefinitionUnavailableTest.php

-
message: "#^Anonymous function should have native return typehint \"string\"\\.$#"
@@ -14,6 +14,7 @@
namespace Ergebnis\FactoryBot\Definition;

use Ergebnis\Classy;
use Ergebnis\FactoryBot\Exception;
use Ergebnis\FactoryBot\FixtureFactory;
use Faker\Generator;

@@ -33,8 +34,8 @@ private function __construct()
*
* @param string $directory
*
* @throws Exception\InvalidDirectory
* @throws Exception\InvalidDefinition
* @throws Exception\InvalidDirectory
*
* @return self
*/
@@ -11,7 +11,7 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot;
namespace Ergebnis\FactoryBot\Exception;

final class EntityDefinitionUnavailable extends \OutOfRangeException implements Exception
{
@@ -11,7 +11,7 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot;
namespace Ergebnis\FactoryBot\Exception;

/**
* Marker interface for FactoryGirl\Provider\Doctrine exceptions.
@@ -11,7 +11,7 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot\Definition\Exception;
namespace Ergebnis\FactoryBot\Exception;

final class InvalidDefinition extends \RuntimeException
{
@@ -11,7 +11,7 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot\Definition\Exception;
namespace Ergebnis\FactoryBot\Exception;

final class InvalidDirectory extends \InvalidArgumentException
{
@@ -65,7 +65,7 @@ public function __construct(ORM\EntityManagerInterface $em)
* @param mixed $name
* @param array $fieldOverrides
*
* @throws EntityDefinitionUnavailable
* @throws Exception\EntityDefinitionUnavailable
*/
public function get($name, array $fieldOverrides = [])
{
@@ -74,7 +74,7 @@ public function get($name, array $fieldOverrides = [])
}

if (!\array_key_exists($name, $this->entityDefs)) {
throw EntityDefinitionUnavailable::for($name);
throw Exception\EntityDefinitionUnavailable::for($name);
}

/** @var EntityDef $def */
@@ -15,8 +15,8 @@

use Ergebnis\FactoryBot\Definition\Definition;
use Ergebnis\FactoryBot\Definition\Definitions;
use Ergebnis\FactoryBot\Definition\Exception;
use Ergebnis\FactoryBot\Definition\FakerAwareDefinition;
use Ergebnis\FactoryBot\Exception;
use Ergebnis\FactoryBot\FixtureFactory;
use Ergebnis\FactoryBot\Test\Fixture;
use Ergebnis\Test\Util\Helper;
@@ -28,8 +28,8 @@
*
* @covers \Ergebnis\FactoryBot\Definition\Definitions
*
* @uses \Ergebnis\FactoryBot\Definition\Exception\InvalidDefinition
* @uses \Ergebnis\FactoryBot\Definition\Exception\InvalidDirectory
* @uses \Ergebnis\FactoryBot\Exception\InvalidDefinition
* @uses \Ergebnis\FactoryBot\Exception\InvalidDirectory
*/
final class DefinitionsTest extends Framework\TestCase
{
@@ -11,22 +11,23 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot\Test\Unit;
namespace Ergebnis\FactoryBot\Test\Unit\Exception;

use Ergebnis\FactoryBot\Exception;
use PHPUnit\Framework;

/**
* @internal
*
* @covers \Ergebnis\FactoryBot\EntityDefinitionUnavailable
* @covers \Ergebnis\FactoryBot\Exception\EntityDefinitionUnavailable
*/
final class EntityDefinitionUnavailableTest extends Framework\TestCase
{
public function testForReturnsException(): void
{
$name = 'foo';

$exception = \Ergebnis\FactoryBot\EntityDefinitionUnavailable::for($name);
$exception = Exception\EntityDefinitionUnavailable::for($name);

self::assertInstanceOf(\OutOfRangeException::class, $exception);

@@ -11,16 +11,16 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot\Test\Unit\Definition\Exception;
namespace Ergebnis\FactoryBot\Test\Unit\Exception;

use Ergebnis\FactoryBot\Definition\Exception;
use Ergebnis\FactoryBot\Exception;
use Ergebnis\Test\Util\Helper;
use PHPUnit\Framework;

/**
* @internal
*
* @covers \Ergebnis\FactoryBot\Definition\Exception\InvalidDefinition
* @covers \Ergebnis\FactoryBot\Exception\InvalidDefinition
*/
final class InvalidDefinitionTest extends Framework\TestCase
{
@@ -11,16 +11,16 @@
* @see https://github.com/ergebnis/factory-bot
*/

namespace Ergebnis\FactoryBot\Test\Unit\Definition\Exception;
namespace Ergebnis\FactoryBot\Test\Unit\Exception;

use Ergebnis\FactoryBot\Definition\Exception;
use Ergebnis\FactoryBot\Exception;
use Ergebnis\Test\Util\Helper;
use PHPUnit\Framework;

/**
* @internal
*
* @covers \Ergebnis\FactoryBot\Definition\Exception\InvalidDirectory
* @covers \Ergebnis\FactoryBot\Exception\InvalidDirectory
*/
final class InvalidDirectoryTest extends Framework\TestCase
{
@@ -15,7 +15,7 @@

use Doctrine\Common;
use Doctrine\ORM;
use Ergebnis\FactoryBot\EntityDefinitionUnavailable;
use Ergebnis\FactoryBot\Exception;
use Ergebnis\FactoryBot\FieldDef;
use Ergebnis\FactoryBot\FixtureFactory;
use Ergebnis\FactoryBot\Test\Fixture;
@@ -26,7 +26,7 @@
* @covers \Ergebnis\FactoryBot\FixtureFactory
*
* @uses \Ergebnis\FactoryBot\EntityDef
* @uses \Ergebnis\FactoryBot\EntityDefinitionUnavailable
* @uses \Ergebnis\FactoryBot\Exception\EntityDefinitionUnavailable
* @uses \Ergebnis\FactoryBot\FieldDef
*/
final class FixtureFactoryTest extends AbstractTestCase
@@ -37,7 +37,7 @@ public function testGetThrowsEntityDefinitionUnavailableWhenDefinitionIsUnavaila

$fixtureFactory = new FixtureFactory($entityManager);

$this->expectException(EntityDefinitionUnavailable::class);
$this->expectException(Exception\EntityDefinitionUnavailable::class);

$fixtureFactory->get('foo');
}

0 comments on commit 1fc1729

Please sign in to comment.
You can’t perform that action at this time.