Make sure getRan() returns migrations in proper order.

nostah · nostah · commit 5792da9476c0 · 2015-04-23T14:25:33.000+02:00
diff --git a/src/Illuminate/Database/Migrations/DatabaseMigrationRepository.php b/src/Illuminate/Database/Migrations/DatabaseMigrationRepository.php
@@ -45,7 +45,7 @@ public function __construct(Resolver $resolver, $table)
 	 */
 	public function getRan()
 	{
-		return $this->table()->lists('migration');
+		return $this->table()->orderBy('migration', 'asc')->lists('migration');
 	}
 
 	/**
diff --git a/tests/Database/DatabaseMigrationRepositoryTest.php b/tests/Database/DatabaseMigrationRepositoryTest.php
@@ -18,6 +18,7 @@ public function testGetRanMigrationsListMigrationsByPackage()
 		$connectionMock = m::mock('Illuminate\Database\Connection');
 		$repo->getConnectionResolver()->shouldReceive('connection')->with(null)->andReturn($connectionMock);
 		$repo->getConnection()->shouldReceive('table')->once()->with('migrations')->andReturn($query);
+		$query->shouldReceive('orderBy')->once()->with('migration', 'asc')->andReturn($query);
 		$query->shouldReceive('lists')->once()->with('migration')->andReturn('bar');
 
 		$this->assertEquals('bar', $repo->getRan());

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ public function __construct(Resolver $resolver, $table)`
`45`	`45`	`*/`
`46`	`46`	`public function getRan()`
`47`	`47`	`{`
`48`		`- return $this->table()->lists('migration');`
	`48`	`+ return $this->table()->orderBy('migration', 'asc')->lists('migration');`
`49`	`49`	`}`
`50`	`50`
`51`	`51`	`/**`