[or-cvs] r17677: {tor} Make unit tests slower but more reliable by disabling an old (tor/trunk/src/or)

[nickm@xxxxxxxx]

Author: nickm
Date: 2008-12-18 01:12:19 -0500 (Thu, 18 Dec 2008)
New Revision: 17677

Modified:
   tor/trunk/src/or/test.c
Log:
Make unit tests slower but more reliable by disabling an old optimization.
To turn it back on, re-define CACHE_GENERATED_KEYS option in test.c.

Modified: tor/trunk/src/or/test.c
===================================================================
--- tor/trunk/src/or/test.c	2008-12-18 06:02:23 UTC (rev 17676)
+++ tor/trunk/src/or/test.c	2008-12-18 06:12:19 UTC (rev 17677)
@@ -106,16 +106,27 @@
   rmdir(temp_dir);
 }
 
+/** Define this if unit tests spend too much time generating public keys*/
+#undef CACHE_GENERATED_KEYS
+
 static crypto_pk_env_t *pregen_keys[5] = {NULL, NULL, NULL, NULL, NULL};
 static crypto_pk_env_t *
 pk_generate(int idx)
 {
+#ifdef CACHE_GENERATED_KEYS
   tor_assert(idx < (int)(sizeof(pregen_keys)/sizeof(pregen_keys[0])));
   if (! pregen_keys[idx]) {
     pregen_keys[idx] = crypto_new_pk_env();
     tor_assert(!crypto_pk_generate_key(pregen_keys[idx]));
   }
   return crypto_pk_dup_key(pregen_keys[idx]);
+#else
+  crypto_pk_env_t *result;
+  (void) idx;
+  result = crypto_new_pk_env();
+  tor_assert(!crypto_pk_generate_key(result));
+  return result;
+#endif
 }
 
 static void