darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-07 16:28:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

Separate test and lint steps

Browse Source
This commit is contained in:
Ryan Ahearn
2022-10-27 11:10:13 -04:00
parent f9cacac204
commit 98b772f959
3 changed files with 19 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
Makefile
View File

@@ -58,12 +58,25 @@ generate-version-file: ## Generates the app version file
@echo -e "__git_commit__ = \"${GIT_COMMIT}\"\n__time__ = \"${DATE}\"" > ${APP_VERSION_FILE}
.PHONY: test
test: ## Run tests
test: py-lint py-test js-test ## Run tests
.PHONY: py-lint
py-lint: ## Run python linting scanners
flake8 .
isort --check-only ./app ./tests
source $(NVMSH) && npm test
.PHONY: py-test
py-test: ## Run python unit tests
py.test -n auto --maxfail=10 tests/
.PHONY: js-lint
js-lint: ## Run javascript linting scanners
source $(NVMSH) && npm run lint
.PHONY: js-test
js-test: ## Run javascript unit tests
source $(NVMSH) && npm test
.PHONY: fix-imports
fix-imports: ## Fix imports using isort
isort ./app ./tests