feat: Update test targets

Signed-off-by: Saalim Quadri <danascape@gmail.com>

Author: danascape

Makefile

@@ -100,14 +100,11 @@ _uninstall-user:
 
 # Development targets
 
-check-scripts:
-	while read -r script; do shellcheck --exclude=SC1090,SC1091 $$script; done < tests/files
-
-execute-shfmt:
+format:
 	while read -r script; do shfmt -ln=bash -fn -ci -sr -w $$script; done < tests/files
 
-test: check-scripts
-	@echo "All tests passed"
+tests:
+	@./tests/run_tests.sh
 
 docs:
 	$(MAKE) -C docs html
@@ -139,9 +136,8 @@ help:
 	@echo "  sudo make uninstall Remove system installation"
 	@echo ""
 	@echo "Development:"
-	@echo "  make check-scripts  Run shellcheck on all scripts"
-	@echo "  make execute-shfmt  Format scripts with shfmt"
-	@echo "  make test           Run all tests"
+	@echo "  make tests          Run all tests"
+	@echo "  make format         Format scripts with shfmt"
 	@echo "  make docs           Build documentation"
 	@echo "  make docs-clean     Clean documentation build"
 	@echo ""
@@ -152,5 +148,5 @@ help:
 .PHONY: install uninstall
 .PHONY: _install-system _install-system-bin _install-system-src _install-system-utils _install-system-configs _install-system-man
 .PHONY: _uninstall-system _install-user _uninstall-user
-.PHONY: check-scripts execute-shfmt test docs docs-clean
+.PHONY: format tests docs docs-clean
 .PHONY: dist clean help

README.md

@@ -149,9 +149,8 @@ The package will be created in the parent directory.
 ## Development
 
 ```bash
-make check-scripts  # Run shellcheck
-make execute-shfmt  # Format scripts
-make test           # Run all tests
+make tests          # Run all tests
+make format         # Format scripts
 make docs           # Build documentation
 make docs-clean     # Clean documentation build
 make help           # Show all targets

tests/run_tests.sh

@@ -1,6 +1,20 @@
 #!/usr/bin/env bash
 
 #
-# Copyright (C) 2019 Saalim Quadri <danascape@gmail.com>
+# SPDX-FileCopyrightText: 2019 Saalim Quadri <danascape@gmail.com>
 # SPDX-License-Identifier: GPL-3.0-or-later
 #
+
+set -e
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+PROJECT_ROOT="$(dirname "$SCRIPT_DIR")"
+
+echo "Running ShellCheck"
+while read -r script; do
+	echo "Checking: $script"
+	shellcheck --exclude=SC1090,SC1091 "$PROJECT_ROOT/$script"
+done < "$SCRIPT_DIR/files"
+
+echo ""
+echo "All tests passed"