Skip to content

[pre-commit.ci] pre-commit autoupdate #155

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
pre-commit-ci wants to merge 2 commits into
base: main
Choose a base branch
from
+92 −174

[pre-commit.ci] auto fixes from pre-commit.com hooks

b395272
Select commit
Loading
Failed to load commit list.
Sign in for the full log view
Open

[pre-commit.ci] pre-commit autoupdate #155

[pre-commit.ci] auto fixes from pre-commit.com hooks
b395272
Select commit
Loading
Failed to load commit list.
GitHub Actions / Test Results failed Mar 9, 2026 in 0s

1 fail, 24 pass in 8s

 3 files  ±0   3 suites  ±0   8s ⏱️ -1s
25 tests ±0  24 ✅ ±0  0 💤 ±0  1 ❌ ±0 
75 runs  ±0  74 ✅ ±0  0 💤 ±0  1 ❌ ±0 

Results for commit b395272. ± Comparison against earlier commit ab9e5c1.

Annotations

Check warning on line 0 in src.tests.mapping_test

See this annotation in the file changed.

@github-actions github-actions / Test Results

1 out of 3 runs failed: test_mapping_gui_map_automatically (src.tests.mapping_test) 

artifacts/pytest-results-3.11/test-results.xml [took 0s]
Raw output 
AssertionError: assert {'T1': {'test...62086033360>}} == {'T1': {'test...62086033360>}}
  Omitting 2 identical items, use -vv to show
  Differing items:
  {'T1': {'test_parameter': None}} != {'T1': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562085538832>}}
  {'T2': {'test_parameter': None}} != {'T2': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562084521424>}}
  Full diff:
    {
  -  'T1': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562085538832>},
  -  'T2': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562084521424>},
  +  'T1': {'test_parameter': None},
  +  'T2': {'test_parameter': None},
     'dac': {'voltage': <qcodes.parameters.parameter.Parameter: voltage at 140562113904528>},
     'dmm': {'current': <qumada.instrument.custom_drivers.Dummies.dummy_dmm.dmm_results_random: current at 140562084564624>,
             'voltage': <qumada.instrument.custom_drivers.Dummies.dummy_dmm.dmm_results_sinus: voltage at 140562086033360>},
    }
mocker = <pytest_mock.plugin.MockerFixture object at 0x7fd72921f250>
qtbot = <pytestqt.qtbot.QtBot object at 0x7fd72921ed90>
station_with_instruments = <qcodes.station.Station object at 0x7fd72928fe10>
script = <qumada.measurement.scripts.generic_measurement.Generic_1D_Sweep object at 0x7fd72921f650>

    def test_mapping_gui_map_automatically(mocker, qtbot, station_with_instruments, script):
        # mock dialogs (specify behaviour in return_value and skip)
        mocker.patch("qumada.instrument.mapping.mapping_gui.MessageBox_notallmapped.exec", return_value=QMessageBox.No)
        mocker.patch("qumada.instrument.mapping.mapping_gui.MessageBox_duplicates.exec", return_value=QMessageBox.No)
        mocker.patch("qumada.instrument.mapping.mapping_gui.MessageBox_overwrite.exec", return_value=QMessageBox.No)
    
        w = MainWindow(
            station_with_instruments.components,
            script.gate_parameters,
        )
        w.show()
        qtbot.addWidget(w)
    
        # wanted mapping
        terminal_params = {
            "dmm": {"voltage": station_with_instruments.dmm.voltage, "current": station_with_instruments.dmm.current},
            "dac": {"voltage": station_with_instruments.dac.ch01.voltage},
            "T1": {"test_parameter": station_with_instruments.dci.A.temperature},
            "T2": {"test_parameter": station_with_instruments.dci.B.temperature},
        }
    
        # check if auto mapping yields wanted result
        # qtbot.mouseClick(w.button_map_auto, Qt.MouseButton.LeftButton)
        qtbot.keyPress(w, Qt.Key_A)
>       assert w.terminal_parameters == terminal_params
E       AssertionError: assert {'T1': {'test...62086033360>}} == {'T1': {'test...62086033360>}}
E         Omitting 2 identical items, use -vv to show
E         Differing items:
E         {'T1': {'test_parameter': None}} != {'T1': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562085538832>}}
E         {'T2': {'test_parameter': None}} != {'T2': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562084521424>}}
E         Full diff:
E           {
E         -  'T1': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562085538832>},
E         -  'T2': {'test_parameter': <qcodes.parameters.parameter.Parameter: temperature at 140562084521424>},
E         +  'T1': {'test_parameter': None},
E         +  'T2': {'test_parameter': None},
E            'dac': {'voltage': <qcodes.parameters.parameter.Parameter: voltage at 140562113904528>},
E            'dmm': {'current': <qumada.instrument.custom_drivers.Dummies.dummy_dmm.dmm_results_random: current at 140562084564624>,
E                    'voltage': <qumada.instrument.custom_drivers.Dummies.dummy_dmm.dmm_results_sinus: voltage at 140562086033360>},
E           }

src/tests/mapping_test.py:394: AssertionError
View more details on GitHub Actions