Skip to content

Commit b04601d

Browse files
author
Saumitra Vikram
committed
Updating unit test
1 parent 7a4e4ce commit b04601d

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

tests/unit/sagemaker/model/test_model_package.py

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,6 @@
5252
"CertifyForMarketplace": False,
5353
}
5454

55-
ENV_KEY_1 = "env_key_1"
56-
ENV_VALUE_1 = "env_key_1"
57-
ENVIRONMENT = {ENV_KEY_1: ENV_VALUE_1}
58-
5955

6056
@pytest.fixture
6157
def sagemaker_session():
@@ -122,12 +118,15 @@ def test_create_sagemaker_model_uses_model_name(name_from_base, sagemaker_sessio
122118
def test_create_sagemaker_model_include_environment_variable(sagemaker_session):
123119
model_name = "my-model"
124120
model_package_name = "my-model-package"
121+
env_key = "env_key"
122+
env_value = "env_value"
123+
environment = {env_key: env_value}
125124

126125
model_package = ModelPackage(
127126
role="role",
128127
name=model_name,
129128
model_package_arn=model_package_name,
130-
env=ENVIRONMENT,
129+
env=environment,
131130
sagemaker_session=sagemaker_session,
132131
)
133132

@@ -136,7 +135,7 @@ def test_create_sagemaker_model_include_environment_variable(sagemaker_session):
136135
sagemaker_session.create_model.assert_called_with(
137136
model_name,
138137
"role",
139-
{"ModelPackageName": model_package_name, "Environment": ENVIRONMENT},
138+
{"ModelPackageName": model_package_name, "Environment": environment},
140139
vpc_config=None,
141140
enable_network_isolation=False,
142141
)

0 commit comments

Comments
 (0)