Apply fixes from StyleCI

carspending · Aug 12, 2017 · add88b7 · add88b7
1 parent 2ffadfe
commit add88b7
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 0 deletions.
diff --git a/tests/DefinitionTest.php b/tests/DefinitionTest.php
@@ -128,6 +128,7 @@ public function testModelNotFound()
         } catch (ModelNotFoundException $e) {
             $this->assertSame("The model class '$model' is undefined.", $e->getMessage());
             $this->assertSame($model, $e->getModelClass());
+
             throw $e;
         }
     }
@@ -210,6 +211,7 @@ public function testShouldThrowExceptionWhenLoadingANonExistentGroup()
         } catch (DefinitionNotFoundException $e) {
             $this->assertSame("The model definition '$model' is undefined.", $e->getMessage());
             $this->assertSame($model, $e->getDefinitionName());
+
             throw $e;
         }
     }
@@ -227,6 +229,7 @@ public function testGroupDefineNoBaseModel()
         } catch (DefinitionNotFoundException $e) {
             $this->assertSame("The model definition 'DogModelStub' is undefined.", $e->getMessage());
             $this->assertSame('DogModelStub', $e->getDefinitionName());
+
             throw $e;
         }
     }
@@ -241,6 +244,7 @@ public function testCannotDefineAgain()
         } catch (DefinitionAlreadyDefinedException $e) {
             $this->assertSame("The model definition 'UserModelStub' has already been defined.", $e->getMessage());
             $this->assertSame('UserModelStub', $e->getDefinitionName());
+
             throw $e;
         }
     }
@@ -311,6 +315,7 @@ public function testShouldThrowExceptionWhenLoadingANonExistentDirectory()
         } catch (DirectoryNotFoundException $e) {
             $this->assertSame("The directory '$path' was not found.", $e->getMessage());
             $this->assertSame($path, $e->getPath());
+
             throw $e;
         }
     }

diff --git a/tests/FactoryMuffinTest.php b/tests/FactoryMuffinTest.php
@@ -87,6 +87,7 @@ public function testShouldThrowDefinitionNotFoundException()
         } catch (DefinitionNotFoundException $e) {
             $this->assertSame("The model definition '$model' is undefined.", $e->getMessage());
             $this->assertSame($model, $e->getDefinitionName());
+
             throw $e;
         }
     }

diff --git a/tests/SaveAndDeleteTest.php b/tests/SaveAndDeleteTest.php
@@ -69,6 +69,7 @@ public function testShouldThrowExceptionAfterSaveMethodRename()
             $this->assertSame($model, $e->getModelClass());
             $this->assertSame('foo', $e->getMethodName());
             $this->reload();
+
             throw $e;
         }
 
@@ -94,6 +95,7 @@ public function testShouldThrowExceptionAfterDeleteMethodRename()
             $this->assertSame($model, $exceptions[0]->getModelClass());
             $this->assertSame('bar', $exceptions[0]->getMethodName());
             $this->reload();
+
             throw $e;
         }
 
@@ -111,6 +113,7 @@ public function testShouldThrowExceptionOnModelSaveFailure()
             $this->assertSame("We could not save the model: '$model'.", $e->getMessage());
             $this->assertSame($model, $e->getModelClass());
             $this->assertNull($e->getValidationErrors());
+
             throw $e;
         }
     }
@@ -127,6 +130,7 @@ public function testShouldThrowExceptionOnModelDeleteFailure()
             $exceptions = $e->getExceptions();
             $this->assertSame('We encountered 1 problem while trying to delete the saved models.', $e->getMessage());
             $this->assertSame("We could not delete the model: '$model'.", $exceptions[0]->getMessage());
+
             throw $e;
         }
     }
@@ -143,6 +147,7 @@ public function testShouldAlsoThrowExceptionOnModelDeleteFailure()
             $exceptions = $e->getExceptions();
             $this->assertSame('We encountered 1 problem while trying to delete the saved models.', $e->getMessage());
             $this->assertSame('OH NOES!', $exceptions[0]->getMessage());
+
             throw $e;
         }
     }
@@ -158,6 +163,7 @@ public function testShouldThrowExceptionWithoutSaveMethod()
             $this->assertSame("The save method 'save' was not found on the model: '$model'.", $e->getMessage());
             $this->assertSame($model, $e->getModelClass());
             $this->assertSame('save', $e->getMethodName());
+
             throw $e;
         }
     }
@@ -176,6 +182,7 @@ public function testShouldThrowExceptionWithoutDeleteMethod()
             $this->assertSame("The delete method 'delete' was not found on the model: '$model'.", $exceptions[0]->getMessage());
             $this->assertSame($model, $exceptions[0]->getModelClass());
             $this->assertSame('delete', $exceptions[0]->getMethodName());
+
             throw $e;
         }
     }
@@ -191,6 +198,7 @@ public function testShouldThrowExceptionWithValidationErrors()
             $this->assertSame("Failed to save! We could not save the model: '$model'.", $e->getMessage());
             $this->assertSame($model, $e->getModelClass());
             $this->assertSame('Failed to save!', $e->getValidationErrors());
+
             throw $e;
         }
     }
@@ -206,6 +214,7 @@ public function testShouldThrowExceptionWithBadValidationErrors()
             $this->assertSame("Oh noes. We could not save the model: '$model'.", $e->getMessage());
             $this->assertSame($model, $e->getModelClass());
             $this->assertSame('Oh noes.', $e->getValidationErrors());
+
             throw $e;
         }
     }
@@ -228,6 +237,7 @@ public function testShouldThrowMultipleDeletionExceptions()
             $this->assertSame('delete', $exceptions[1]->getMethodName());
             $this->assertInternalType('array', $e->getExceptions());
             $this->assertCount(2, $e->getExceptions());
+
             throw $e;
         }
     }