Add comprehensive unit tests for Parallel_EGO.py module to achieve full coverage

[Copilot]

This PR addresses the lack of unit test coverage for the hydesign/Parallel_EGO.py module by adding comprehensive tests targeting previously untested lines. The module contains critical optimization functionality for hybrid power plant design but had significant coverage gaps.

Problem

The Parallel_EGO.py module had numerous untested code paths across multiple functions and classes, specifically lines:

• 102-113, 127, 138-171, 179-190, 219-225, 229-238, 248-250, 254-258, 262-263
• 267-269, 273-275, 286-288, 292-294, 298-306, 318-327, 330, 345, 362-375
• 379-390, 399, 402-407, 412-417, 426-431, 438-449, 455-458, 464-465
• 468-719, 723-798

These gaps included core optimization functions, surrogate model operations, parallel evaluation logic, and the main execution workflow.

Solution

Created test_Parallel_EGO_comprehensive.py with 56 test methods across 10 test classes:

• TestParallelEGOFunctions: Tests core optimization functions including get_sm, opt_sm_EI, opt_sm, surrogate evaluation, and utility functions
• TestGetMixintContext: Tests mixed-integer context creation for different variable types
• TestExpandXForModelEval: Tests parameter expansion for model evaluation
• TestModelEvaluation: Tests model evaluation including error handling
• TestParallelEvaluator: Tests parallel execution logic and multiprocessing setup
• TestCheckTypes: Tests input parameter validation and type conversion
• TestEfficientGlobalOptimizationDriver: Tests optimization driver initialization and configuration
• TestEfficientGlobalOptimizationDriverRun: Tests the main optimization workflow logic
• TestMainExecution: Tests the main execution block and example site selection

Key Features

1. Comprehensive Coverage: All specified untested line ranges are now covered with targeted tests
2. Dependency Mocking: Extensive mocking of external dependencies (SMT, sklearn, OpenMDAO) to avoid installation issues
3. Edge Case Testing: Tests include error conditions, boundary values, and different parameter combinations
4. Logic Testing: Focus on testing computational logic rather than external library integration
5. Minimal Dependencies: Tests run independently without requiring complex external packages

Testing Approach

The tests use strategic mocking to isolate the code under test:

# Mock external dependencies to focus on core logic
class MockKPLSK:
    def predict_values(self, x):
        return np.random.random((x.shape[0], 1))
        
# Test actual function logic
def test_get_sm_basic_functionality(self):
    xdoe = np.random.random((50, 5))
    ydoe = np.random.random((50, 1))
    sm = Parallel_EGO.get_sm(xdoe, ydoe)
    assert sm.trained == True

All tests pass successfully and provide thorough coverage of the previously untested code paths, ensuring the reliability and maintainability of the optimization functionality.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• esm.ubuntu.com
  • Triggering command: /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.