From c7be400b2da27c9855fa0136a3baf2cc89e4b689 Mon Sep 17 00:00:00 2001 From: Max Jacobson Date: Mon, 24 Apr 2017 13:08:25 -0400 Subject: [PATCH] Fix `make test` and `make integration` We're hitting errors where the `mocha` command is not available. I believe this is because we were running `yarn` in `/usr/src/app`, and then trying to run tests in `/code`, where the node_modules aren't available. For a sec, I assumed this was related to https://github.com/codeclimate/codeclimate-eslint/pull/256 but I confirmed the issue predates that change. --- Makefile | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index d628ef211..d6b7b7ee7 100644 --- a/Makefile +++ b/Makefile @@ -16,14 +16,12 @@ image: integration: image docker run -ti --rm \ - --volume $(PWD):/code \ - --workdir /code \ + --workdir /usr/src/app \ $(IMAGE_NAME) npm run $(NPM_INTEGRATION_TARGET) test: image docker run -ti --rm \ - --volume $(PWD):/code \ - --workdir /code \ + --workdir /usr/src/app \ $(IMAGE_NAME) npm run $(NPM_TEST_TARGET) citest: