Skip to content

Commit

Permalink
Simplify the unit test build step (#1769)
Browse files Browse the repository at this point in the history
Remove the buildTests script, since we can bundle the unit test files
with Browserify directly from the Makefile. In the future, we should
remove shared/content-scope-scripts entirely, but making that work
with Browserify is proving to be tricky.
  • Loading branch information
kzar committed Mar 9, 2023
1 parent 166f40d commit 48b4025
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 35 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ shared/content-scope-scripts/src/%: shared/content-scope-scripts
UNIT_TEST_SRC = unit-test/background/*.js unit-test/background/classes/*.js unit-test/background/events/*.js unit-test/background/storage/*.js unit-test/background/reference-tests/*.js
build/test/background.js: $(TEST_FILES) $(SOURCE_FILES) shared/content-scope-scripts
mkdir -p `dirname $@`
node ./scripts/buildTests.js $@
$(BROWSERIFY) -t brfs -t ./scripts/browserifyFileMapTransform $(UNIT_TEST_SRC) -o $@

build/test/ui.js: $(TEST_FILES)
$(BROWSERIFY) shared/js/ui/base/index.js unit-test/ui/**/*.js -o $@
Expand Down
34 changes: 0 additions & 34 deletions scripts/buildTests.js

This file was deleted.

0 comments on commit 48b4025

Please sign in to comment.