Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Patch all task unit tests #19

Merged
merged 1 commit into from
Sep 20, 2022
Merged

Patch all task unit tests #19

merged 1 commit into from
Sep 20, 2022

Conversation

atreyasha
Copy link
Member

Migrate away from using set to sorted in order to compare actual and expected data. This has an advantage of not potentially removing duplicates while still being able to compare data.

Migrate away from using `set` to `sorted` in order to compare actual and
expected data. This has an advantage of not potentially removing
duplicates while still being able to compare data.
@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
parser.py00100% 
privacy_glue.py0220%4–6, 9, 13, 15–16, 19–20, 26–27, 29, 32–36, 43, 50, 52, 56–57
tasks
   opp_115.py00100% 
   piextract.py00100% 
   policy_detection.py00100% 
   policy_ie_a.py00100% 
   policy_ie_b.py00100% 
   policy_qa.py00100% 
   privacy_qa.py00100% 
utils
   logging_utils.py00100% 
   parser_utils.py00100% 
   pipeline_utils.py01260%4–21, 24–25, 28–29, 36–39, 41, 43, 46–59, 61, 63, 65, 67, 69–71, 77, 81–83, 92, 95–98, 104, 106, 111, 114–115, 117, 119, 124, 126, 140, 142, 144, 146, 148, 151–152, 158, 160–161, 164, 166–167, 169, 172–173, 175–177, 179–181, 183–185, 187–189, 191–193, 195–203, 205–210, 212–213, 215–217, 219–225, 227
TOTAL35414858% 

Tests Skipped Failures Errors Time
37 0 💤 0 ❌ 0 🔥 2.305s ⏱️

@atreyasha atreyasha merged commit ebb30d7 into main Sep 20, 2022
@atreyasha atreyasha deleted the as/patch-tests branch September 20, 2022 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant