Added tests and fixed broken build

Author: jenssegers

src/Jenssegers/Eloquent/Model.php

@@ -176,15 +176,4 @@ protected function newBaseQueryBuilder()
         return parent::newBaseQueryBuilder();
     }
 
-    /**
-     * Create a new Eloquent query builder for the model.
-     *
-     * @param  \Jenssegers\Mongodb\Query\Builder $query
-     * @return \Jenssegers\Mongodb\Eloquent\Builder|static
-     */
-    public function newEloquentBuilder($query)
-    {
-        return new Builder($query);
-    }
-
 }

src/Jenssegers/Eloquent/Builder.php renamed to src/Jenssegers/Mongodb/Eloquent/Builder.php

@@ -1,4 +1,4 @@
-<?php namespace Jenssegers\Eloquent;
+<?php namespace Jenssegers\Mongodb\Eloquent;
 
 class Builder extends \Illuminate\Database\Eloquent\Builder {
 
@@ -12,4 +12,4 @@ class Builder extends \Illuminate\Database\Eloquent\Builder {
         'count', 'min', 'max', 'avg', 'sum', 'exists', 'push', 'pull'
     );
 
-}
+}

src/Jenssegers/Mongodb/Model.php

@@ -5,6 +5,7 @@
 use Illuminate\Database\Eloquent\Relations\HasMany;
 
 use Jenssegers\Mongodb\DatabaseManager as Resolver;
+use Jenssegers\Mongodb\Eloquent\Builder;
 use Jenssegers\Mongodb\Query\Builder as QueryBuilder;
 use Jenssegers\Mongodb\Relations\BelongsTo;
 use Jenssegers\Mongodb\Relations\BelongsToMany;
@@ -203,6 +204,17 @@ public function dropColumn($columns)
         return $query = $this->newQuery()->where($this->getKeyName(), $this->getKey())->unset($columns);
     }
 
+    /**
+     * Create a new Eloquent query builder for the model.
+     *
+     * @param  \Jenssegers\Mongodb\Query\Builder $query
+     * @return \Jenssegers\Mongodb\Eloquent\Builder|static
+     */
+    public function newEloquentBuilder($query)
+    {
+        return new Builder($query);
+    }
+
     /**
      * Handle dynamic method calls into the method.
      *

src/Jenssegers/Mongodb/Relations/BelongsToMany.php

@@ -130,10 +130,10 @@ public function attach($id, array $attributes = array(), $touch = true)
 		$foreignIds = array_pluck($records, $this->foreignKey);
 
 		// Attach to the parent model
-		$parent->push($this->otherKey, $otherIds[0])->update(array());
+		$parent->push($this->otherKey, $otherIds[0]);
 
 		// Attach to the related model
-		$related->push($this->foreignKey, $foreignIds[0])->update(array());
+		$related->push($this->foreignKey, $foreignIds[0]);
 	}
 
 	/**
@@ -192,7 +192,7 @@ public function detach($ids = array(), $touch = true)
 		{
 			$query->pull($this->otherKey, $id);
 		}
-		
+
 		// Remove the relation from the related model
 		$related->pull($this->foreignKey, $this->parent->getKey());
 

tests/ModelTest.php

@@ -332,4 +332,16 @@ public function testIdAttribute()
 		$this->assertNotEquals($user->id, $user->_id);
 	}
 
+	public function testPushPull()
+	{
+		$user = User::create(array('name' => 'John Doe', 'tags' => array()));
+
+		$result = User::where('_id', $user->_id)->push('tags', 'tag1');
+		$user = User::where('_id', $user->_id)->first();
+
+		#$this->assertTrue(is_int($result));
+		$this->assertTrue(is_array($user->tags));
+		$this->assertEquals(1, count($user->tags));
+	}
+
 }