Browse Source

headless-browser: Ensure test path is always absolute

Previously, if you ran with a relative path, then everything would run
fine except that the test name would be blank in the output, eg:

  1/1234:

instead of:

  1/1234: Text/input/canvas/export.html
Sam Atkins 9 months ago
parent
commit
260074af87
1 changed files with 2 additions and 0 deletions
  1. 2 0
      Userland/Utilities/headless-browser.cpp

+ 2 - 0
Userland/Utilities/headless-browser.cpp

@@ -666,6 +666,8 @@ ErrorOr<int> serenity_main(Main::Arguments arguments)
         if (!app->test_dry_run)
         if (!app->test_dry_run)
             view = TRY(HeadlessWebContentView::create(move(theme), window_size, app->resources_folder));
             view = TRY(HeadlessWebContentView::create(move(theme), window_size, app->resources_folder));
 
 
+        auto absolute_test_root_path = LexicalPath::absolute_path(TRY(FileSystem::current_working_directory()), app->test_root_path);
+        app->test_root_path = absolute_test_root_path;
         auto test_glob = ByteString::formatted("*{}*", app->test_glob);
         auto test_glob = ByteString::formatted("*{}*", app->test_glob);
         return run_tests(view, app->test_root_path, test_glob, app->dump_failed_ref_tests, app->dump_gc_graph, app->test_dry_run);
         return run_tests(view, app->test_root_path, test_glob, app->dump_failed_ref_tests, app->dump_gc_graph, app->test_dry_run);
     }
     }