Skip to content

Commit

Permalink
test.mk: do not write checked files for output tests
Browse files Browse the repository at this point in the history
Since we have both human and json tests, they can clash, and create a
race during CI runs. We also do not need them.
  • Loading branch information
mtzguido committed Nov 6, 2024
1 parent 2e20e7e commit db1ff49
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion mk/test.mk
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ OUTPUT_DIR ?= _output
CACHE_DIR ?= _cache

FSTAR = $(FSTAR_EXE) $(SIL) \
--cache_checked_modules \
$(if $(NO_WRITE_CHECKED),,--cache_checked_modules) \
--odir $(OUTPUT_DIR) \
--cache_dir $(CACHE_DIR) \
--already_cached Prims,FStar \
Expand Down Expand Up @@ -78,18 +78,22 @@ endif
$(FSTAR) $<
touch -c $@

$(OUTPUT_DIR)/%.fst.output: NO_WRITE_CHECKED=1
$(OUTPUT_DIR)/%.fst.output: %.fst
$(call msg, "OUTPUT", $(basename $(notdir $@)))
$(FSTAR) --message_format human -f --print_expected_failures $< >$@ 2>&1

$(OUTPUT_DIR)/%.fsti.output: NO_WRITE_CHECKED=1
$(OUTPUT_DIR)/%.fsti.output: %.fsti
$(call msg, "OUTPUT", $(basename $(notdir $@)))
$(FSTAR) --message_format human -f --print_expected_failures $< >$@ 2>&1

$(OUTPUT_DIR)/%.fst.json_output: NO_WRITE_CHECKED=1
$(OUTPUT_DIR)/%.fst.json_output: %.fst
$(call msg, "JSONOUT", $(basename $(notdir $@)))
$(FSTAR) --message_format json -f --print_expected_failures $< >$@ 2>&1

$(OUTPUT_DIR)/%.fsti.json_output: NO_WRITE_CHECKED=1
$(OUTPUT_DIR)/%.fsti.json_output: %.fsti
$(call msg, "JSONOUT", $(basename $(notdir $@)))
$(FSTAR) --message_format json -f --print_expected_failures $< >$@ 2>&1
Expand Down

0 comments on commit db1ff49

Please sign in to comment.