Skip to content

Commit

Permalink
Add new make target 'integration', have 'test' exclude them
Browse files Browse the repository at this point in the history
The rationale for this new target is that for ponylang#28, we'd probably have
these tests depend on git being installed; and 'make test' should still
"just work" for everyone.

Also, this is differentiating test cases by their _names_, not labels.
This is because I haven't found a way to _exclude_ based on a label, to
have `make test` not run any of the integration tests. (Short of adding
label=unit to all the unit tests, which would of course be another
option.)

Signed-off-by: Stephan Renatus <[email protected]>
  • Loading branch information
srenatus committed Jun 14, 2018
1 parent e7a259c commit 7d1e34b
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,11 @@ install: $(binary)
$(tests_binary): $(GEN_FILES) $(SOURCE_FILES) | $(BUILD_DIR)
${PONYC} $(arch_arg) --debug -o ${BUILD_DIR} $(SRC_DIR)/test

integration: $(binary) $(tests_binary)
$(tests_binary) --only=integration

test: $(tests_binary)
$^
$^ --exclude=integration

clean:
rm -rf $(BUILD_DIR)
Expand Down Expand Up @@ -90,4 +93,4 @@ endef

$(eval $(call EXPAND_DEPLOY))

.PHONY: all clean deploy install test
.PHONY: all clean deploy install test integration

0 comments on commit 7d1e34b

Please sign in to comment.