Add flags for persist data w/ db.

Author: chrishalbert

src/NomadicMigration.php

@@ -52,23 +52,29 @@ public function setProperty($key, $value)
 
     /**
      * Syncs up w/ the database and returns the property.
-     * @param string $key The column name of the migration.
+     * @param string $key     The column name of the migration.
+     * @param bool   $persist By default, we get the property from the db.
      * @return mixed
      */
-    public function getProperty($key)
+    public function getProperty($key, $persist = true)
     {
-        $this->syncWithDb();
+        if ($persist) {
+            $this->syncWithDb();
+        }
         return $this->properties[$key];
     }
 
     /**
      * Returns all properties of the migration.
-     * @return array
+     * @param bool   $persist Whether to sync with the db first.
+     * @return mixed
      */
-    public function getProperties()
+    public function getProperties($persist = false)
     {
-        $this->syncWithDb();
-        return $this->properties;
+        if ($persist) {
+            $this->syncWithDb();
+        }
+            return $this->properties;
     }
 
     /**

tests/NomadicMigrationTest.php

@@ -17,7 +17,8 @@ public function setUp()
 
         $this->repo->expects($this->once())
             ->method('getProperties')
-            ->willReturn($this->mockProperties());
+            ->withAnyParameters()
+            ->willReturn($this->mockDbProperties());
 
         $this->migration = $this->getMockForAbstractClass(
             NomadicMigration::class,
@@ -29,12 +30,14 @@ public function setUp()
     public function testSetAndGetProperty()
     {
         $this->migration->setProperty('author', 'Chris');
-        $this->assertEquals('Chris', $this->migration->getProperty('author'));
+        $this->assertEquals('Chris', $this->migration->getProperty('author', false));
+        $this->assertEquals('DB Chris', $this->migration->getProperty('author'));
     }
 
     public function testGetProperties()
     {
-        $this->assertEquals($this->mockProperties(), $this->migration->getProperties());
+        $this->assertEquals([], $this->migration->getProperties());
+        $this->assertEquals($this->mockDbProperties(), $this->migration->getProperties(true));
     }
 
     public function tearDown()
@@ -43,12 +46,12 @@ public function tearDown()
         unset($this->migration);
     }
 
-    protected function mockProperties()
+    protected function mockDbProperties()
     {
         return [
             'migration' => '2018_04_21_Migration',
             'batch' => 1,
-            'author' => 'Chris'
+            'author' => 'DB Chris'
         ];
     }
 }

tests/files/2018_04_04_000000_NomadicMockMigration.php

@@ -3,7 +3,7 @@
 use ChrisHalbert\LaravelNomadic\NomadicMigration;
 
 class NomadicMockMigration extends NomadicMigration {
-    public function getProperties()
+    public function getProperties($syncWithDb = false)
     {
         return ['property' => 'value'];
     }