Skip to content

Commit

Permalink
NPI-3430 added possibly redundant check in SP3 unit test, for empty S…
Browse files Browse the repository at this point in the history
…V names, in case there is a way to mess up the parsing and get an empty one.
  • Loading branch information
treefern committed Aug 2, 2024
1 parent 8af0a69 commit 97ab47f
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions tests/test_sp3.py
Original file line number Diff line number Diff line change
Expand Up @@ -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")
Expand Down

0 comments on commit 97ab47f

Please sign in to comment.