From 6f6d7696e56237a494af3decb00cee83c735c429 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20P=C3=A9dron?= Date: Thu, 22 Oct 2015 10:28:39 +0200 Subject: Replace "if $(MAKE) then false" by "! $(MAKE)" --- test/Makefile | 4 ++-- test/core_app.mk | 2 +- test/core_deps.mk | 10 +++++----- test/core_upgrade.mk | 8 ++++---- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/test/Makefile b/test/Makefile index 04b31af..4779c9f 100644 --- a/test/Makefile +++ b/test/Makefile @@ -348,7 +348,7 @@ ct: app1 "all() -> [testcase1]." \ "testcase1(_) -> 42 = m:succ(1)." \ > app1/test/failing_SUITE.erl - $t if $(MAKE) -C app1 ct-failing $v ; then false ; fi + $t ! $(MAKE) -C app1 ct-failing $v $i "Checking that '$(MAKE) distclean-ct' deletes logs." $t $(MAKE) -C app1 distclean-ct $v $t [ ! -e app1/logs ] @@ -396,7 +396,7 @@ eunit: app1 " ?assertEqual(42, t:succ(1))." \ > app1/eunit/t_tests.erl $t $(MAKE) -C app1 distclean TEST_DIR=eunit $v - $t if $(MAKE) -C app1 eunit TEST_DIR=eunit $v ; then false ; fi + $t ! $(MAKE) -C app1 eunit TEST_DIR=eunit $v $t rm -rf app1/eunit app1/src/t.erl app1/test-eunit.log $i "Test 'eunit' passed." diff --git a/test/core_app.mk b/test/core_app.mk index 14720df..0f1af19 100644 --- a/test/core_app.mk +++ b/test/core_app.mk @@ -300,7 +300,7 @@ core-app-error: build clean-core-app-error $t echo "-module(girl)." > $(APP)/src/girl.erl $i "Check that trying to build returns non-zero" - $t if $(MAKE) -C $(APP) $v; then false; fi + $t ! $(MAKE) -C $(APP) $v core-app-generate-erl: build clean-core-app-generate-erl diff --git a/test/core_deps.mk b/test/core_deps.mk index e2e5be7..415deb1 100644 --- a/test/core_deps.mk +++ b/test/core_deps.mk @@ -121,7 +121,7 @@ core-deps-apps-conflict: build clean-core-deps-apps-conflict $t $(MAKE) -C $(APP) new-lib in=cowlib $v $i "Check that building the application fails because of a conflict" - $t if $(MAKE) -C $(APP) $v; then false; fi + $t ! $(MAKE) -C $(APP) $v $i "Check that Cowlib wasn't fetched" $t test ! -e $(APP)/deps/cowlib @@ -140,7 +140,7 @@ core-deps-apps-deep-conflict: build clean-core-deps-apps-deep-conflict $t $(MAKE) -C $(APP) new-lib in=cowlib $v $i "Check that building the application fails because of a conflict" - $t if $(MAKE) -C $(APP) $v; then false; fi + $t ! $(MAKE) -C $(APP) $v $i "Check that Cowlib wasn't fetched" $t test ! -e $(APP)/deps/cowlib @@ -658,7 +658,7 @@ core-deps-fetch-fail-bad: build clean-core-deps-fetch-fail-bad $t perl -ni.bak -e 'print;if ($$.==1) {print "DEPS = cowlib\ndep_cowlib = oops https://github.com/ninenines/cowlib 1.0.0\n"}' $(APP)/Makefile $i "Check that building the application fails" - $t if $(MAKE) -C $(APP) $v; then false; fi + $t ! $(MAKE) -C $(APP) $v core-deps-fetch-fail-unknown: build clean-core-deps-fetch-fail-unknown @@ -671,7 +671,7 @@ core-deps-fetch-fail-unknown: build clean-core-deps-fetch-fail-unknown $t perl -ni.bak -e 'print;if ($$.==1) {print "DEPS = unknown\n"}' $(APP)/Makefile $i "Check that building the application fails" - $t if $(MAKE) -C $(APP) $v; then false; fi + $t ! $(MAKE) -C $(APP) $v core-deps-fetch-git: build clean-core-deps-fetch-git @@ -776,7 +776,7 @@ core-deps-fetch-legacy: build clean-core-deps-fetch-legacy $t perl -ni.bak -e 'print;if ($$.==1) {print "DEPS = cowlib\ndep_cowlib = https://github.com/ninenines/cowlib 1.0.0\n"}' $(APP)/Makefile $i "Check that building the application fails" - $t if $(MAKE) -C $(APP) $v; then false; fi + $t ! $(MAKE) -C $(APP) $v $i "Check that building the application works with IS_DEP=1" $t $(MAKE) -C $(APP) IS_DEP=1 $v diff --git a/test/core_upgrade.mk b/test/core_upgrade.mk index d7b1744..c4a8725 100644 --- a/test/core_upgrade.mk +++ b/test/core_upgrade.mk @@ -34,7 +34,7 @@ core-upgrade-custom-build-dir: build clean-core-upgrade-custom-build-dir $t ERLANG_MK_BUILD_DIR=custom $(MAKE) -C $(APP) erlang-mk $v $i "Check that the rule is gone" - $t if $(MAKE) -C $(APP) erlang_mk_upgrade_test_rule $v; then false; fi + $t ! $(MAKE) -C $(APP) erlang_mk_upgrade_test_rule $v $i "Check that the custom build directory is gone" $t test ! -d $(APP)/custom/ @@ -53,7 +53,7 @@ core-upgrade-custom-config: build clean-core-upgrade-custom-config $t $(MAKE) -C $(APP) erlang-mk $v $i "Check that the bootstrap plugin is gone" - $t if $(MAKE) -C $(APP) list-templates $v; then false; fi + $t ! $(MAKE) -C $(APP) list-templates $v core-upgrade-custom-repo: build clean-core-upgrade-custom-repo @@ -97,7 +97,7 @@ core-upgrade-no-config: build clean-core-upgrade-no-config $t $(MAKE) -C $(APP) erlang-mk $v $i "Check that the rule is gone" - $t if $(MAKE) -C $(APP) erlang_mk_upgrade_test_rule $v; then false; fi + $t ! $(MAKE) -C $(APP) erlang_mk_upgrade_test_rule $v core-upgrade-renamed-config: build clean-core-upgrade-renamed-config @@ -116,4 +116,4 @@ core-upgrade-renamed-config: build clean-core-upgrade-renamed-config $t $(MAKE) -C $(APP) erlang-mk $v $i "Check that the bootstrap plugin is gone" - $t if $(MAKE) -C $(APP) list-templates $v; then false; fi + $t ! $(MAKE) -C $(APP) list-templates $v -- cgit v1.2.3