Add support for multiple KeyVaults

[mkah91]

@fbufler it's still very much work in progress, but I wanted to see what you think from a users point of view before continuing.

Fixes: #8

[fbufler]

Looks pretty cool to me.
Some minor things I've found that disturbed my experience a bit.

azkv vaults list let's you select vaults (not sure if thats intended by list, maybe we shoud use a set commands idk.

Scrolling through azkv show or azkv edit is a bit unstructured (search obviously helps a lot)

• Maybe we could only show the vault name if there are duplicate secret names?
• Or we put the vault name in front with vault/secret so it's a bit more structured.

[mkah91]

Sounds good. If I find some time this week I will come up with an adapted Version (probably still wip, though)

[mkah91]

@fbufler I added your suggestions and finished the refactoring (tests are missing), If you find some time, please have a look :)

[fbufler]

@fbufler I added your suggestions and finished the refactoring (tests are missing), If you find some time, please have a look :)

I'm loving it, awesome work :)

[github-actions]

Coverage Report

File	Stmts	Miss	Cover	Missing
__init__.py	0	0	100%
azkv.py	10	10	0%	1, 3, 6–11, 14–15
decorators.py	17	17	0%	1–2, 4, 7–10, 12, 15–22, 24
main.py	62	62	0%	1–2, 4–5, 7–15, 17–18, 21–25, 27–35, 38–39, 43–46, 49–53, 55, 58–62, 64, 67–70, 72, 75–84
client
__init__.py	0	0	100%
keyvault_client.py	81	51	37%	38–39, 42–44, 48, 51–53, 56–57, 60, 63–67, 70–73, 76–77, 79–80, 83–91, 94–99, 101, 104–113
keyvault_clients.py	75	53	29%	12–16, 21–25, 29, 32–35, 38–42, 45–47, 49–50, 53–54, 57–68, 71–72, 75, 80, 83, 86–94
keyvault_secret.py	17	6	64%	14–16, 19–21
commands
__init__.py	0	0	100%
check.py	25	0	100%
common.py	42	1	97%	89
edit.py	37	7	81%	20–26
show.py	45	7	84%	20–26
commands/vaults
__init__.py	0	0	100%
add.py	16	16	0%	1, 3, 5, 8–10, 16–18, 21–27
login.py	10	10	0%	1–2, 5–10, 16–17
main.py	33	33	0%	1, 3–7, 10–13, 15, 18–21, 23–30, 33–36, 38, 41–44, 46
remove.py	11	11	0%	1–2, 5–10, 16, 19–20
select.py	11	11	0%	1, 4–5, 8–15
TOTAL	492	295	40%

[mkah91]

@fbufler I finally found some time to at least implement some unit tests. To ship the functionality I'd say it's okay to merge now and increase unit test coverage later. If you find some time please have a look and try it out :)

[fbufler]

LGMT :) Awesome work btw