From 97ab47fb86b1cd76b97e455eca48f9845a2d65af Mon Sep 17 00:00:00 2001
From: Nathan <95725385+treefern@users.noreply.github.com>
Date: Fri, 2 Aug 2024 08:21:38 +0000
Subject: [PATCH] NPI-3430 added possibly redundant check in SP3 unit test, for
 empty SV names, in case there is a way to mess up the parsing and get an
 empty one.

---
 tests/test_sp3.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/test_sp3.py b/tests/test_sp3.py
index 17f770d..9f97785 100644
--- a/tests/test_sp3.py
+++ b/tests/test_sp3.py
@@ -109,6 +109,9 @@ def test_read_sp3_header_svs_detailed(self):
         sv_count = sv_info.shape[0]  # Effectively len()
         self.assertEqual(sv_count, 30, msg="There should be 30 SVs parsed from the test data")
 
+        # Ensure no SVs are read as empty
+        self.assertFalse(any(len(sv.strip())==0 for sv in sv_info.index), msg="No SV name should be empty")
+
         # Focus on potential line wraparound issues
         first_sv = sv_info.index[0]
         self.assertEqual(first_sv, 'G02', msg="First SV in test data should be G02")