SIENTIAPDE-1314

Refactor OPC Class and Enhance Testing for Output Management

- Removed the unused 'success' parameter from the manage_output_tags method in the OPC class to streamline its signature.
- Commented out the error handling logic in the OpcRepository for clarity and future reference.
- Added new tests for manage_output_tags to validate successful and failed write operations, ensuring accurate assertions for output data and metrics.

tests/laborious/activities/test_opc.py

@@ -257,6 +257,85 @@ async def test_write_data_exception(opc):
         raise AssertionError('Expected an exception to be raised')
 
 
+@mark.asyncio
+async def test_manage_output_tags_success(opc):
+    opc.write_data = AsyncMock(return_value=0.1)
+
+    data = DataFrame({'prediction': [0.75], 'prediction_confidence': [0.95]})
+    config = {
+        'prediction_tags': {'tag1': {'data_type': 'float'}},
+        'confidence_tags': {'tag2': {'data_type': 'float'}},
+    }
+
+    output_data, opc_metrics = await opc.manage_output_tags(
+        server_id='server1',
+        config=config,
+        data=data,
+        metadata=metadata['metadata'],
+    )
+
+    assert output_data is True
+    assert opc_metrics == {'tag1': 0.1, 'tag2': 0.1}
+    opc.write_data.assert_has_calls(
+        [
+            call(
+                server_id='server1',
+                tag='tag1',
+                data=0.75,
+                data_type='float',
+                tag_type='prediction',
+                metadata=metadata['metadata'],
+            ),
+            call(
+                server_id='server1',
+                tag='tag2',
+                data=0.95,
+                data_type='float',
+                tag_type='confidence',
+                metadata=metadata['metadata'],
+            ),
+        ]
+    )
+
+
+@mark.asyncio
+async def test_manage_output_tags_failed(opc):
+    opc.write_data = AsyncMock(side_effect=[0.1, None])
+    data = DataFrame({'prediction': [0.75], 'prediction_confidence': [0.95]})
+    config = {
+        'prediction_tags': {'tag1': {'data_type': 'float'}},
+        'confidence_tags': {'tag2': {'data_type': 'float'}},
+    }
+    output_data, opc_metrics = await opc.manage_output_tags(
+        server_id='server1',
+        config=config,
+        data=data,
+        metadata=metadata['metadata'],
+    )
+    assert output_data is False
+    assert opc_metrics == {'tag1': 0.1, 'tag2': None}
+    opc.write_data.assert_has_calls(
+        [
+            call(
+                server_id='server1',
+                tag='tag1',
+                data=0.75,
+                data_type='float',
+                tag_type='prediction',
+                metadata=metadata['metadata'],
+            ),
+            call(
+                server_id='server1',
+                tag='tag2',
+                data=0.95,
+                data_type='float',
+                tag_type='confidence',
+                metadata=metadata['metadata'],
+            ),
+        ]
+    )
+
+
 @mark.asyncio
 @patch('laborious.activities.opc.DataFrame')
 async def test_write_opc_data_success(mock_dataframe, opc):
@@ -286,7 +365,6 @@ async def test_write_opc_data_success(mock_dataframe, opc):
         input_data['opc_output_config']['server1'],
         mock_dataframe.return_value,
         metadata['metadata'],
-        True,
     )
     opc.process_confidence.assert_called_once_with(
         mock_dataframe.return_value,