Skip to content

Commit

Permalink
Merge pull request #80 from dr-rodriguez/main
Browse files Browse the repository at this point in the history
Fixing integrity test
  • Loading branch information
Will-Cooper authored Oct 27, 2020
2 parents 63bb2d8 + e3bcdc8 commit c31da1e
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
2 changes: 0 additions & 2 deletions .github/workflows/gen-db.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@ name: Generate Database

on:
workflow_dispatch: # manual execution
schedule:
- cron: '0 12 15 * *' # 12:00 UTC on the 15th of each month

jobs:
build:
Expand Down
7 changes: 6 additions & 1 deletion tests/test_integrity.py
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,12 @@ def test_source_simbad(db):
# Get a nicely formatted list of Simbad names for each input row
duplicate_count = 0
for row in simbad_results[['TYPED_ID', 'IDS']].iterrows():
name, ids = row[0].decode("utf-8"), row[1].decode("utf-8")
try:
name, ids = row[0].decode("utf-8"), row[1].decode("utf-8")
except AttributeError:
# Catch decoding error
name, ids = row[0], row[1]

simbad_names = [_name_formatter(s) for s in ids.split('|')
if _name_formatter(s) != '' and _name_formatter(s) is not None]

Expand Down

0 comments on commit c31da1e

Please sign in to comment.