Fix context commands - Githubissues

alan-turing-institute / data-safe-haven

https://data-safe-haven.readthedocs.io

BSD 3-Clause "New" or "Revised" License

61 stars 15 forks source link

Fix context commands #2065

Closed jemrobinson closed 3 months ago

jemrobinson commented 3 months ago

:white_check_mark: Checklist

[x] You have given your pull request a meaningful title (e.g. Enable foobar integration rather than 515 foobar).
[x] You are targeting the appropriate branch. If you're not certain which one this is, it should be develop.
[x] Your branch is up-to-date with the target branch (it probably was when you started, but it may have changed since then).

:vertical_traffic_light: Depends on

n/a

:arrow_heading_up: Summary

Switch context on add
Change the context key whenever the name changes to avoid mismatches
Ensure the the SHM storage account only contains alphanumeric characters

:closed_umbrella: Related issues

Closes #2055 Closes #2061

:microscope: Tests

Tested locally

github-actions[bot] commented 3 months ago

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
data_safe_haven/commands
context.py
data_safe_haven/config
context.py
context_manager.py
Project Total

This report was generated by python-coverage-comment-action