[Gnucash-changes] tweak messages

Linas Vepstas linas at cvs.gnucash.org
Thu Jul 8 23:13:53 EDT 2004


Log Message:
-----------
tweak messages

Modified Files:
--------------
    gnucash/src/engine/test:
        test-transaction-reversal.c

Revision Data
-------------
Index: test-transaction-reversal.c
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/src/engine/test/test-transaction-reversal.c,v
retrieving revision 1.4
retrieving revision 1.5
diff -Lsrc/engine/test/test-transaction-reversal.c -Lsrc/engine/test/test-transaction-reversal.c -u -r1.4 -r1.5
--- src/engine/test/test-transaction-reversal.c
+++ src/engine/test/test-transaction-reversal.c
@@ -49,12 +49,15 @@
   if (!acc1 || !acc2)
   {
     failure("accounts not created");
+    return;
   }
 
+  /* Find a transaction that isn't voided */
   do
   {
     transaction = get_random_transaction (book);
-    if (xaccTransGetVoidStatus (transaction))
+    gboolean voyd = xaccTransGetVoidStatus (transaction);
+    if (voyd)
     {
       xaccTransBeginEdit (transaction);
       xaccTransDestroy (transaction);
@@ -62,13 +65,14 @@
       transaction = NULL;
     }
   } while (!transaction);
-
   transaction_set_splits_to_accounts(transaction, acc1, acc2);
   xaccTransSortSplits(transaction);
 
   new_trans = xaccTransClone(transaction);
   if (!xaccTransEqual(transaction, new_trans, FALSE, TRUE, FALSE, TRUE))
+  {
     failure("xaccTransClone failed.");
+  }
 
   xaccTransReverse(new_trans);
   for (i = 0; i < 2; i++) 
@@ -78,7 +82,7 @@
     result = gnc_numeric_add(old, new, GNC_DENOM_AUTO, GNC_DENOM_FIXED);
     if (gnc_numeric_eq(old, gnc_numeric_neg(new))) 
     {
-      msg = g_strdup_printf("Amount of split %d wrong after reversal", i);
+      msg = g_strdup_printf("Amount of split %d wrong after reversal\n", i);
       failure(msg);
     }
 
@@ -87,7 +91,7 @@
     result = gnc_numeric_add(old, new, GNC_DENOM_AUTO, GNC_DENOM_FIXED);
     if (gnc_numeric_eq(old, gnc_numeric_neg(new))) 
     {
-      msg = g_strdup_printf("Value of split %d wrong after reversal", i);
+      msg = g_strdup_printf("Value of split %d wrong after reversal\n", i);
       failure(msg);
     }
 
@@ -101,6 +105,7 @@
   gnc_module_load("gnucash/engine", 0);
 
   xaccLogDisable ();
+  set_success_print (TRUE);
 
   run_test ();
 


More information about the gnucash-changes mailing list