fix(test): correct implicit string concatenation in test_ship.py message arrays

[pankaj-bind]

Related Issue / Discussion:

Fixes #162

Changes:

• Modified tests/test_ship.py - added missing comma in message array literal (2 occurrences)

Further Details:

Summary:

In both test_shipparams_get_element and test_shipparams_get_single, the message array was defined as:

message = np.array(['OK', 'OK', 'Error' 'OK'])

Python's implicit string concatenation silently merged 'Error' and 'OK' into 'ErrorOK', producing a 3-element array instead of the intended 4-element array. The tests still passed because the corrupted value round-tripped unchanged through ShipParams, masking the defect.

This fix adds the missing comma to restore ['OK', 'OK', 'Error', 'OK'] in both locations.

Dependencies:

None.

PR Checklist:

• have (already previously) filled the 52North Contributor License Agreement and received positive feedback on this matter
• have filled the 52North Contributor License Agreement and am waiting for feedback
• provide unit tests embedded in the WRT test framework (WeatherRoutingTool/tests) that allow the simple testing of the new/modified functionalities. All (previous and new) unit tests execute without new error messages.
• ensure that the code formatter runs without errors/warnings
• ensure that my changes follow the WRT's guidelines for contributing at the time of the contribution

[Copilot]

Pull request overview

Fixes a subtle Python test-data bug in tests/test_ship.py where implicit string concatenation ('Error' 'OK' -> 'ErrorOK') caused the message NumPy array to have the wrong number of elements, potentially masking defects in ShipParams-related tests.

Changes:

• Add missing comma in message = np.array([...]) within test_shipparams_get_element.
• Add missing comma in message = np.array([...]) within test_shipparams_get_single.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.