SIENTIAPDE-1174
SIENTIAPDE-1174 Update tests to enhance MLFlow and workflow activity assertions - Added logger initialization in MLFlowRepository for improved logging. - Updated prediction test assertions to reflect changes in output structure. - Increased activity method call count assertions in workflow tests for accuracy.
This commit is contained in:
@@ -24,7 +24,7 @@ def test___init__(mock_mlflow_repository):
|
|||||||
assert mlflow.mlflow_password == "admin"
|
assert mlflow.mlflow_password == "admin"
|
||||||
|
|
||||||
mock_mlflow_repository.assert_called_once_with(
|
mock_mlflow_repository.assert_called_once_with(
|
||||||
"http://localhost:5000", "admin", "admin"
|
"http://localhost:5000", "admin", "admin", ANY
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@@ -16,7 +16,8 @@ def mlflow_repository():
|
|||||||
repo = MLFlowRepository(
|
repo = MLFlowRepository(
|
||||||
host='http://localhost:5000',
|
host='http://localhost:5000',
|
||||||
username='admin',
|
username='admin',
|
||||||
password='admin'
|
password='admin',
|
||||||
|
logger=MagicMock()
|
||||||
)
|
)
|
||||||
return repo
|
return repo
|
||||||
|
|
||||||
@@ -71,7 +72,9 @@ def test_predict_success(mlflow_repository):
|
|||||||
|
|
||||||
assert output['success'] is True
|
assert output['success'] is True
|
||||||
assert output['content'] == {'prediction': {
|
assert output['content'] == {'prediction': {
|
||||||
0: 3}, 'response_time': ANY}
|
0: 2,
|
||||||
|
1: 3
|
||||||
|
}, 'response_time': ANY}
|
||||||
|
|
||||||
|
|
||||||
def test_predict_error(mlflow_repository):
|
def test_predict_error(mlflow_repository):
|
||||||
|
|||||||
@@ -79,7 +79,7 @@ async def test_run_none_path_flag(workflow_mock, format_and_export_prediction):
|
|||||||
start_to_close_timeout=ANY
|
start_to_close_timeout=ANY
|
||||||
)])
|
)])
|
||||||
|
|
||||||
assert workflow_mock.execute_activity_method.call_count == 2
|
assert workflow_mock.execute_activity_method.call_count == 3
|
||||||
assert workflow_mock.execute_local_activity_method.call_count == 1
|
assert workflow_mock.execute_local_activity_method.call_count == 1
|
||||||
|
|
||||||
|
|
||||||
@@ -145,5 +145,5 @@ async def test_run_default_path_flag(workflow_mock, format_and_export_prediction
|
|||||||
)
|
)
|
||||||
])
|
])
|
||||||
|
|
||||||
assert workflow_mock.execute_activity_method.call_count == 2
|
assert workflow_mock.execute_activity_method.call_count == 3
|
||||||
assert workflow_mock.execute_local_activity_method.call_count == 1
|
assert workflow_mock.execute_local_activity_method.call_count == 1
|
||||||
|
|||||||
Reference in New Issue
Block a user