Skip to content

Commit 5de1e55

Browse files
committed
fix: unit tests
1 parent e9f2f65 commit 5de1e55

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

tests/unit/test_estimator.py

+8-8
Original file line numberDiff line numberDiff line change
@@ -341,7 +341,7 @@ def test_set_accept_eula_for_input_data_config_no_input_data_config():
341341
train_args = {}
342342
accept_eula = True
343343

344-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
344+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
345345

346346
# Verify train_args remains unchanged
347347
assert train_args == {}
@@ -352,7 +352,7 @@ def test_set_accept_eula_for_input_data_config_none_accept_eula():
352352
train_args = {"InputDataConfig": [{"DataSource": {"S3DataSource": {}}}]}
353353
accept_eula = None
354354

355-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
355+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
356356

357357
# Verify train_args remains unchanged
358358
assert train_args == {"InputDataConfig": [{"DataSource": {"S3DataSource": {}}}]}
@@ -366,7 +366,7 @@ def test_set_accept_eula_for_input_data_config_single_data_source():
366366
}
367367
accept_eula = True
368368

369-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
369+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
370370

371371
# Verify ModelAccessConfig and AcceptEula are set correctly
372372
assert train_args["InputDataConfig"][0]["DataSource"]["S3DataSource"][
@@ -388,7 +388,7 @@ def test_set_accept_eula_for_input_data_config_multiple_data_sources():
388388
}
389389
accept_eula = True
390390

391-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
391+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
392392

393393
# Verify ModelAccessConfig and AcceptEula are set correctly for both data sources
394394
assert train_args["InputDataConfig"][0]["DataSource"]["S3DataSource"][
@@ -422,7 +422,7 @@ def test_set_accept_eula_for_input_data_config_existing_model_access_config():
422422
}
423423
accept_eula = True
424424

425-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
425+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
426426

427427
# Verify AcceptEula is added to existing ModelAccessConfig
428428
assert train_args["InputDataConfig"][0]["DataSource"]["S3DataSource"]["ModelAccessConfig"] == {
@@ -436,7 +436,7 @@ def test_set_accept_eula_for_input_data_config_missing_s3_data_source():
436436
train_args = {"InputDataConfig": [{"DataSource": {"OtherDataSource": {}}}]}
437437
accept_eula = True
438438

439-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
439+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
440440

441441
# Verify train_args remains unchanged
442442
assert train_args == {"InputDataConfig": [{"DataSource": {"OtherDataSource": {}}}]}
@@ -447,7 +447,7 @@ def test_set_accept_eula_for_input_data_config_missing_data_source():
447447
train_args = {"InputDataConfig": [{"OtherKey": {}}]}
448448
accept_eula = True
449449

450-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
450+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
451451

452452
# Verify train_args remains unchanged
453453
assert train_args == {"InputDataConfig": [{"OtherKey": {}}]}
@@ -464,7 +464,7 @@ def test_set_accept_eula_for_input_data_config_mixed_data_sources():
464464
}
465465
accept_eula = True
466466

467-
EstimatorBase._set_accept_eula_for_input_data_config(train_args, accept_eula)
467+
_TrainingJob._set_accept_eula_for_input_data_config(train_args, accept_eula)
468468

469469
# Verify ModelAccessConfig and AcceptEula are set correctly for S3DataSource only
470470
assert train_args["InputDataConfig"][0]["DataSource"]["S3DataSource"][

0 commit comments

Comments
 (0)