[test] ignore smolder upload failures on make smoke

and smolder_fulltest. fixes GH #1040. Travis should now be useful again
parrot · Mar 5, 2014 · 42f2c4e · 42f2c4e
1 parent cea93bf
commit 42f2c4e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/t/fullharness b/t/fullharness
@@ -78,7 +78,12 @@ if ($archive) {
     } );
     my $overall_aggregator = $archive->runtests(\@targets);
     $archive->summary($overall_aggregator);
-    send_archive_to_smolder(%env_data) if $smolder;
+    # smolder is currently (2014) too unstable to accept most reports: GH #1040
+    # osuosl blames their 32bit VM
+    eval { send_archive_to_smolder(%env_data); } if $smolder;
+    warn $@ if $@;
+    my $exit_value = ($overall_aggregator->get_status() eq 'PASS') ? 0 : 1;
+    exit($exit_value);
 }
 else {
     require TAP::Formatter::Console;

diff --git a/t/harness b/t/harness
@@ -85,7 +85,10 @@ if ($longopts->{archive}) {
         }
     );
     my $results = $harness->runtests(@tests);
-    send_archive_to_smolder(%env_data) if $longopts->{send_to_smolder};
+    # smolder is currently (2014) too unstable to accept most reports: GH #1040
+    # osuosl blames their 32bit VM
+    eval { send_archive_to_smolder(%env_data); } if $longopts->{send_to_smolder};
+    warn $@ if $@;
     exit ( $results->all_passed() ? 0 : 1 );
 }
 elsif ($longopts->{html}) {