Skip to content

Commit

Permalink
Fix the input argument in secrets.kv.v1
Browse files Browse the repository at this point in the history
  • Loading branch information
tungbq committed Feb 23, 2024
1 parent 22b3405 commit b3d4d8d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -468,7 +468,7 @@ def create_or_update_secret(
mount_point, secret_path = self._parse_secret_path(secret_path)
if self.kv_engine_version == 1:
response = self.client.secrets.kv.v1.create_or_update_secret(
secret_path=secret_path, secret=secret, mount_point=mount_point, method=method
path=secret_path, secret=secret, mount_point=mount_point, method=method
)
else:
response = self.client.secrets.kv.v2.create_or_update_secret(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1111,7 +1111,7 @@ def test_create_or_update_secret_v1(self, mock_hvac):
)
vault_client.create_or_update_secret(secret_path="path", secret={"key": "value"})
mock_client.secrets.kv.v1.create_or_update_secret.assert_called_once_with(
mount_point="secret", secret_path="path", secret={"key": "value"}, method=None
mount_point="secret", path="path", secret={"key": "value"}, method=None
)

@mock.patch("airflow.providers.hashicorp._internal_client.vault_client.hvac")
Expand Down Expand Up @@ -1145,7 +1145,7 @@ def test_create_or_update_secret_v1_post(self, mock_hvac):
)
vault_client.create_or_update_secret(secret_path="path", secret={"key": "value"}, method="post")
mock_client.secrets.kv.v1.create_or_update_secret.assert_called_once_with(
mount_point="secret", secret_path="path", secret={"key": "value"}, method="post"
mount_point="secret", path="path", secret={"key": "value"}, method="post"
)

@mock.patch("airflow.providers.hashicorp._internal_client.vault_client.hvac")
Expand Down
2 changes: 1 addition & 1 deletion tests/providers/hashicorp/hooks/test_vault.py
Original file line number Diff line number Diff line change
Expand Up @@ -1253,7 +1253,7 @@ def test_create_or_update_secret_v1(self, mock_hvac, mock_get_connection, method
test_hook = VaultHook(**kwargs)
test_hook.create_or_update_secret(secret_path="path", secret={"key": "value"}, method=method)
mock_client.secrets.kv.v1.create_or_update_secret.assert_called_once_with(
mount_point="secret", secret_path="path", secret={"key": "value"}, method=expected_method
mount_point="secret", path="path", secret={"key": "value"}, method=expected_method
)


Expand Down

0 comments on commit b3d4d8d

Please sign in to comment.