When running a "test" or "assemble" target, run-tests now returns zero

[pintos-anon] / grading / vm / run-tests

diff --git a/grading/vm/run-tests b/grading/vm/run-tests
index c04faea87af0753035e93351e8fc0dd6681de766..8f9e635db421bcc0c9e1b7492a9f563a7c3d63ee 100755 (executable)
--- a/grading/vm/run-tests
+++ b/grading/vm/run-tests
@@ -19,14 +19,10 @@ our (@TESTS);               # Tests to run.
 our ($test);
 our ($action);
 
-parse_cmd_line ();
-
-# Default set of tests.
-@TESTS = qw (pt-grow-stack pt-big-stk-obj pt-bad-addr pt-write-code
-            page-linear page-parallel page-merge-seq page-merge-par
-            page-shuffle mmap-read mmap-close mmap-unmap mmap-overlap
-            mmap-twice mmap-write mmap-exit mmap-shuffle
-           ) unless @TESTS > 0;
+parse_cmd_line qw (pt-grow-stack pt-big-stk-obj pt-bad-addr pt-write-code
+                  page-linear page-parallel page-merge-seq page-merge-par
+                  page-shuffle mmap-read mmap-close mmap-unmap mmap-overlap
+                  mmap-twice mmap-write mmap-exit mmap-shuffle);
 
 clean_dir (), exit if $action eq 'clean';
 
@@ -39,10 +35,10 @@ exit if $action eq 'build';
 run_and_grade_tests (); 
 write_grades (); 
 write_details ();
-exit if $action eq 'test';
+exit success () if $action eq 'test';
 
 assemble_final_grade ();
-exit if $action eq 'assemble';
+exit success () if $action eq 'assemble';
 
 die "Don't know how to '$action'";