Просмотр исходного кода

Add a test for ESCRIPT_ZIP_FILE and abspath the value

This way we can just configure it to a local folder
without having to use $(CURDIR).
Loïc Hoguin 1 год назад
Родитель
Сommit
f368ac4fc7
2 измененных файлов с 43 добавлено и 6 удалено
  1. 6 6
      plugins/escript.mk
  2. 37 0
      test/plugin_escript.mk

+ 6 - 6
plugins/escript.mk

@@ -29,11 +29,11 @@ help::
 
 escript-zip:: FULL=1
 escript-zip:: deps app
-	$(verbose) mkdir -p $(dir $(ESCRIPT_ZIP_FILE))
-	$(verbose) rm -f $(ESCRIPT_ZIP_FILE)
-	$(gen_verbose) cd .. && $(ESCRIPT_ZIP) $(ESCRIPT_ZIP_FILE) $(PROJECT)/ebin/*
+	$(verbose) mkdir -p $(dir $(abspath $(ESCRIPT_ZIP_FILE)))
+	$(verbose) rm -f $(abspath $(ESCRIPT_ZIP_FILE))
+	$(gen_verbose) cd .. && $(ESCRIPT_ZIP) $(abspath $(ESCRIPT_ZIP_FILE)) $(PROJECT)/ebin/*
 ifneq ($(DEPS),)
-	$(verbose) cd $(DEPS_DIR) && $(ESCRIPT_ZIP) $(ESCRIPT_ZIP_FILE) \
+	$(verbose) cd $(DEPS_DIR) && $(ESCRIPT_ZIP) $(abspath $(ESCRIPT_ZIP_FILE)) \
 		$(subst $(DEPS_DIR)/,,$(addsuffix /*,$(wildcard \
 			$(addsuffix /ebin,$(shell cat $(ERLANG_MK_TMP)/deps.log)))))
 endif
@@ -43,8 +43,8 @@ escript:: escript-zip
 		"#!$(ESCRIPT_SHEBANG)" \
 		"%% $(ESCRIPT_COMMENT)" \
 		"%%! $(ESCRIPT_EMU_ARGS)" > $(ESCRIPT_FILE)
-	$(verbose) cat $(ESCRIPT_ZIP_FILE) >> $(ESCRIPT_FILE)
+	$(verbose) cat $(abspath $(ESCRIPT_ZIP_FILE)) >> $(ESCRIPT_FILE)
 	$(verbose) chmod +x $(ESCRIPT_FILE)
 
 distclean-escript:
-	$(gen_verbose) rm -f $(ESCRIPT_FILE)
+	$(gen_verbose) rm -f $(ESCRIPT_FILE) $(abspath $(ESCRIPT_ZIP_FILE))

+ 37 - 0
test/plugin_escript.mk

@@ -175,3 +175,40 @@ escript-extra: init
 	$i "Check that the escript contains the extra files"
 	$t unzip -l $(APP)/$(APP) 2> /dev/null | grep -q Makefile
 	$t unzip -l $(APP)/$(APP) 2> /dev/null | grep -q erlang.mk
+
+escript-zip-file: init
+
+	$i "Bootstrap a new OTP library named $(APP)"
+	$t mkdir $(APP)/
+	$t cp ../erlang.mk $(APP)/
+	$t $(MAKE) -C $(APP) -f erlang.mk bootstrap-lib $v
+
+	$i "Set ESCRIPT_ZIP_FILE to a custom location"
+	$t perl -ni.bak -e 'print;if ($$.==1) {print "ESCRIPT_ZIP_FILE = tmp/my_escript.zip\n"}' $(APP)/Makefile
+
+	$i "Generate a module containing a function main/1"
+	$t printf "%s\n" \
+		"-module($(APP))." \
+		"-export([main/1])." \
+		'main(_) -> io:format("good~n").' > $(APP)/src/$(APP).erl
+
+	$i "Build the escript"
+	$t $(MAKE) -C $(APP) escript $v
+
+	$i "Check that the file at ESCRIPT_ZIP_FILE exists"
+	$t test -f $(APP)/tmp/my_escript.zip
+
+	$i "Check that the escript exists"
+	$t test -f $(APP)/$(APP)
+
+	$i "Check that the escript runs"
+	$t $(APP)/$(APP) | grep -q good
+
+	$i "Distclean the application"
+	$t $(MAKE) -C $(APP) distclean $v
+
+	$i "Check that the file at ESCRIPT_ZIP_FILE was removed"
+	$t test ! -e $(APP)/tmp/my_escript.zip
+
+	$i "Check that the escript was removed"
+	$t test ! -e $(APP)/$(APP)