Merge pull request #289 from bebehei/testsuite

Testsuite

diff --git a/.travis.yml b/.travis.yml
index fc58769edf7161955829bed9434a7c1a0774b5ae..71bc27954f4cc11c7a336556feb392a23445e0e9 100644 (file)
--- a/.travis.yml
+++ b/.travis.yml
@@ -22,4 +22,4 @@ addons:
 script:
   - make -j
   - clang-format-3.5 -i $(find . -name "*.[ch]" | tr '\n' ' ') && git diff --exit-code || (echo 'Code was not formatted using clang-format!'; false)
-  - ./travis/run-tests.pl
+  - make test

diff --git a/Makefile b/Makefile
index 86830ef49a2d811655d004ef495e2f126758cdfc..5ec587172475fc03fbc40e74558086722167753d 100644 (file)
--- a/Makefile
+++ b/Makefile
@@ -99,6 +99,9 @@ i3status: ${OBJS}
        $(CC) $(LDFLAGS) -o $@ $^ $(LIBS)
        @echo " LD $@"
 
+test: i3status
+       LC_ALL=C ./travis/run-tests.pl
+
 clean:
        rm -f *.o src/*.o
 

diff --git a/travis/run-tests.pl b/travis/run-tests.pl
index 5936b7eb1e2606cd248ddb92a12367ebfd0a1830..c335ff34e1b2afbfe99d9105e58359c470f5204a 100755 (executable)
--- a/travis/run-tests.pl
+++ b/travis/run-tests.pl
@@ -16,6 +16,7 @@ sub TestCase {
 
     my $conf = "$dir/i3status.conf";
     my $testres = `./i3status --run-once -c $conf`;
+    my $exitcode = $?;
     my $refres = "";
 
     if ( -f "@_/expected_output.txt") {
@@ -28,24 +29,33 @@ sub TestCase {
         system($EXECUTABLE_NAME, "@_/cleanup.pl", ($dir));
     }
 
+    if ( $exitcode != 0 ) {
+        say "Testing test case '", basename($dir), "'… ", BOLD, RED, "Crash!", RESET;
+        return 0;
+    }
+
     if ( "$testres" eq "$refres" ) {
         say "Testing test case '", basename($dir), "'… ", BOLD, GREEN, "OK", RESET;
         return 1;
     } else {
         say "Testing test case '", basename($dir), "'… ", BOLD, RED, "Failed!", RESET;
+        say "Expected: '$refres'";
+        say "Got: '$testres'";
         return 0;
     }
 }
 
 my $testcases = 'testcases';
-my $testresults = 1;
+my $testresults = 0;
 
 opendir(my $dir, $testcases) or die "Could not open directory $testcases: $!";
 
 while (my $entry = readdir($dir)) {
     next unless (-d "$testcases/$entry");
     next if ($entry =~ m/^\./);
-    $testresults = $testresults && TestCase("$testcases/$entry");
+    if (not TestCase("$testcases/$entry") ) {
+        $testresults = 1;
+    }
 }
 closedir($dir);
-exit 0;
+exit $testresults;