[Gnucash-changes] add to-do notes

Linas Vepstas linas at cvs.gnucash.org
Mon Jul 5 23:46:41 EDT 2004


Log Message:
-----------
add to-do notes

Modified Files:
--------------
    gnucash/src/engine/test-core:
        test-engine-stuff.c

Revision Data
-------------
Index: test-engine-stuff.c
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/src/engine/test-core/test-engine-stuff.c,v
retrieving revision 1.71
retrieving revision 1.72
diff -Lsrc/engine/test-core/test-engine-stuff.c -Lsrc/engine/test-core/test-engine-stuff.c -u -r1.71 -r1.72
--- src/engine/test-core/test-engine-stuff.c
+++ src/engine/test-core/test-engine-stuff.c
@@ -4,6 +4,18 @@
  * Create transactions with random values, random accounts, random
  * account heirarchies, etc.
  *
+ * XXX see notes below for the 'add_random_split()', which state:
+ * "this routine creates a random, but otherwise self-consistent,
+ * 'legal' transaction.  It should really be suplemented with another
+ * routine that creates cruddy, inconsistent transactions, so that the
+ * engine 'scrub' routines get tested. "
+ *
+ * XXX We should really modify that routine to create really, ugly, dirty
+ * transactions -- 3 or more splits, some without parent accounts, 
+ * other splits that have accounts but aren't in a transaction, 
+ * splits that share a currency with the transaction, but whose
+ * value doesn't equal amount, etc.
+ *
  * Created by Linux Developers Group, 2001
  * Updates Linas Vepstas July 2004
  */
@@ -1002,6 +1014,11 @@
     }
 }
 
+/* XXX this routine creates a random, but otherwise self-consistent,
+ * 'legal' transaction.  It should really be suplemented with another
+ * routine that creates cruddy, inconsistent transactions, so that the
+ * engine 'scrub' routines get tested.
+ */
 static void
 add_random_splits(QofBook *book, Transaction *trn, GList *account_list)
 {


More information about the gnucash-changes mailing list