Test aiida 2.7

[danielhollas]

Both notebook tests and unit tests are updated to use aiida-core 2.7.1.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.29%. Comparing base (3f9e956) to head (2c9d850).
⚠️ Report is 7 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #698   +/-   ##
=======================================
  Coverage   84.29%   84.29%           
=======================================
  Files          18       18           
  Lines        3584     3584           
=======================================
  Hits         3021     3021           
  Misses        563      563           

Flag	Coverage Δ
python-3.11	84.29% <ø> (ø)
python-3.9	84.30% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[agoscinski]

Some of the errors I think are related to the usage tempfile.mkdtemp (only checked the export_aiida_subgraph so far), I at least remember that I also had issues with this. That it was already removed when I created it, so I switched to the pytest fixture or tempfile.TemporaryDirectory. It might be related due to an update in the dependencies of aiida-core.

[danielhollas]

I think it has something to do with the profile management. The actual error from the export_aiida_subgraph is comes from a failing verdi archive create command, which complains that a profile is not set.

Command '['verdi', 'archive', 'create', '/tmp/pytest-of-runner/pytest-0/test_export_button_widget0/export.aiida', '-N', '5']' returned non-zero exit status 1.
b"Critical: no default profile defined: None\n{'CONFIG_VERSION': {'CURRENT': 9, 'OLDEST_COMPATIBLE': 9}, 'profiles': {'909900f38064e37d3f242f6336098653': {'storage': {'backend': 'core.sqlite_dos', 'config': {'filepath': '/tmp/pytest-of-runner/pytest-0/test_sqlite_dos_storage0'}}, 'process_control': {'backend': 'core.rabbitmq', 'config': {'broker_protocol': 'amqp', 'broker_username': 'guest', 'broker_password': 'guest', 'broker_host': '127.0.0.1', 'broker_port': 5672, 'broker_virtual_host': ''}}, 'test_profile': True, 'PROFILE_UUID': '53e42cace0c14a31816a52dbd0a11632', 'options': {}, 'default_user_email': 'test@localhost'}}}\n"

[danielhollas]

Just a small simplification

[danielhollas]

This was super-painful to fix (these tests now fail on master as well). Really these tests should be rewritten to use mocked-out data instead of real data that can change at any moment.

[danielhollas]

See also this note.

https://github.com/aiidalab/aiidalab-widgets-base/pull/698/files#diff-71fe007c958acd11b69e1011aaaa2116a3a7f62a2e5ef8ec3743a8d191dcec88R49

[superstar54]

LGTM!