Merge pull request #505 from Clever/INFRANG-6880

Infrang 6880

Author: ChrisScotMartin

.circleci/config.yml

@@ -26,14 +26,10 @@ jobs:
       - run: make install_deps
       - run: make build
       - run: sudo npm install -g jsdoc-to-markdown@^4.0.0
-      # We are not running tests in CI for now as many of them are failing.
-      # We will enable this once we have fixed the tests.
-      # Many of these may be bad tests, but we need to investigate.
-      # Please run tests locally before merging.
-      # - run: make test
       - run:
           name: vet
           command: go vet -mod=readonly $(go list ./...)
+      - run: make test
       - run: if [ "${CIRCLE_BRANCH}" == "master" ]; then make release && $HOME/ci-scripts/circleci/submodule-github-release $GH_RELEASE_TOKEN clientconfig; fi;
       - run: if [ "${CIRCLE_BRANCH}" == "master" ]; then make release && $HOME/ci-scripts/circleci/submodule-github-release $GH_RELEASE_TOKEN logging/wagclientlogger; fi;
       - run: if [ "${CIRCLE_BRANCH}" == "master" ]; then make release && $HOME/ci-scripts/circleci/github-release $GH_RELEASE_TOKEN release; fi;

Makefile

@@ -20,6 +20,9 @@ build: go-generate
 
 test: build generate $(PKGS) js-tests
 	$(MAKE) -C samples test
+	echo "Currently DB tests are disabled because they are failing and aren't able to prioritize" \
+	"invesigating and fixing them. Remove t.Skip() from server/gendb/dynamodb_test.go.tmpl to re-enable" \
+	"https://clever.atlassian.net/browse/INFRANG-6880"
 
 js-tests:
 	cd samples/gen-js && rm -rf node_modules && npm install