Skip to content

Commit

Permalink
NPI-3688 todo for better tests for comment handling functions. Lintin…
Browse files Browse the repository at this point in the history
…g fix
  • Loading branch information
treefern committed Jan 30, 2025
1 parent 4417c4f commit b0e10db
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion tests/test_sp3.py
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ def test_read_sp3_validation_sv_count_mismatch_header_vs_content(self, mock_file
"Header says there should be 1 epochs, however there are 2 (unique) epochs in the content (duplicate epoch check comes later).",
"Loading SP3 with mismatch between SV count in header and in content, should raise exception",
)

@patch("builtins.open", new_callable=mock_open, read_data=sp3c_example2_data)
def test_read_sp3_correct_svs_read_when_ev_ep_present(self, mock_file):
# This should not raise an exception; SV count should match header if parsed correctly.
Expand Down Expand Up @@ -218,6 +218,9 @@ def test_gen_sp3_fundamentals(self):
# - Not including column names (can just test that output matches expected format)
# - Not including any NaN value *anywhere*

# TODO add tests for SP3 comment handling, such as comment line reflow, append vs overwrite comments, comment
# format exception handling.

def test_gen_sp3_content_velocity_exception_handling(self):
"""
gen_sp3_content() velocity output should raise exception (currently unsupported).\
Expand Down

0 comments on commit b0e10db

Please sign in to comment.