Skip to content

Commit

Permalink
test: fix test start behaviour
Browse files Browse the repository at this point in the history
Better first start and then get ip address ;-)

Signed-off-by: Silvio Fricke <[email protected]>
  • Loading branch information
silvio committed Dec 18, 2023
1 parent b385ac8 commit 90d560a
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -25,17 +25,17 @@ LanguageTool-$(VERSION).zip:
curl -L https://www.languagetool.org/download/LanguageTool-$(VERSION).zip -o LanguageTool-$(VERSION).zip

test: test-cleanup.1
test: test-start
test: TESTIPADDRESS=$(subst ",,$(shell docker inspect languagetool | jq '.[0].NetworkSettings.IPAddress'))
test: test-print-ip-address
test: test-start
test: test-run-test-lang
test: test-run-test-en
test: test-run-test-fr
test: test-cleanup.2

test-start:
docker run -d --name languagetool -p 8010:8010 $(IMAGENAME):latest
sleep 3
sleep 6

test-print-ip-address:
@echo "IP address of languagetools docker container: $(TESTIPADDRESS)"
Expand Down Expand Up @@ -63,8 +63,8 @@ test-run-test-fr:

.PHONY: test-cleanup
test-cleanup.%:
-docker stop languagetool
-docker rm languagetool
-docker container stop languagetool
-docker container rm languagetool

.PHONY: tag
tag: tag-push
Expand Down

0 comments on commit 90d560a

Please sign in to comment.