50 - If re-running a job with debug on, set log level to debug (#51)

- Added check of workflow debug regime.

Author: miroslavpojer

.github/workflows/build.yml

@@ -65,11 +65,18 @@ jobs:
           CHANGED_FILES=$(echo "$CHANGED_FILES" | awk '{printf "%s,", $0} END {print ""}' | sed 's/,$//')
           
           # Generate coverage report for changed files
-          CHANGED_FILES_REPORT=$(coverage report --include="$CHANGED_FILES")
-          echo "Changed Python files: $CHANGED_FILES_REPORT"
-
+          coverage report --include="$CHANGED_FILES" > coverage_report.txt
+          echo -e "\nChanged Python files report:\n\n"
+          cat coverage_report.txt
+          
           # Fail if the coverage for changed files is below threshold
-          coverage report --include="$CHANGED_FILES" | grep TOTAL | awk '{if ($4 < 80.0) exit 1}'
+          COVERAGE_TOTAL=$(awk '/TOTAL/ {print $4}' coverage_report.txt)
+          echo "Total coverage for changed files: $COVERAGE_TOTAL"
+
+          if (( $(echo "$COVERAGE_TOTAL < 80.0" | bc -l) )); then
+            echo "Coverage is below 80%"
+            exit 1
+          fi
 
       - name: Upload coverage report
         uses: actions/upload-artifact@v3

README.md

@@ -72,7 +72,7 @@ Generate Release Notes action is dedicated to enhance the quality and organizati
 - **Description**: Set to true to enable verbose logging for detailed output during the action's execution.
 - **Required**: No
 - **Default**: false
-
+- **Note**: If workflow run in debug regime, 'verbose' logging is activated.
 
 ## Outputs
 The output of the action is a markdown string containing the release notes for the specified tag. This string can be used in subsequent steps to publish the release notes to a file, create a GitHub release, or send notifications.

release_notes_generator/action_inputs.py

@@ -16,6 +16,7 @@
 
 import json
 import logging
+import os
 
 from release_notes_generator.utils.gh_action import get_action_input
 
@@ -31,6 +32,7 @@ class ActionInputs:
     PRINT_EMPTY_CHAPTERS = 'print-empty-chapters'
     CHAPTERS_TO_PR_WITHOUT_ISSUE = 'chapters-to-pr-without-issue'
     VERBOSE = 'verbose'
+    RUNNER_DEBUG = 'RUNNER_DEBUG'
 
     @staticmethod
     def get_github_repository() -> str:
@@ -70,7 +72,7 @@ def get_chapters_to_pr_without_issue() -> bool:
 
     @staticmethod
     def get_verbose() -> bool:
-        return get_action_input(ActionInputs.VERBOSE).lower() == 'true'
+        return os.getenv('RUNNER_DEBUG', 0) == 1 or get_action_input(ActionInputs.VERBOSE).lower() == 'true'
 
     @staticmethod
     def validate_inputs():

tests/test_action_inputs.py

@@ -125,6 +125,25 @@ def test_get_chapters_to_pr_without_issue(mocker):
     assert ActionInputs.get_chapters_to_pr_without_issue() is False
 
 
-def test_get_verbose(mocker):
+def test_get_verbose_verbose_by_action_input(mocker):
     mocker.patch('release_notes_generator.action_inputs.get_action_input', return_value='true')
+    mocker.patch('os.getenv', return_value=0)
     assert ActionInputs.get_verbose() is True
+
+
+def test_get_verbose_verbose_by_workflow_debug(mocker):
+    mocker.patch('release_notes_generator.action_inputs.get_action_input', return_value='false')
+    mocker.patch('os.getenv', return_value=1)
+    assert ActionInputs.get_verbose() is True
+
+
+def test_get_verbose_verbose_by_both(mocker):
+    mocker.patch('release_notes_generator.action_inputs.get_action_input', return_value='true')
+    mocker.patch('os.getenv', return_value=1)
+    assert ActionInputs.get_verbose() is True
+
+
+def test_get_verbose_not_verbose(mocker):
+    mocker.patch('release_notes_generator.action_inputs.get_action_input', return_value='false')
+    mocker.patch('os.getenv', return_value=0)
+    assert ActionInputs.get_verbose() is False