Jelajahi Sumber

test-js: Remove run-tests.sh

The shell script is no longer necessary -- simply run "test-js" from
inside Serenity, or $SERENITY_ROOT/Build/Meta/Lagom/test-js from the
host.
Matthew Olsson 5 tahun lalu
induk
melakukan
cf537311e4
3 mengubah file dengan 14 tambahan dan 21 penghapusan
  1. 2 2
      .travis.yml
  2. 0 16
      Libraries/LibJS/Tests/run-tests.sh
  3. 12 3
      Userland/test-js.cpp

+ 2 - 2
.travis.yml

@@ -43,7 +43,7 @@ script:
 - cmake .. -DBUILD_LAGOM=1
 - make
 - CTEST_OUTPUT_ON_FAILURE=1 make test
-- cd "$SERENITY_ROOT"/Libraries/LibJS/Tests
-- ./run-tests.sh
+- cd Meta/Lagom
+- ./test-js
 - cd "$SERENITY_ROOT"/Toolchain/Cache
 - du -ch * || true

+ 0 - 16
Libraries/LibJS/Tests/run-tests.sh

@@ -1,16 +0,0 @@
-#!/bin/bash
-
-if [ "$(uname)" = "SerenityOS" ]; then
-    js_program=/bin/test-js
-    test_root=/home/anon/js-tests
-else
-    [ -z "$js_program" ] && js_program="$SERENITY_ROOT/Build/Meta/Lagom/test-js"
-    test_root="$SERENITY_ROOT/Libraries/LibJS/Tests"
-
-    # Enable back traces if sanitizers are enabled
-    export UBSAN_OPTIONS=print_stacktrace=1
-fi
-
-$js_program "$test_root"
-
-exit $!

+ 12 - 3
Userland/test-js.cpp

@@ -515,9 +515,18 @@ void TestRunner::print_test_results() const
     printf("Time:        %-.3fs\n\n", m_end_time - m_start_time);
 }
 
-int main(int argc, char** argv)
+int main(int, char**)
 {
-    ASSERT(argc == 2);
-    TestRunner(argv[1]).run();
+#ifdef __serenity__
+    TestRunner("/home/anon/js-tests").run();
+#else
+    char* serenity_root = getenv("SERENITY_ROOT");
+    if (!serenity_root) {
+        printf("test-js requires the SERENITY_ROOT environment variable to be set");
+        return 1;
+    }
+    TestRunner(String::format("%s/Libraries/LibJS/Tests", serenity_root)).run();
+#endif
+
     return 0;
 }