aiidateam / aiida-core

The official repository for the AiiDA code
https://aiida-core.readthedocs.io
Other
437 stars 192 forks source link

CLI: Fix `verdi storage migrate` for profile without broker #6550

Closed sphuber closed 4 months ago

sphuber commented 4 months ago

Fixes #6546

The command needs to make sure the daemon of the profile is not running so it instantiates the DaemonClient but this raises for profiles that do not define a broker. Since the daemon cannot be started for brokerless profiles anyway the command does not have to check in this case.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.83%. Comparing base (ef60b66) to head (6019e8b). Report is 119 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #6550 +/- ## ========================================== + Coverage 77.51% 77.83% +0.32% ========================================== Files 560 566 +6 Lines 41444 41967 +523 ========================================== + Hits 32120 32659 +539 + Misses 9324 9308 -16 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.