Skip to content

Commit

Permalink
took a guess at what was causing the pre-commit.ci falure 'add traili…
Browse files Browse the repository at this point in the history
…ng commas'
  • Loading branch information
upsonp committed Dec 4, 2023
1 parent ac213dc commit 4f415bb
Showing 1 changed file with 3 additions and 4 deletions.
7 changes: 3 additions & 4 deletions tests/test_read.py
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,9 @@ def test_header_parse():
def test_header_parse_blank_line():
# check that a BTL file can still be loaded if the header section contains blank lines
try:
btl = ctd.from_btl(data_path.joinpath("btl", "blank_line_header.btl"))
btl = ctd.from_btl(data_path.joinpath("btl", "blank_line_header.btl",))
# if a value error wasn't thrown, ensure the names array for the _metadata was set
assert btl._metadata["names"].index("Date")
except ValueError as ex:
# if the blank line in the header causes the reader to exit before reading the file
# the line looking for the Date in the ctd.from_btl() will throw a ValueError.
Expand All @@ -126,9 +128,6 @@ def test_header_parse_blank_line():
raise AssertionError(
"Blank line in BTL file causes _parse_seabird function to exit before setting names") from ex

# if a value error wasn't thrown, ensure the names array for the _metadata was set
assert btl._metadata["names"].index("Date")


def test_pressure_field_labels():
"""
Expand Down

0 comments on commit 4f415bb

Please sign in to comment.