Skip to content

Commit

Permalink
update Makefile to include new changes to test suite
Browse files Browse the repository at this point in the history
  • Loading branch information
bgunnar5 committed Nov 12, 2024
1 parent 6dab66b commit dcefd22
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 13 deletions.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,11 +11,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
- New test condition `StepFinishedFilesCount` to help search for `MERLIN_FINISHED` files in output workspaces
- Added "Unit-tests" GitHub action to run the unit test suite
- Added `CeleryTaskManager` context manager to the test suite to ensure tasks are safely purged from queues if tests fail
- Added `command-tests`, `workflow-tests`, and `integration-tests` to the Makefile

### Changed
- Ported all distributed tests of the integration test suite to pytest
- There is now a `commands/` directory and a `workflows/` directory under the integration suite to house these tests
- Removed the "Distributed-tests" GitHub action as these tests will now be run under "Integration-tests"
- Removed `e2e-distributed*` definitions from the Makefile

## [1.12.2]
### Added
Expand Down
29 changes: 16 additions & 13 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -34,12 +34,13 @@ include config.mk
.PHONY : install-workflow-deps
.PHONY : install-dev
.PHONY : unit-tests
.PHONY : command-tests
.PHONY : workflow-tests
.PHONY : integration-tests
.PHONY : e2e-tests
.PHONY : e2e-tests-diagnostic
.PHONY : e2e-tests-local
.PHONY : e2e-tests-local-diagnostic
.PHONY : e2e-tests-distributed
.PHONY : e2e-tests-distributed-diagnostic
.PHONY : tests
.PHONY : check-flake8
.PHONY : check-black
Expand Down Expand Up @@ -89,6 +90,18 @@ unit-tests:
. $(VENV)/bin/activate; \
$(PYTHON) -m pytest -v --order-scope=module $(UNIT); \

command-tests:
. $(VENV)/bin/activate; \
$(PYTHON) -m pytest -v $(TEST)/integration/commands/; \


workflow-tests:
. $(VENV)/bin/activate; \
$(PYTHON) -m pytest -v $(TEST)/integration/workflows/; \


integration-tests: command-tests workflow-tests


# run CLI tests - these require an active install of merlin in a venv
e2e-tests:
Expand All @@ -111,18 +124,8 @@ e2e-tests-local-diagnostic:
$(PYTHON) $(TEST)/integration/run_tests.py --local --verbose


e2e-tests-distributed:
. $(VENV)/bin/activate; \
$(PYTHON) $(TEST)/integration/run_tests.py --distributed; \


e2e-tests-distributed-diagnostic:
. $(VENV)/bin/activate; \
$(PYTHON) $(TEST)/integration/run_tests.py --distributed --verbose


# run unit and CLI tests
tests: unit-tests e2e-tests
tests: unit-tests integration-tests e2e-tests


check-flake8:
Expand Down

0 comments on commit dcefd22

Please sign in to comment.