Skip to content

Commit

Permalink
Merge pull request #30 from dschoenbauer/feature/27-no-records-affect…
Browse files Browse the repository at this point in the history
…ed-core-change

Update NoRecordsAffectedException parent
  • Loading branch information
dschoenbauer committed Sep 11, 2018
2 parents c1b0ff2 + 801c673 commit 1f97b5d
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/Sql/Exception/NoRecordsAffectedException.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,14 @@
*/
namespace DSchoenbauer\Sql\Exception;

use DSchoenbauer\Exception\Http\ClientError\NotFoundException;
use DSchoenbauer\Exception\Http\ClientError\ConflictException;

/**
* No Records are being impacted by the query
*
* @author David Schoenbauer
*/
class NoRecordsAffectedException extends NotFoundException implements SqlExceptionInterface
class NoRecordsAffectedException extends ConflictException implements SqlExceptionInterface
{

}
4 changes: 3 additions & 1 deletion tests/src/Sql/Exception/NoRecordsAffectedExceptionTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
*/
namespace DSchoenbauer\Sql\Exception;

use DSchoenbauer\Exception\Http\ClientError\ConflictException;
use PHPUnit\Framework\TestCase;

/**
Expand All @@ -33,6 +34,7 @@
*/
class NoRecordsAffectedExceptionTest extends TestCase
{
private $object;

protected function setUp()
{
Expand All @@ -45,6 +47,6 @@ public function testHasBaseInterface()
}

public function testHasCorrectParent(){

$this->assertInstanceOf(ConflictException::class, $this->object);
}
}

0 comments on commit 1f97b5d

Please sign in to comment.