Formatting, small unit test fixes

Author: kaleko

lib/idp_common_pkg/idp_common/agents/external_mcp/agent.py

@@ -60,7 +60,9 @@ def create_external_mcp_agent(
         "cognito_username",
         "cognito_password",
     ]
-    missing_fields = [field for field in required_fields if field not in mcp_server_config]
+    missing_fields = [
+        field for field in required_fields if field not in mcp_server_config
+    ]
     if missing_fields:
         error_msg = f"MCP server config missing required fields: {missing_fields}"
         logger.error(error_msg)

lib/idp_common_pkg/tests/unit/agents/common/test_monitoring.py

@@ -126,7 +126,7 @@ def test_tracker_initialization_disabled(self):
         assert not tracker.enabled
         assert tracker.db_logger is None
 
-    @patch.dict("os.environ", {"ANALYTICS_TABLE": "test-table"})
+    @patch.dict("os.environ", {"AGENT_TABLE": "test-table"})
     @patch("idp_common.agents.common.dynamodb_logger.DynamoDBMessageLogger")
     def test_tracker_initialization_enabled(self, mock_logger_class):
         """Test tracker initialization when enabled."""