62
62
("kwargs" , "expected_artifact_bucket" , "expected_artifact_prefix" ),
63
63
[
64
64
({}, None , _DEFAULT_ARTIFACT_PREFIX ),
65
- ({"artifact_bucket" : TEST_ARTIFACT_BUCKET , "artifact_prefix" : TEST_ARTIFACT_PREFIX }, TEST_ARTIFACT_BUCKET , TEST_ARTIFACT_PREFIX ),
65
+ (
66
+ {
67
+ "artifact_bucket" : TEST_ARTIFACT_BUCKET ,
68
+ "artifact_prefix" : TEST_ARTIFACT_PREFIX ,
69
+ },
70
+ TEST_ARTIFACT_BUCKET ,
71
+ TEST_ARTIFACT_PREFIX ,
72
+ ),
66
73
],
67
74
)
68
75
@patch (
79
86
MagicMock (side_effect = mock_tc_load_or_create_func ),
80
87
)
81
88
@patch .object (_TrialComponent , "save" )
82
- def test_run_init (mock_tc_save , sagemaker_session , kwargs , expected_artifact_bucket , expected_artifact_prefix ):
89
+ def test_run_init (
90
+ mock_tc_save ,
91
+ sagemaker_session ,
92
+ kwargs ,
93
+ expected_artifact_bucket ,
94
+ expected_artifact_prefix ,
95
+ ):
83
96
with Run (
84
- experiment_name = TEST_EXP_NAME , run_name = TEST_RUN_NAME , sagemaker_session = sagemaker_session , ** kwargs
97
+ experiment_name = TEST_EXP_NAME ,
98
+ run_name = TEST_RUN_NAME ,
99
+ sagemaker_session = sagemaker_session ,
100
+ ** kwargs ,
85
101
) as run_obj :
86
102
assert not run_obj ._in_load
87
103
assert not run_obj ._inside_load_context
@@ -135,11 +151,19 @@ def test_run_init_name_length_exceed_limit(sagemaker_session):
135
151
err
136
152
)
137
153
154
+
138
155
@pytest .mark .parametrize (
139
156
("kwargs" , "expected_artifact_bucket" , "expected_artifact_prefix" ),
140
157
[
141
158
({}, None , _DEFAULT_ARTIFACT_PREFIX ),
142
- ({"artifact_bucket" : TEST_ARTIFACT_BUCKET , "artifact_prefix" : TEST_ARTIFACT_PREFIX }, TEST_ARTIFACT_BUCKET , TEST_ARTIFACT_PREFIX ),
159
+ (
160
+ {
161
+ "artifact_bucket" : TEST_ARTIFACT_BUCKET ,
162
+ "artifact_prefix" : TEST_ARTIFACT_PREFIX ,
163
+ },
164
+ TEST_ARTIFACT_BUCKET ,
165
+ TEST_ARTIFACT_PREFIX ,
166
+ ),
143
167
],
144
168
)
145
169
@patch .object (_TrialComponent , "save" , MagicMock (return_value = None ))
@@ -157,7 +181,13 @@ def test_run_init_name_length_exceed_limit(sagemaker_session):
157
181
MagicMock (side_effect = mock_tc_load_or_create_func ),
158
182
)
159
183
@patch ("sagemaker.experiments.run._RunEnvironment" )
160
- def test_run_load_no_run_name_and_in_train_job (mock_run_env , sagemaker_session , kwargs , expected_artifact_bucket , expected_artifact_prefix ):
184
+ def test_run_load_no_run_name_and_in_train_job (
185
+ mock_run_env ,
186
+ sagemaker_session ,
187
+ kwargs ,
188
+ expected_artifact_bucket ,
189
+ expected_artifact_prefix ,
190
+ ):
161
191
client = sagemaker_session .sagemaker_client
162
192
job_name = "my-train-job"
163
193
rv = Mock ()
@@ -234,11 +264,19 @@ def test_run_load_no_run_name_and_not_in_train_job_but_no_obj_in_context(sagemak
234
264
235
265
assert "Failed to load a Run object" in str (err )
236
266
267
+
237
268
@pytest .mark .parametrize (
238
269
("kwargs" , "expected_artifact_bucket" , "expected_artifact_prefix" ),
239
270
[
240
271
({}, None , _DEFAULT_ARTIFACT_PREFIX ),
241
- ({"artifact_bucket" : TEST_ARTIFACT_BUCKET , "artifact_prefix" : TEST_ARTIFACT_PREFIX }, TEST_ARTIFACT_BUCKET , TEST_ARTIFACT_PREFIX ),
272
+ (
273
+ {
274
+ "artifact_bucket" : TEST_ARTIFACT_BUCKET ,
275
+ "artifact_prefix" : TEST_ARTIFACT_PREFIX ,
276
+ },
277
+ TEST_ARTIFACT_BUCKET ,
278
+ TEST_ARTIFACT_PREFIX ,
279
+ ),
242
280
],
243
281
)
244
282
@patch .object (_TrialComponent , "save" , MagicMock (return_value = None ))
@@ -255,7 +293,9 @@ def test_run_load_no_run_name_and_not_in_train_job_but_no_obj_in_context(sagemak
255
293
"sagemaker.experiments.run._TrialComponent._load_or_create" ,
256
294
MagicMock (side_effect = mock_tc_load_or_create_func ),
257
295
)
258
- def test_run_load_with_run_name_and_exp_name (sagemaker_session , kwargs , expected_artifact_bucket , expected_artifact_prefix ):
296
+ def test_run_load_with_run_name_and_exp_name (
297
+ sagemaker_session , kwargs , expected_artifact_bucket , expected_artifact_prefix
298
+ ):
259
299
with load_run (
260
300
run_name = TEST_RUN_NAME ,
261
301
experiment_name = TEST_EXP_NAME ,
0 commit comments