Fixed: editing account deletes transactions

Fixes codinguser#351

app/src/androidTest/java/org/gnucash/android/test/ui/AccountsActivityTest.java

@@ -37,6 +37,8 @@
 import org.gnucash.android.model.Account;
 import org.gnucash.android.model.AccountType;
 import org.gnucash.android.model.Money;
+import org.gnucash.android.model.Split;
+import org.gnucash.android.model.Transaction;
 import org.gnucash.android.receivers.AccountCreator;
 import org.gnucash.android.ui.account.AccountsActivity;
 import org.gnucash.android.ui.account.AccountsListFragment;
@@ -45,6 +47,7 @@
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
+import java.math.BigDecimal;
 import java.util.Currency;
 import java.util.List;
 
@@ -71,6 +74,7 @@
 @RunWith(AndroidJUnit4.class)
 public class AccountsActivityTest extends ActivityInstrumentationTestCase2<AccountsActivity> {
 	private static final String DUMMY_ACCOUNT_CURRENCY_CODE = "USD";
+    private static final Currency DUMMY_ACCOUNT_CURRENCY = Currency.getInstance(DUMMY_ACCOUNT_CURRENCY_CODE);
 	private static final String DUMMY_ACCOUNT_NAME = "Dummy account";
     public static final String  DUMMY_ACCOUNT_UID   = "dummy-account";
     private DatabaseHelper mDbHelper;
@@ -226,7 +230,7 @@ public void testChangeParentAccount() {
      */
     @Test
     public void shouldHideParentAccountViewWhenNoParentsExist(){
-        onView(withText(DUMMY_ACCOUNT_NAME)).perform(click());
+        onView(allOf(withText(DUMMY_ACCOUNT_NAME), isDisplayed())).perform(click());
         onView(withId(R.id.fragment_transaction_list)).perform(swipeRight());
         onView(withText(R.string.label_create_account)).check(matches(isDisplayed())).perform(click());
         sleep(1000);
@@ -264,6 +268,31 @@ public void testEditAccount(){
         assertThat(latest.getCurrency().getCurrencyCode()).isEqualTo(DUMMY_ACCOUNT_CURRENCY_CODE);
 	}
 
+    @Test
+    public void editingAccountShouldNotDeleteTransactions(){
+        onView(allOf(withText(DUMMY_ACCOUNT_NAME), isDisplayed()))
+                .perform(longClick());
+        Account account = new Account("Transfer Account");
+
+        Transaction transaction = new Transaction("Simple trxn");
+        Split split = new Split(new Money(BigDecimal.TEN, DUMMY_ACCOUNT_CURRENCY), account.getUID());
+        transaction.addSplit(split);
+        transaction.addSplit(split.createPair(DUMMY_ACCOUNT_UID));
+        account.addTransaction(transaction);
+        mAccountsDbAdapter.addAccount(account);
+
+        assertThat(mAccountsDbAdapter.getAccount(DUMMY_ACCOUNT_UID).getTransactionCount()).isEqualTo(1);
+        assertThat(mSplitsDbAdapter.getSplitsForTransaction(transaction.getUID())).hasSize(2);
+
+        onView(withId(R.id.context_menu_edit_accounts)).perform(click());
+
+        onView(withId(R.id.menu_save)).perform(click());
+        assertThat(mAccountsDbAdapter.getAccount(DUMMY_ACCOUNT_UID).getTransactionCount()).isEqualTo(1);
+        assertThat(mSplitsDbAdapter.fetchSplitsForAccount(DUMMY_ACCOUNT_UID).getCount()).isEqualTo(1);
+        assertThat(mSplitsDbAdapter.getSplitsForTransaction(transaction.getUID())).hasSize(2);
+
+    }
+
     /**
      * Sleep the thread for a specified period
      * @param millis Duration to sleep in milliseconds

app/src/main/java/org/gnucash/android/db/AccountsDbAdapter.java

@@ -148,6 +148,7 @@ public long addAccount(Account account){
      * @return number of rows inserted
      */
     public long bulkAddAccounts(List<Account> accountList){
+        List<Transaction> transactionList = new ArrayList<>(accountList.size()*2);
         long nRow = 0;
         try {
             mDb.beginTransaction();
@@ -187,12 +188,17 @@ public long bulkAddAccounts(List<Account> accountList){
                 //Log.d(LOG_TAG, "Replacing account in db");
                 replaceStatement.execute();
                 nRow ++;
+                transactionList.addAll(account.getTransactions());
             }
             mDb.setTransactionSuccessful();
         }
         finally {
             mDb.endTransaction();
         }
+
+        if (nRow > 0 && !transactionList.isEmpty()){
+            mTransactionsAdapter.bulkAddTransactions(transactionList);
+        }
         return nRow;
     }
     /**

app/src/main/java/org/gnucash/android/ui/account/AccountFormFragment.java

@@ -30,6 +30,7 @@
 import android.support.v4.app.FragmentManager;
 import android.support.v4.widget.SimpleCursorAdapter;
 import android.text.TextUtils;
+import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.View;
 import android.view.ViewGroup;
@@ -701,8 +702,9 @@ public void onDestroy() {
      * Reads the fields from the account form and saves as a new account
      */
 	private void saveAccount() {
+        Log.i("AccountFormFragment", "Saving account");
         // accounts to update, in case we're updating full names of a sub account tree
-        ArrayList<Account> accountsToUpdate = new ArrayList<Account>();
+        ArrayList<Account> accountsToUpdate = new ArrayList<>();
         boolean nameChanged = false;
 		if (mAccount == null){
 			String name = getEnteredName();

app/src/test/java/org/gnucash/android/test/unit/db/AccountsDbAdapterTest.java

@@ -22,6 +22,7 @@
 import org.robolectric.annotation.Config;
 
 import java.math.BigDecimal;
+import java.util.ArrayList;
 import java.util.Currency;
 import java.util.List;
 
@@ -64,6 +65,30 @@ public void shouldBeAlphabeticallySortedByDefault(){
         assertThat(accountsList).contains(second, Index.atIndex(1));
 	}
 
+    @Test
+    public void bulkAddAccountsShouldNotModifyTransactions(){
+        Account account1 = new Account("AlphaAccount");
+        Account account2 = new Account("BetaAccount");
+        Transaction transaction = new Transaction("MyTransaction");
+        Split split = new Split(Money.getZeroInstance(), account1.getUID());
+        transaction.addSplit(split);
+        transaction.addSplit(split.createPair(account2.getUID()));
+        account1.addTransaction(transaction);
+        account2.addTransaction(transaction);
+
+        List<Account> accounts = new ArrayList<>();
+        accounts.add(account1);
+        accounts.add(account2);
+
+        mAccountsDbAdapter.bulkAddAccounts(accounts);
+
+        SplitsDbAdapter splitsDbAdapter = SplitsDbAdapter.getInstance();
+        assertThat(splitsDbAdapter.getSplitsForTransactionInAccount(transaction.getUID(), account1.getUID())).hasSize(1);
+        assertThat(splitsDbAdapter.getSplitsForTransactionInAccount(transaction.getUID(), account2.getUID())).hasSize(1);
+
+        assertThat(mAccountsDbAdapter.getAccount(account1.getUID()).getTransactions()).hasSize(1);
+    }
+
     @Test
     public void shouldAddAccountsToDatabase(){
         Account account1 = new Account("AlphaAccount");

app/src/test/java/org/gnucash/android/test/unit/util/GnucashTestRunner.java

@@ -2,9 +2,11 @@
 
 import org.junit.runners.model.InitializationError;
 import org.robolectric.RobolectricGradleTestRunner;
+import org.robolectric.annotation.Config;
 import org.robolectric.internal.bytecode.ClassInfo;
 import org.robolectric.internal.bytecode.InstrumentingClassLoaderConfig;
 import org.robolectric.internal.bytecode.ShadowMap;
+import org.robolectric.manifest.AndroidManifest;
 
 import java.util.Arrays;
 import java.util.Collections;
@@ -29,6 +31,11 @@ protected ShadowMap createShadowMap() {
                 .newBuilder().addShadowClass(ShadowCrashlytics.class).build();
     }
 
+    @Override
+    protected AndroidManifest getAppManifest(Config config) {
+        return super.getAppManifest(config);
+    }
+
     @Override
     public InstrumentingClassLoaderConfig createSetup() {
         return new InstrumenterConfig();