github-actions: added saving debug logs by default for all functional…

… tests (#7931)

<!--
Thank you for your contribution.

Before making a PR, please read our contributing guidelines at

https://github.com/DevExpress/testcafe/blob/master/CONTRIBUTING.md#code-contribution

We recommend creating a *draft* PR, so that you can mark it as 'ready
for review' when you are done.
-->
[closes DevExpress/testcafe-private#232]

## Purpose
_Describe the problem you want to address or the feature you want to
implement._

## Approach
_Describe how your changes address the issue or implement the desired
functionality in as much detail as possible._

## References
_Provide a link to the existing issue(s), if any._

## Pre-Merge TODO
- [ ] Write tests for your proposed changes
- [ ] Make sure that existing tests do not fail

.github/workflows/deploy-to-artifacts.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       base_sha:
         description: 'The base commit SHA'
-      debug:
-        required: false
-        type: string
-        default: ''
 
 jobs:
   build:
@@ -109,12 +105,11 @@ jobs:
           github-token: ${{secrets.ACTIVE_TOKEN}}
           script: |
             function getInputs () {
-              const { sha, merged_sha, debug } = context.payload.inputs;
+              const { sha, merged_sha } = context.payload.inputs;
 
               return {
                 ...merged_sha ? { merged_sha } : {},
                 sha,
-                debug,
                 deploy_run_id: '${{github.run_id}}'
               }
             }

.github/workflows/test-functional-docker.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-chrome.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-edge.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-esm.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-firefox.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-headed-browsers.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-legacy.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-multiple-windows.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-native-automation.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional-local-safari.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'	
       deploy_run_id:	
         description: 'The ID of a deployment workspace run with artifacts'	
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:	
   test:	
     uses: ./.github/workflows/test-functional.yml	

.github/workflows/test-functional-remote-mobile.yml

@@ -11,10 +11,6 @@ on:
         description: 'The merge commit SHA'
       deploy_run_id:
         description: 'The ID of a deployment workspace run with artifacts'
-      debug:
-        required: false
-        type: string
-        default: ''
 jobs:
   test:
     uses: ./.github/workflows/test-functional.yml

.github/workflows/test-functional.yml

@@ -73,7 +73,7 @@ jobs:
       RETRY_FAILED_TESTS: ${{ inputs.retry_failed_tests }}
       TEST_GROUPS_COUNT: ${{ inputs.matrix-jobs-count }}
       TEST_GROUP_NUMBER: ${{ matrix.test-group }}
-      DEBUG: ${{ github.event.inputs.debug }}
+      DEBUG: 'testcafe*'
       DISPLAY: ${{ inputs.display }}
     outputs:
       # Matrix jobs cannot collect outputs about each of them automatically. It rewrites the same outputs. Improve it once it will be possible.
@@ -142,7 +142,7 @@ jobs:
           local-logging-level: all-logs
           local-identifier: random
 
-      - run: ${{ inputs.test-script }} 2> testcafe-debug-log.log
+      - run: ${{ inputs.test-script }} 2> testcafe-debug-log-${{ matrix.test-group }}.log
         timeout-minutes: ${{ inputs.timeout }}
 
       - name: 'Stop BrowserStackLocal'
@@ -152,11 +152,11 @@ jobs:
           local-testing: stop
 
       - name: Save debug log
+        if: always()
         uses: actions/upload-artifact@v3
         with:
             name: testcafe-debug-log
             path: testcafe-debug-log-${{ matrix.test-group }}.log
-        if: ${{ always() && github.event.inputs.debug != '' }}
 
       - name: Save result
         id: save-result