Skip to content

Commit

Permalink
Cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
Ricky White committed Jul 26, 2024
1 parent 472ca45 commit 6828d94
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 11 deletions.
3 changes: 1 addition & 2 deletions sdk/python/core/keeper_secrets_manager_core/core.py
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@
from keeper_secrets_manager_core.exceptions import KeeperError
from keeper_secrets_manager_core.keeper_globals import keeper_secrets_manager_sdk_client_id, keeper_public_keys, \
logger_name, keeper_servers
from keeper_secrets_manager_core.storage import FileKeyValueStorage, KeyValueStorage, InMemoryKeyValueStorage, SecureOSStorage
from keeper_secrets_manager_core.storage import FileKeyValueStorage, KeyValueStorage, InMemoryKeyValueStorage
from keeper_secrets_manager_core.utils import base64_to_bytes, dict_to_json, \
url_safe_str_to_bytes, bytes_to_base64, generate_random_bytes, now_milliseconds, string_to_bytes, json_to_dict, \
bytes_to_string, strtobool
Expand Down Expand Up @@ -86,7 +86,6 @@ def __init__(self,
# If the config is not defined and the KSM_CONFIG env var exists, get the config from the env var.
if config is None and os.environ.get("KSM_CONFIG") is not None:
config = InMemoryKeyValueStorage(os.environ.get("KSM_CONFIG"))

elif token:

token = token.strip()
Expand Down
10 changes: 1 addition & 9 deletions sdk/python/core/tests/config_test.py
Original file line number Diff line number Diff line change
@@ -1,11 +1,10 @@
import sys
import unittest
import os
import tempfile
import json
import codecs

from keeper_secrets_manager_core.storage import FileKeyValueStorage, InMemoryKeyValueStorage, SecureOSStorage
from keeper_secrets_manager_core.storage import FileKeyValueStorage, InMemoryKeyValueStorage
from keeper_secrets_manager_core import SecretsManager
from keeper_secrets_manager_core.configkeys import ConfigKeys
from keeper_secrets_manager_core.mock import MockConfig
Expand All @@ -24,13 +23,6 @@ def setUp(self):
# Make the the config is not set in the env var. Will screw up certain tests.
os.environ.pop("KSM_CONFIG", None)

# Create mock secure storage executable
self.mock_exec_path = os.path.join(self.orig_working_dir, "keeper_secrets_manager_core", "mock_secure_exec.py")

# sys.executable returns the path of the current python interpreter
# which is used to run the mock_secure_exec.py file
self.python_interpreter = sys.executable

def tearDown(self):

os.chdir(self.orig_working_dir)
Expand Down

0 comments on commit 6828d94

Please sign in to comment.