Add support for Alpine (Linux Distro)

.github/CONTRIBUTING.md

@@ -67,6 +67,33 @@ this will require to have installed [fswatcher](https://github.com/emcrisostomo/
 make test/watch
 ```
 
+### Testing different OS
+
+- Linux
+    - Ubuntu latest
+    - Alpine latest
+- Windows latest
+- MacOS latest
+
+#### Docker
+
+##### Ubuntu latest
+
+```bash
+docker run --rm -it -v "$(pwd)":/project -w /project ubuntu:latest \
+    sh -c  "apt-get update && \
+            apt-get install -y bash make shellcheck git curl perl && make test"
+```
+
+##### Alpine latest
+
+```bash
+make test/alpine
+# or
+docker run --rm -it -v "$(pwd)":/project -w /project alpine:latest \
+    sh -c  "apk add bash make shellcheck git curl perl && make test"
+```
+
 ## Coding Guidelines
 
 ### ShellCheck

.github/workflows/tests.yml

@@ -19,10 +19,10 @@ jobs:
         include:
           - os: windows-latest
             script_name: 'bash -c "./bashunit -e tests/globals.sh tests/**/*_test.sh"'
-          - os: ubuntu-latest
-            script_name: 'make test'
           - os: macos-latest
             script_name: 'make test'
+          - os: ubuntu-latest
+            script_name: 'make test'
     steps:
       - name: Checkout
         uses: actions/checkout@v4
@@ -32,3 +32,20 @@ jobs:
       - name: Run Tests
         run: ${{ matrix.script_name }}
 
+  alpine:
+    name: "Run tests on alpine-latest"
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+
+      - name: Run Tests
+        run: |
+          docker run --rm -v "$(pwd)":/project alpine:latest /bin/sh -c " \
+            apk update && \
+            apk add --no-cache bash make git curl perl && \
+            adduser -D builder && \
+            chown -R builder /project && \
+            su - builder -c 'cd /project; make test';"

CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## Unreleased
+## [Unreleased](https://github.com/TypedDevs/bashunit/compare/0.16.0...main)
 
-- Add optional file-path as 2nd arg to `--debug` option
-- Add runtime per test
-- Simplify total tests display on header
-- Rename `BASHUNIT_TESTS_ENV` to `BASHUNIT_LOAD_FILE`
+- Added support for Alpine (Linux Distro)
+- Added optional file-path as 2nd arg to `--debug` option
+- Added runtime per test
+- Simplified total tests display on header
+- Renamed `BASHUNIT_TESTS_ENV` to `BASHUNIT_LOAD_FILE`
 
 ## [0.16.0](https://github.com/TypedDevs/bashunit/compare/0.15.0...0.16.0) - 2024-09-15
 

Makefile

@@ -43,6 +43,7 @@ help:
 	@echo "  test                     Run the tests"
 	@echo "  test/list                List all tests under the tests directory"
 	@echo "  test/watch               Automatically run tests every second"
+	@echo "  test/alpine              Run the tests in a Linux/Alpine:latest image"
 	@echo "  env/example              Copy variables without the values from .env into .env.example"
 	@echo "  pre_commit/install       Install the pre-commit hook"
 	@echo "  pre_commit/run           Function that will be called when the pre-commit hook runs"
@@ -67,6 +68,10 @@ test/watch: $(TEST_SCRIPTS)
 	@./bashunit $(TEST_SCRIPTS)
 	@fswatch -m poll_monitor -or $(SRC_SCRIPTS_DIR) $(TEST_SCRIPTS_DIR) .env Makefile | xargs -n1 ./bashunit $(TEST_SCRIPTS)
 
+test/alpine:
+	@docker run --rm -it -v "$(shell pwd)":/project -w /project alpine:latest \
+		sh -c  "apk add bash make shellcheck git curl perl && make test"
+
 env/example:
 	@echo "Copying variables without the values from .env into .env.example"
 	@sed 's/=.*/=/' .env > .env.example

src/assert.sh

@@ -1,7 +1,7 @@
 #!/bin/bash
 
 function fail() {
-  local message=$1
+  local message="${1:-${FUNCNAME[1]}}"
 
   local label
   label="$(helper::normalize_test_function_name "${FUNCNAME[1]}")"

src/check_os.sh

@@ -2,11 +2,22 @@
 
 # shellcheck disable=SC2034
 _OS="Unknown"
+_DISTRO="Unknown"
 
 if [[ "$(uname)" == "Linux" ]]; then
   _OS="Linux"
+  if command -v apt > /dev/null; then
+    _DISTRO="Ubuntu"
+  elif command -v apk > /dev/null; then
+    _DISTRO="Alpine"
+  else
+    _DISTRO="Other"
+  fi
 elif [[ "$(uname)" == "Darwin" ]]; then
   _OS="OSX"
-elif [[ $(uname) == *"MINGW"* ]]; then
+elif [[ "$(uname)" == *"MINGW"* ]]; then
   _OS="Windows"
 fi
+
+export _OS
+export _DISTRO

src/clock.sh

@@ -2,7 +2,7 @@
 
 function clock::now() {
   if perl -MTime::HiRes -e "" > /dev/null 2>&1; then
-    perl -MTime::HiRes -e 'printf("%.0f\n",Time::HiRes::time()*1000)'
+    perl -MTime::HiRes -e 'printf("%.0f\n", Time::HiRes::time() * 1000)'
   elif [[ "${_OS:-}" != "OSX" ]]; then
     date +%s%N
   else

src/default_env_config.sh

@@ -14,4 +14,27 @@ _DEFAULT_BASHUNIT_LOAD_FILE=
 _DEFAULT_TERMINAL_WIDTH=150
 
 CAT="$(which cat)"
-TERMINAL_WIDTH=${TERM:+$(tput cols)} || TERMINAL_WIDTH=$_DEFAULT_TERMINAL_WIDTH
+
+function find_terminal_width() {
+  if [[ -n "$TERM" && $(command -v tput) ]]; then
+      _cols=$(tput cols 2>/dev/null) || _cols=""
+
+      # If tput fails, fallback to stty if available
+      if [[ -z "$_cols" ]] && command -v stty > /dev/null; then
+          _cols=$(stty size | cut -d' ' -f2)
+      fi
+  else
+      # Fallback to stty if TERM is not set and stty is available
+      if command -v stty > /dev/null; then
+          _cols=$(stty size | cut -d' ' -f2)
+      fi
+  fi
+
+  if [[ -z "$_cols" ]]; then
+      _cols=$_DEFAULT_TERMINAL_WIDTH
+  fi
+
+  echo "$_cols"
+}
+
+TERMINAL_WIDTH="$(find_terminal_width)"

src/helpers.sh

@@ -23,20 +23,18 @@ function helper::normalize_test_function_name() {
   echo "$result"
 }
 
+
 function helper::check_duplicate_functions() {
   local script="$1"
 
   local filtered_lines
-  filtered_lines=$(grep -E '^\s*(function)?\s*test[a-zA-Z_][a-zA-Z_0-9]*\s*\(\)?\s*{' "$script")
+  filtered_lines=$(grep -E '^[[:space:]]*(function[[:space:]]+)?test[a-zA-Z_][a-zA-Z0-9_]*\s*\(\)\s*\{' "$script")
 
   local function_names
   function_names=$(echo "$filtered_lines" | awk '{gsub(/\(|\)/, ""); print $2}')
 
-  local sorted_names
-  sorted_names=$(echo "$function_names" | sort)
-
   local duplicates
-  duplicates=$(echo "$sorted_names" | uniq -d)
+  duplicates=$(echo "$function_names" | sort | uniq -d)
   if [ -n "$duplicates" ]; then
     state::set_duplicated_functions_merged "$script" "$duplicates"
     return 1

tests/acceptance/fixtures/.env.default

@@ -6,3 +6,4 @@ BASHUNIT_SHOW_EXECUTION_TIME=false
 BASHUNIT_DEFAULT_PATH=
 BASHUNIT_LOG_JUNIT=
 BASHUNIT_REPORT_HTML=
+BASHUNIT_LOAD_FILE=

tests/acceptance/install_test.sh

@@ -56,6 +56,7 @@ function test_install_downloads_the_given_version() {
 
 function test_install_downloads_the_non_stable_beta_version() {
   mock date echo "2023-11-13"
+  mock tput echo ""
   local installed_bashunit="./deps/bashunit"
   local output
 

tests/functional/provider_test.sh

@@ -1,17 +1,6 @@
 #!/bin/bash
 set -euo pipefail
 
-_TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE=""
-
-function set_up_before_script() {
-  _TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE=$(mktemp)
-  echo 0 > "$_TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE"
-}
-
-function tear_down_after_script() {
-  rm "$_TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE"
-}
-
 function set_up() {
   _GLOBAL="aa-bb"
 }
@@ -29,31 +18,6 @@ function provide_multiples_values() {
   echo "aa" "bb"
 }
 
-# data_provider provide_single_values
-function test_single_values_from_data_provider() {
-  local current_data="$1"
-  local current_iteration=0
-
-  echo "$(awk 'BEGIN{FS=OFS=""} {$1++} {print $1}' "$_TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE")"\
-    > "$_TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE"
-  current_iteration=$(cat "$_TEST_GET_DATA_FROM_PROVIDER_ITERATION_FILE")
-
-  case $current_iteration in
-    1)
-      assert_same "one" "$current_data"
-      ;;
-    2)
-      assert_same "two" "$current_data"
-      ;;
-    3)
-      assert_same "three" "$current_data"
-      ;;
-    *)
-      fail
-      ;;
-  esac
-}
-
 function provide_single_values() {
   echo "one"
   echo "two"

tests/unit/clock_test.sh

@@ -20,13 +20,23 @@ function test_now_with_perl() {
   assert_same "1720705883457" "$(clock::now)"
 }
 
-function test_now_on_linux_without_perl() {
+function test_now_on_linux_unknown() {
   export _OS="Linux"
+  export _DISTRO="Unknown"
   mock perl mock_non_existing_fn
   mock date echo "1720705883457"
 
   assert_same "1720705883457" "$(clock::now)"
 }
+
+function test_now_on_linux_alpine() {
+  export _OS="Linux"
+  export _DISTRO="Alpine"
+  mock perl echo "1720705883457"
+
+  assert_same "1720705883457" "$(clock::now)"
+}
+
 function test_now_on_windows_without_perl() {
   export _OS="Windows"
   mock perl mock_non_existing_fn

tests/unit/directory_test.sh

@@ -127,7 +127,7 @@ function test_unsuccessful_assert_is_directory_readable_when_a_file_is_given() {
 }
 
 function test_unsuccessful_assert_is_directory_readable_without_execution_permission() {
-  if [[ "$_OS" == "Windows" ]]; then
+  if [[ "$_OS" == "Windows" || $_DISTRO = "Alpine" ]]; then
     return
   fi
 
@@ -145,7 +145,7 @@ function test_unsuccessful_assert_is_directory_readable_without_execution_permis
 }
 
 function test_unsuccessful_assert_is_directory_readable_without_read_permission() {
-  if [[ "$_OS" == "Windows" ]]; then
+  if [[ "$_OS" == "Windows" || $_DISTRO = "Alpine" ]]; then
       return
   fi
 
@@ -163,7 +163,7 @@ function test_unsuccessful_assert_is_directory_readable_without_read_permission(
 }
 
 function test_successful_assert_is_directory_not_readable_without_read_permission() {
-  if [[ "$_OS" == "Windows" ]]; then
+  if [[ "$_OS" == "Windows" || $_DISTRO = "Alpine" ]]; then
       return
   fi
 
@@ -177,7 +177,7 @@ function test_successful_assert_is_directory_not_readable_without_read_permissio
 }
 
 function test_successful_assert_is_directory_not_readable_without_execution_permission() {
-  if [[ "$_OS" == "Windows" ]]; then
+  if [[ "$_OS" == "Windows" || $_DISTRO = "Alpine" ]]; then
       return
   fi
 
@@ -212,7 +212,7 @@ function test_successful_assert_is_directory_writable() {
 }
 
 function test_unsuccessful_assert_is_directory_writable() {
-  if [[ "$_OS" == "Windows" ]]; then
+  if [[ "$_OS" == "Windows" || $_DISTRO = "Alpine" ]]; then
       return
   fi
 
@@ -240,7 +240,7 @@ function test_unsuccessful_assert_is_directory_writable_when_a_file_is_given() {
 }
 
 function test_successful_assert_is_directory_not_writable() {
-  if [[ "$_OS" == "Windows" ]]; then
+  if [[ "$_OS" == "Windows" || $_DISTRO = "Alpine" ]]; then
       return
   fi