Improve CI tests runtime (halved)

.github/workflows/tests.yml

@@ -8,51 +8,47 @@ on:
 
 jobs:
   tests:
-    name: "Run tests on ${{ matrix.os }}"
+    name: "Tests on ${{ matrix.os }} (${{ matrix.test_chunk }})"
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
+        test_chunk: [acceptance, functional, unit]
         os:
           - ubuntu-latest
           - macos-latest
           - windows-latest
-        include:
-          - os: windows-latest
-            script_name: 'bash -c "./bashunit -e tests/bootstrap.sh tests/**/*_test.sh"'
-          - os: macos-latest
-            script_name: 'make test'
-          - os: ubuntu-latest
-            script_name: 'make test'
     steps:
       - name: Checkout
         uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
 
       - name: Run Tests
-        run: ${{ matrix.script_name }}
+        shell: bash
+        run: |
+          ./bashunit -e tests/bootstrap.sh tests/${{ matrix.test_chunk }}/*_test.sh
 
   simple-output:
-    name: "Run tests with simple output"
+    name: "Tests simple output"
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
 
       - name: Run Tests
         run: |
           ./bashunit -e tests/bootstrap.sh --simple tests/
 
   alpine:
-    name: "Run tests on alpine-latest"
+    name: "Tests on alpine-latest"
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
         uses: actions/checkout@v4
         with:
-          fetch-depth: 0
+          fetch-depth: 1
 
       - name: Run Tests
         run: |

CHANGELOG.md

@@ -16,6 +16,7 @@
     - cleanup_temp_files
     - log
 - Improved clean up temporal files and directories
+- Improved CI test speed by running them in parallel
 
 ## [0.17.0](https://github.com/TypedDevs/bashunit/compare/0.16.0...0.17.0) - 2024-10-01
 

src/console_results.sh

@@ -103,8 +103,8 @@ function console_results::print_execution_time() {
     return
   fi
 
-  _EXECUTION_TIME=$(clock::total_runtime_in_milliseconds)
-  printf "${_COLOR_BOLD}%s${_COLOR_DEFAULT}\n" "Time taken: ${_EXECUTION_TIME} ms"
+  printf "${_COLOR_BOLD}%s${_COLOR_DEFAULT}\n" \
+    "Time taken: $(clock::total_runtime_in_milliseconds) ms"
 }
 
 function console_results::print_successful_test() {

src/globals.sh

@@ -16,7 +16,9 @@ function current_timestamp() {
 }
 
 function is_command_available() {
-  command -v "$1" >/dev/null 2>&1
+  if ! command -v "$1" >/dev/null 2>&1; then
+    return 1
+  fi
 }
 
 function random_str() {

tests/unit/console_results_test.sh

@@ -273,7 +273,7 @@ function test_render_execution_time() {
 
     console_results::render_result
   )
-  assert_matches "Time taken: [[:digit:]]+ ms" "$render_result"
+  assert_matches "Time taken: [[:digit:]]+(\.[[:digit:]]+)? ms" "$render_result"
 }
 
 function test_not_render_execution_time() {
@@ -304,7 +304,7 @@ function test_render_execution_time_on_osx_without_perl() {
     console_results::render_result
   )
 
-  assert_matches "Time taken: [[:digit:]]+ ms" "$render_result"
+  assert_matches "Time taken: [[:digit:]]+(\.[[:digit:]]+)? ms" "$render_result"
 }
 
 function test_render_execution_time_on_osx_with_perl() {
@@ -326,7 +326,7 @@ function test_render_execution_time_on_osx_with_perl() {
     console_results::render_result
   )
 
-  assert_matches "Time taken: [[:digit:]]+ ms" "$render_result"
+  assert_matches "Time taken: [[:digit:]]+(\.[[:digit:]]+)? ms" "$render_result"
 }
 
 function test_render_file_with_duplicated_functions_if_found_true() {

tests/unit/globals_test.sh

@@ -25,11 +25,15 @@ function test_globals_current_timestamp() {
 }
 
 function test_globals_is_command_available() {
-  assert_successful_code "$(is_command_available ls)"
+  function existing_fn(){
+    # shellcheck disable=SC2317
+    return 0
+  }
+  assert_successful_code "$(is_command_available existing_fn)"
 }
 
 function test_globals_is_command_not_available() {
-  assert_general_error "$(is_command_available non-existing-command)"
+  assert_general_error "$(is_command_available non_existing_fn)"
 }
 
 function test_globals_random_str_default() {