Delegate dev commands to scripts/dev.py

Author: AntoniaSzecsi

Makefile

@@ -5,11 +5,23 @@ target:
 
 .PHONY: init
 init:
-	poetry install
+	python scripts/dev.py init
 
 .PHONY: test
-test: check-format
-	poetry run pytest --cov awslambdaric --cov-report term-missing --cov-fail-under 90 tests
+test:
+	python scripts/dev.py test
+
+.PHONY: lint
+lint:
+	python scripts/dev.py lint
+
+.PHONY: clean
+clean:
+	python scripts/dev.py clean
+
+.PHONY: build
+build:
+	python scripts/dev.py build
 
 .PHONY: setup-codebuild-agent
 setup-codebuild-agent:
@@ -45,30 +57,21 @@ pr: init check-format check-security dev
 codebuild: setup-codebuild-agent
 	CODEBUILD_IMAGE_TAG=codebuild-agent DISTRO="$(DISTRO)" tests/integration/codebuild-local/test_all.sh tests/integration/codebuild
 
-.PHONY: clean
-clean:
-	rm -rf dist
-	rm -rf awslambdaric.egg-info
-	find . -type d -name "__pycache__" -exec rm -r {} +
-
-.PHONY: build
-build: clean
-	poetry build
-
 define HELP_MESSAGE
 
 Usage: $ make [TARGETS]
 
 TARGETS
 	check-security	Run bandit to find security issues.
 	format       	Run black to automatically update your code to match formatting.
-	build       	Builds the package with poetry.
-	clean       	Cleans the working directory by removing built artifacts.
-	dev         	Run all development tests after a change.
-	init        	Install dependencies via Poetry.
+	build       	Build the package using scripts/dev.py.
+	clean       	Cleans the working directory using scripts/dev.py.
+	dev         	Run all development tests using scripts/dev.py.
+	init        	Install dependencies via scripts/dev.py.
 	pr          	Perform all checks before submitting a Pull Request.
-	test        	Run the unit tests.
+	test        	Run unit tests using scripts/dev.py.
+	lint        	Run all linters via scripts/dev.py.
 	test-smoke  	Run smoke tests inside Docker.
 	test-integ  	Run all integration tests.
 
-endef
+endef

scripts/dev.py

@@ -0,0 +1,57 @@
+#!/usr/bin/env python3
+import argparse
+import subprocess
+import shutil
+import sys
+from pathlib import Path
+
+ROOT = Path(__file__).resolve().parent.parent
+
+
+def run(cmd, check=True):
+    print(f"\n$ {' '.join(cmd) if isinstance(cmd, list) else cmd}")
+    result = subprocess.run(cmd, shell=isinstance(cmd, str), check=check)
+    if result.returncode != 0 and check:
+        sys.exit(result.returncode)
+
+
+def init():
+    print("Initializing environment")
+    run(["poetry", "install"])
+
+
+def test():
+    print("Running tests")
+    run(["poetry", "run", "pytest", "tests"])
+
+
+def lint():
+    print("Running lint checks")
+    run(["poetry", "run", "flake8", "awslambdaric"])
+    run(["poetry", "run", "pylint", "awslambdaric"])
+    run(["poetry", "run", "black", "--check", "awslambdaric"])
+    run(["poetry", "run", "bandit", "-r", "awslambdaric"])
+
+
+def clean():
+    print("Cleaning build and cache files")
+    patterns = ["__pycache__", "*.pyc", "*.pyo", ".pytest_cache", ".mypy_cache"]
+    for pattern in patterns:
+        for path in ROOT.rglob(pattern):
+            if path.is_dir():
+                shutil.rmtree(path)
+            elif path.is_file():
+                path.unlink()
+
+
+def build():
+    print("Building package")
+    run(["poetry", "build"])
+
+
+if __name__ == "__main__":
+    parser = argparse.ArgumentParser(description="Development command-line tool")
+    parser.add_argument("command", choices=["init", "test", "lint", "clean", "build"])
+    args = parser.parse_args()
+
+    globals()[args.command]()