Fixed a bug where the sync migration test would only work the first time

astrid/plugin-src/com/todoroo/astrid/tags/AstridNewSyncMigrator.java

@@ -35,7 +35,7 @@ public class AstridNewSyncMigrator {
     @Autowired private UpdateDao updateDao;
     @Autowired private UserDao userDao;
 
-    private static final String PREF_SYNC_MIGRATION = "sync_migration";
+    public static final String PREF_SYNC_MIGRATION = "sync_migration";
 
     private static final String LOG_TAG = "sync-migrate";
 

tests-sync/src/com/todoroo/astrid/sync/AstridNewSyncMigrationTest.java

@@ -9,6 +9,7 @@ import com.todoroo.andlib.data.Table;
 import com.todoroo.andlib.data.TodorooCursor;
 import com.todoroo.andlib.service.Autowired;
 import com.todoroo.andlib.sql.Query;
+import com.todoroo.andlib.utility.Preferences;
 import com.todoroo.astrid.dao.MetadataDao;
 import com.todoroo.astrid.dao.MetadataDao.MetadataCriteria;
 import com.todoroo.astrid.dao.RemoteModelDao;
@@ -24,8 +25,9 @@ public class AstridNewSyncMigrationTest extends NewSyncTestCase {
 	@Autowired
 	private MetadataDao metadataDao;
 	
-	public void testAstrid44Migration() {
+	public void testAstridSyncMigration() {
 		setupOldDatabase();
+		Preferences.clear(AstridNewSyncMigrator.PREF_SYNC_MIGRATION);
 		new AstridNewSyncMigrator().performMigration();
 		assertAllModelsHaveUUID();
 		assertAllTagsHaveTagData();