Remove unused code

[jemrobinson]

✅ Checklist

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

🚦 Depends on

⤴️ Summary

• Remove unused ConfigSection::update() functions
• Remove unused ConfigName validation and tests
• Add tests for ConfigSection and SafeString
• Simplify dsh sre deploy/teardown logic
• BUGFIX: create DSHPulumiConfig in sre deploy or this will not be done

🌂 Related issues

n/a

🔬 Tests

Tested locally

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
data_safe_haven/administration/users
guacamole_users.py					45
data_safe_haven/commands
sre.py					53-56, 96-100, 137-147
users.py					160
data_safe_haven/config
config_sections.py
sre_config.py
data_safe_haven/external/api
azure_api.py
data_safe_haven/infrastructure/programs
declarative_sre.py
data_safe_haven/types
annotated_types.py
data_safe_haven/validators
validators.py
Project Total

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

[JimMadge]

LGTM

Clears out a lot of code we don't need 🎉.