Skip to content

Commit 772cba5

Browse files
committed
fix lint
1 parent f4c735d commit 772cba5

File tree

2 files changed

+11
-9
lines changed

2 files changed

+11
-9
lines changed

secrets_env/providers/vault/config.py

+7-4
Original file line numberDiff line numberDiff line change
@@ -134,16 +134,19 @@ def get_connection_info(data: dict) -> VaultConnectionInfo | None:
134134
logger.error("Unknown auth method: <data>%s</data>", parsed.auth.get("method"))
135135
return
136136

137-
conn_info = VaultConnectionInfo(
138-
url=str(parsed.url),
139-
auth=auth,
137+
conn_info = typing.cast(
138+
VaultConnectionInfo,
139+
{
140+
"url": str(parsed.url),
141+
"auth": auth,
142+
},
140143
)
141144

142145
if parsed.proxy:
143146
conn_info["proxy"] = str(parsed.proxy)
144147
if parsed.tls.ca_cert:
145148
conn_info["ca_cert"] = parsed.tls.ca_cert
146-
if parsed.tls.client_key:
149+
if parsed.tls.client_cert and parsed.tls.client_key:
147150
conn_info["client_cert"] = (parsed.tls.client_cert, parsed.tls.client_key)
148151
elif parsed.tls.client_cert:
149152
conn_info["client_cert"] = parsed.tls.client_cert

tests/providers/vault/test_vault_config.py

+4-5
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,8 @@
1-
import os
21
from pathlib import Path
3-
from unittest.mock import patch
42

53
import pytest
6-
from pydantic_core import Url, ValidationError
4+
from pydantic_core import Url
75

8-
import secrets_env.providers.vault.config as t
96
from secrets_env.providers.vault.auth.base import NullAuth
107
from secrets_env.providers.vault.config import (
118
RawVaultUserConfig,
@@ -63,7 +60,9 @@ def test_auth(self):
6360
assert config.auth == {"method": "token"}
6461

6562
# missing method
66-
with pytest.raises(ValueError):
63+
with pytest.raises(
64+
ValueError, match="Missing required config <mark>auth method</mark>"
65+
):
6766
RawVaultUserConfig.model_validate(
6867
{
6968
"url": "https://example.com",

0 commit comments

Comments
 (0)