[git commit] unit-tests: remove code depending on WANT_TIMING
Denys Vlasenko
vda.linux at googlemail.com
Mon Aug 17 15:01:49 UTC 2015
commit: http://git.busybox.net/busybox/commit/?id=718e228adcd7b9592b504b55217d13f4216a0450
branch: http://git.busybox.net/busybox/commit/?id=refs/heads/master
Since there is no interest in merging a config option for WANT_TIMING,
remove the parts of code depending on it altogether.
While we're at it: add some newlines to improve readability.
Signed-off-by: Bartosz Golaszewski <bartekgola at gmail.com>
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
libbb/bbunit.c | 30 +++---------------------------
1 files changed, 3 insertions(+), 27 deletions(-)
diff --git a/libbb/bbunit.c b/libbb/bbunit.c
index 66a7df9..db67b10 100644
--- a/libbb/bbunit.c
+++ b/libbb/bbunit.c
@@ -17,8 +17,6 @@
#include "libbb.h"
-#define WANT_TIMING 0
-
static llist_t *tests = NULL;
static unsigned tests_registered = 0;
static int test_retval;
@@ -34,38 +32,22 @@ void bbunit_settestfailed(void)
test_retval = -1;
}
-#if WANT_TIMING
-static void timeval_diff(struct timeval* res,
- const struct timeval* x,
- const struct timeval* y)
-{
- long udiff = x->tv_usec - y->tv_usec;
-
- res->tv_sec = x->tv_sec - y->tv_sec - (udiff < 0);
- res->tv_usec = (udiff >= 0 ? udiff : udiff + 1000000);
-}
-#endif
-
int unit_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM) MAIN_EXTERNALLY_VISIBLE;
int unit_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
{
unsigned tests_run = 0;
unsigned tests_failed = 0;
-#if WANT_TIMING
- struct timeval begin;
- struct timeval end;
- struct timeval time_spent;
- gettimeofday(&begin, NULL);
-#endif
bb_error_msg("Running %d test(s)...", tests_registered);
for (;;) {
struct bbunit_listelem* el = llist_pop(&tests);
if (!el)
break;
+
bb_error_msg("Case: [%s]", el->name);
test_retval = 0;
el->testfunc();
+
if (test_retval < 0) {
bb_error_msg("[ERROR] [%s]: TEST FAILED", el->name);
tests_failed++;
@@ -73,17 +55,11 @@ int unit_main(int argc UNUSED_PARAM, char **argv UNUSED_PARAM)
tests_run++;
}
-#if WANT_TIMING
- gettimeofday(&end, NULL);
- timeval_diff(&time_spent, &end, &begin);
- bb_error_msg("Elapsed time %u.%06u seconds",
- (int)time_spent.tv_sec,
- (int)time_spent.tv_usec);
-#endif
if (tests_failed > 0) {
bb_error_msg("[ERROR] %u test(s) FAILED", tests_failed);
return EXIT_FAILURE;
}
+
bb_error_msg("All tests passed");
return EXIT_SUCCESS;
}
More information about the busybox-cvs
mailing list