Skip to content

Commit 315e63a

Browse files
jasonyatesjyates
andauthored
Fix Verizon parser (#198)
* Fix Verizon parser Parser was looking for <tbody> in circuits table. Not all VZ notifications include the <tbody> element. Confirmed all existing tests pass with this change. * Adding additional Verizon test Adding test for a notification without <tbody> * Adding test_e2e * Fixing Verizon5 test results Co-authored-by: jyates <[email protected]>
1 parent e6bcab8 commit 315e63a

File tree

5 files changed

+399
-1
lines changed

5 files changed

+399
-1
lines changed

circuit_maintenance_parser/parsers/verizon.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ def parse_tables(self, tables: ResultSet, data: Dict): # pylint: disable=too-ma
5757
data["start"] = self.dt2ts(start)
5858
data["end"] = self.dt2ts(end)
5959

60-
for row in circuit_table.find("tbody").find_all("tr"):
60+
for row in circuit_table.find_all("tr"):
6161
cells = row.find_all("td")
6262
cells_text = [cell.string.strip() for cell in cells if cell.string]
6363
if not cells_text or cells_text[0].startswith("Company Name"):

0 commit comments

Comments
 (0)