Fix #1 and added gitattributes file

santigarcor · santigarcor · commit 8c5102c5aa15 · 2020-05-15T13:36:49.000-05:00
diff --git a/.gitattributes b/.gitattributes
@@ -0,0 +1,7 @@
+* text=auto
+
+/.github export-ignore
+/tests export-ignore
+.gitattributes export-ignore
+.gitignore export-ignore
+phpunit.xml export-ignore
diff --git a/.phpunit.result.cache b/.phpunit.result.cache
@@ -0,0 +1 @@
+C:37:"PHPUnit\Runner\DefaultTestResultCache":406:{a:2:{s:7:"defects";a:0:{}s:5:"times";a:4:{s:49:"Ownable\Tests\OwnsModelsTraitTest::testOwnsMethod";d:0.115;s:54:"Ownable\Tests\OwnsModelsTraitTest::testdoesntOwnMethod";d:0.01;s:102:"Ownable\Tests\OwnsModelsTraitTest::testItShouldUseTheInterfaceImplementationIfClassImplementsInterface";d:0.012;s:93:"Ownable\Tests\OwnsModelsTraitTest::testItShouldOverrideTheContractAndUserTheRegularValidation";d:0.009;}}}
diff --git a/README.md b/README.md
@@ -54,7 +54,7 @@ $video = Video::first();
 if ($user->owns($video)){}
 
 // Check If the user doesn't owns the video
-if ($user->doesntOwns($video)){}
+if ($user->doesntOwn($video)){}
 
 // Check If the user owns the video but the foreign key is the_user_id
 if ($user->owns($video, 'the_user_id')){}
diff --git a/src/OwnsModels.php b/src/OwnsModels.php
@@ -33,7 +33,7 @@ public function owns($model, $foreignKey = null, $overrideContract = false)
      * @param bool $overrideContract If true the Ownable contract is implemented it won't be used.
      * @return bool
      */
-    public function doesntOwns($model, $foreignKey = null, $overrideContract = false)
+    public function doesntOwn($model, $foreignKey = null, $overrideContract = false)
     {
         return !$this->owns($model, $foreignKey);
     }
diff --git a/tests/OwnsModelsTraitTest.php b/tests/OwnsModelsTraitTest.php
@@ -34,14 +34,14 @@ public function testOwnsMethod()
         $this->assertFalse($user->owns($posts->last()));
     }
 
-    public function testDoesntOwnsMethod()
+    public function testdoesntOwnMethod()
     {
         $user = User::first();
         $posts = Post::all();
 
-        $this->assertFalse($user->doesntOwns($posts->first()));
-        $this->assertTrue($user->doesntOwns($posts->last()));
-        $this->assertTrue($user->doesntOwns($posts->first(), 'other_key'));
+        $this->assertFalse($user->doesntOwn($posts->first()));
+        $this->assertTrue($user->doesntOwn($posts->last()));
+        $this->assertTrue($user->doesntOwn($posts->first(), 'other_key'));
     }
 
     public function testItShouldUseTheInterfaceImplementationIfClassImplementsInterface()
@@ -60,6 +60,6 @@ public function testItShouldOverrideTheContractAndUserTheRegularValidation()
 
         $this->assertFalse($user->owns($otherModels->first(), null, true));
         $this->assertFalse($user->owns($otherModels->last(), null, true));
-        $this->assertTrue($user->doesntOwns($otherModels->last(), null, true));
+        $this->assertTrue($user->doesntOwn($otherModels->last(), null, true));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+C:37:"PHPUnit\Runner\DefaultTestResultCache":406:{a:2:{s:7:"defects";a:0:{}s:5:"times";a:4:{s:49:"Ownable\Tests\OwnsModelsTraitTest::testOwnsMethod";d:0.115;s:54:"Ownable\Tests\OwnsModelsTraitTest::testdoesntOwnMethod";d:0.01;s:102:"Ownable\Tests\OwnsModelsTraitTest::testItShouldUseTheInterfaceImplementationIfClassImplementsInterface";d:0.012;s:93:"Ownable\Tests\OwnsModelsTraitTest::testItShouldOverrideTheContractAndUserTheRegularValidation";d:0.009;}}}`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public function owns($model, $foreignKey = null, $overrideContract = false)`
`33`	`33`	`* @param bool $overrideContract If true the Ownable contract is implemented it won't be used.`
`34`	`34`	`* @return bool`
`35`	`35`	`*/`
`36`		`- public function doesntOwns($model, $foreignKey = null, $overrideContract = false)`
	`36`	`+ public function doesntOwn($model, $foreignKey = null, $overrideContract = false)`
`37`	`37`	`{`
`38`	`38`	`return !$this->owns($model, $foreignKey);`
`39`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -34,14 +34,14 @@ public function testOwnsMethod()`
`34`	`34`	`$this->assertFalse($user->owns($posts->last()));`
`35`	`35`	`}`
`36`	`36`
`37`		`- public function testDoesntOwnsMethod()`
	`37`	`+ public function testdoesntOwnMethod()`
`38`	`38`	`{`
`39`	`39`	`$user = User::first();`
`40`	`40`	`$posts = Post::all();`
`41`	`41`
`42`		`- $this->assertFalse($user->doesntOwns($posts->first()));`
`43`		`- $this->assertTrue($user->doesntOwns($posts->last()));`
`44`		`- $this->assertTrue($user->doesntOwns($posts->first(), 'other_key'));`
	`42`	`+ $this->assertFalse($user->doesntOwn($posts->first()));`
	`43`	`+ $this->assertTrue($user->doesntOwn($posts->last()));`
	`44`	`+ $this->assertTrue($user->doesntOwn($posts->first(), 'other_key'));`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`public function testItShouldUseTheInterfaceImplementationIfClassImplementsInterface()`
`@@ -60,6 +60,6 @@ public function testItShouldOverrideTheContractAndUserTheRegularValidation()`
`60`	`60`
`61`	`61`	`$this->assertFalse($user->owns($otherModels->first(), null, true));`
`62`	`62`	`$this->assertFalse($user->owns($otherModels->last(), null, true));`
`63`		`- $this->assertTrue($user->doesntOwns($otherModels->last(), null, true));`
	`63`	`+ $this->assertTrue($user->doesntOwn($otherModels->last(), null, true));`
`64`	`64`	`}`
`65`	`65`	`}`