Update interface-unit-tests.yml handling of pytest warnings

.github/workflows/interface-unit-tests.yml

@@ -89,7 +89,7 @@ jobs:
       default_runner: ubuntu-latest
       force_large_runner: ${{ inputs.use_large_runner }}
 
-  disable-fail-fast-wae:
+  warnings-as-errors-setup:
     needs:
       - determine_runner
 
@@ -100,8 +100,22 @@ jobs:
         run: |
           echo "fail_fast=${{ contains(inputs.python_warning_level, 'default') && 'default' || 'error' }}" >> $GITHUB_OUTPUT
 
+      - name: Set pytest arguments for setting warnings level
+        id: pytest_warning_flags
+        env:
+          PYTEST_WARNING_ARGS: -W "${{ inputs.python_warning_level }}" --continue-on-collection-errors
+        run: |
+          if [ "${{ inputs.python_warning_level }}" != "default" ]; then
+            echo "Setting pytest warning flags"
+            echo "pytest_warning_args=$PYTEST_WARNING_ARGS" >> $GITHUB_OUTPUT
+          else
+            echo "No pytest warning flags needed"
+            echo "pytest_warning_args=" >> $GITHUB_OUTPUT
+          fi
+
     outputs:
       fail_fast: ${{ steps.mat_fail_fast.outputs.fail_fast }}
+      pytest_warning_args: ${{ steps.pytest_warning_flags.outputs.pytest_warning_args }}
 
   setup-ci-load:
     needs:
@@ -127,7 +141,7 @@ jobs:
           then
             cat >python_versions.json <<-EOF
           {
-            "default": ["3.10"]       
+            "default": ["3.10"]
           }
           EOF
           else
@@ -215,9 +229,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).torch-tests
@@ -241,7 +255,7 @@ jobs:
         ${{ needs.default-dependency-versions.outputs.pytorch-version }}
         ${{ inputs.additional_python_packages }}
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: torch and not qcut and not finite-diff and not param-shift
       requirements_file: ${{ github.event_name == 'schedule' && strategy.job-index == 0 && 'torch.txt' || '' }}
@@ -252,9 +266,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).autograd-tests
@@ -277,7 +291,7 @@ jobs:
       additional_pip_packages: ${{ inputs.additional_python_packages }}
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       pytest_markers: autograd and not qcut and not finite-diff and not param-shift
 
 
@@ -286,9 +300,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).tf-tests
@@ -315,7 +329,7 @@ jobs:
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: tf and not qcut and not finite-diff and not param-shift
-      pytest_additional_args: --splits 3 --group ${{ matrix.group }} -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: --splits 3 --group ${{ matrix.group }} ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       pytest_durations_file_path: '.github/durations/tf_tests_durations.json'
       requirements_file: ${{ github.event_name == 'schedule' && strategy.job-index == 0 && 'tf.txt' || '' }}
 
@@ -324,9 +338,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).jax-tests
@@ -353,7 +367,7 @@ jobs:
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: jax and not qcut and not finite-diff and not param-shift
-      pytest_additional_args: --dist=loadscope --splits 4 --group ${{ matrix.group }} -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: --dist=loadscope --splits 4 --group ${{ matrix.group }} ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       pytest_durations_file_path: '.github/durations/jax_tests_durations.json'
       requirements_file: ${{ github.event_name == 'schedule' && strategy.job-index == 0 && 'jax.txt' || '' }}
 
@@ -363,9 +377,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).core-tests
@@ -390,7 +404,7 @@ jobs:
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: core and not qcut and not finite-diff and not param-shift
-      pytest_additional_args: --splits 6 --group ${{ matrix.group }} -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: --splits 6 --group ${{ matrix.group }} ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       pytest_durations_file_path: '.github/durations/core_tests_durations.json'
       requirements_file: ${{ github.event_name == 'schedule' && strategy.job-index == 0 && 'core.txt' || '' }}
 
@@ -400,9 +414,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).all-interfaces-tests
@@ -430,7 +444,7 @@ jobs:
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: all_interfaces
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       requirements_file: ${{ github.event_name == 'schedule' && strategy.job-index == 0 && 'all_interfaces.txt' || '' }}
 
 
@@ -439,9 +453,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).external-libraries-tests
@@ -463,7 +477,7 @@ jobs:
       python_version: ${{ matrix.python-version }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: external
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       additional_pip_packages: |
         pyzx==0.8.0 matplotlib stim quimb mitiq ply optax scipy-openblas32>=0.3.26
         git+https://github.com/PennyLaneAI/pennylane-qiskit.git@master
@@ -480,9 +494,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).qcut-tests
@@ -504,7 +518,7 @@ jobs:
       python_version: ${{ matrix.python-version }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: qcut
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       additional_pip_packages: |
         kahypar==1.1.7 
         opt_einsum
@@ -520,9 +534,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).qchem-tests
@@ -545,7 +559,7 @@ jobs:
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
       pytest_markers: qchem
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       additional_pip_packages: |
         openfermionpyscf basis-set-exchange geometric scikit-learn
         ${{ inputs.additional_python_packages }}
@@ -555,9 +569,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).gradients-tests
@@ -580,7 +594,7 @@ jobs:
       branch: ${{ inputs.branch }}
       coverage_artifact_name: gradients-${{ matrix.config.suite }}-coverage
       python_version: ${{ matrix.python-version }}
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       additional_pip_packages: |
         ${{ needs.default-dependency-versions.outputs.jax-version }}
         ${{ needs.default-dependency-versions.outputs.tensorflow-version }}
@@ -596,9 +610,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).data-tests
@@ -620,7 +634,7 @@ jobs:
       python_version: ${{ matrix.python-version }}
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
-      pytest_additional_args: -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
       pytest_markers: data
       additional_pip_packages: |
         h5py
@@ -632,9 +646,9 @@ jobs:
       - setup-ci-load
       - determine_runner
       - default-dependency-versions
-      - disable-fail-fast-wae
+      - warnings-as-errors-setup
     strategy:
-      fail-fast: ${{ needs.disable-fail-fast-wae.outputs.fail_fast == 'default' }}
+      fail-fast: ${{ needs.warnings-as-errors-setup.outputs.fail_fast == 'default' }}
       max-parallel: >-
         ${{
            fromJSON(needs.setup-ci-load.outputs.matrix-max-parallel).device-tests
@@ -669,7 +683,7 @@ jobs:
       additional_pip_packages_post: ${{ needs.default-dependency-versions.outputs.pennylane-lightning-latest }}
       pytest_test_directory: pennylane/devices/tests
       pytest_coverage_flags: ${{ inputs.pytest_coverage_flags }}
-      pytest_additional_args: --device=${{ matrix.config.device }} --shots=${{ matrix.config.shots }} -W ${{ inputs.python_warning_level }} ${{ inputs.python_warning_level != 'default' && '--continue-on-collection-errors' || '' }}
+      pytest_additional_args: --device=${{ matrix.config.device }} --shots=${{ matrix.config.shots }} ${{ needs.warnings-as-errors-setup.outputs.pytest_warning_args }}
 
 
   upload-to-codecov: