#10944 fix CI: remove misplaced Python test, drop redundant stream clears

Python-API test in EMSManager.unit.cc violated testing pattern and failed
Linux CI (missing include + state.get() on raw pointer). Keep single
Python test in datatransfer.unit.cc where fixture belongs. Also drop
compare_err_stream("") pre-clears per review - substring search is
targeted, collision with setup chatter impossible.

Author: brianlball

tst/EnergyPlus/unit/EMSManager.unit.cc

@@ -3059,8 +3059,6 @@ TEST_F(EnergyPlusFixture, UnusedActuatorWarning)
     EMSManager::ManageEMS(*state, EMSManager::EMSCallFrom::SetupSimulation, anyRan, ObjexxFCL::Optional_int_const());
     EMSManager::ManageEMS(*state, EMSManager::EMSCallFrom::BeginTimestepBeforePredictor, anyRan, ObjexxFCL::Optional_int_const());
 
-    compare_err_stream(""); // drop any setup chatter
-
     EMSManager::checkForUnusedActuatorsAtEnd(*state);
 
     EXPECT_TRUE(compare_err_stream_substring("Unused EMS Actuator detected", false));
@@ -3112,49 +3110,7 @@ TEST_F(EnergyPlusFixture, UnusedActuatorWarning_ConditionalNullBranchNotFalsePos
     EMSManager::ManageEMS(*state, EMSManager::EMSCallFrom::SetupSimulation, anyRan, ObjexxFCL::Optional_int_const());
     EMSManager::ManageEMS(*state, EMSManager::EMSCallFrom::BeginTimestepBeforePredictor, anyRan, ObjexxFCL::Optional_int_const());
 
-    compare_err_stream(""); // drop any setup chatter
-
     EMSManager::checkForUnusedActuatorsAtEnd(*state);
 
     EXPECT_FALSE(compare_err_stream_substring("Unused EMS Actuator detected", false, false));
 }
-
-TEST_F(EnergyPlusFixture, UnusedActuatorWarning_PythonHandleMarksActuatorAsUsed)
-{
-    // Issue #10944: IDF-declared actuator + Python grabs handle but no Erl SET anywhere.
-    // Python API path must flip wasActuated on handle retrieval so the end-of-sim check
-    // doesn't false-positive on legitimate Python-driven actuators.
-    std::string const idf_objects = delimited_string({
-
-        "OutdoorAir:Node, Test node;",
-
-        "EnergyManagementSystem:Actuator,",
-        "TempSetpointLo,                  !- Name",
-        "Test node,                       !- Actuated Component Unique Name",
-        "System Node Setpoint,            !- Actuated Component Type",
-        "Temperature Minimum Setpoint;    !- Actuated Component Control Type",
-
-    });
-
-    ASSERT_TRUE(process_idf(idf_objects));
-    state->init_state(*state);
-
-    OutAirNodeManager::SetOutAirNodes(*state);
-    EMSManager::CheckIfAnyEMS(*state);
-    state->dataEMSMgr->FinishProcessingUserInput = true;
-
-    bool anyRan;
-    EMSManager::ManageEMS(*state, EMSManager::EMSCallFrom::SetupSimulation, anyRan, ObjexxFCL::Optional_int_const());
-    ASSERT_EQ(1, state->dataRuntimeLang->numActuatorsUsed);
-
-    EXPECT_FALSE(state->dataRuntimeLang->EMSActuatorUsed(1).wasActuated);
-
-    int hActuator = getActuatorHandle(state.get(), "System Node Setpoint", "Temperature Minimum Setpoint", "Test node");
-    EXPECT_GT(hActuator, -1);
-
-    EXPECT_TRUE(state->dataRuntimeLang->EMSActuatorUsed(1).wasActuated);
-
-    compare_err_stream("", true); // drop the expected duplicate-definition chatter
-    EMSManager::checkForUnusedActuatorsAtEnd(*state);
-    EXPECT_FALSE(compare_err_stream_substring("Unused EMS Actuator detected", false, false));
-}

tst/EnergyPlus/unit/api/datatransfer.unit.cc

@@ -739,7 +739,6 @@ TEST_F(DataExchangeAPIUnitTestFixture, DataTransfer_PythonHandle_MarksActuatorAs
     // getActuatorHandle should have flipped the flag
     EXPECT_TRUE(state->dataRuntimeLang->EMSActuatorUsed(1).wasActuated);
 
-    compare_err_stream("", true); // drop expected duplicate-definition warning
     EMSManager::checkForUnusedActuatorsAtEnd(*state);
     EXPECT_FALSE(compare_err_stream_substring("Unused EMS Actuator detected", false, false));
 }