Skip to content

Commit

Permalink
refactor.
Browse files Browse the repository at this point in the history
  • Loading branch information
rshewitt committed May 12, 2023
1 parent f26d89c commit 4f59571
Showing 1 changed file with 4 additions and 5 deletions.
9 changes: 4 additions & 5 deletions tests/extract/test_dcatus.py
Original file line number Diff line number Diff line change
@@ -1,16 +1,15 @@
from datagovharvester.extract.utils import extract_catalog
from datagovharvester.extract.dcatus import extract_dcatus_catalog


def test_extract_dcatus(get_dcatus_job, create_client, create_bucket):
success = False

url = get_dcatus_job["url"]
job_id = get_dcatus_job["job_id"]

S3_client, S3_msg = create_client
bucket_name, bucket_msg = create_bucket

final_res, final_error_msg = extract_catalog(url, job_id, S3_client, bucket_name)
final_res, final_error_msg = extract_dcatus_catalog(
get_dcatus_job["url"], S3_client, get_dcatus_job["job_id"], bucket_name
)

if final_error_msg is None:
success = True
Expand Down

1 comment on commit 4f59571

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report
FileStmtsMissCoverMissing
datagovharvester/extract
   __init__.py00100% 
   dcatus.py1722 88%
   utils.py1622 88%
datagovharvester/utils
   __init__.py00100% 
   json_utilities.py1022 80%
   s3_utilities.py2744 85%
datagovharvester/validate
   __init__.py00100% 
   dcat_us.py200100% 
TOTAL901089% 

Tests Skipped Failures Errors Time
13 0 💤 0 ❌ 0 🔥 24.447s ⏱️

Please sign in to comment.