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

Log the actual exception that caused a ParseError #322

Merged
merged 6 commits into from Jul 6, 2022

Conversation

EnricoMi
Copy link
Owner

@EnricoMi EnricoMi commented Jul 5, 2022

Also reduce code duplication parsing XMl files

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (setup-python)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (macOS 12 python installed)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Test File)

97 tests  ±0   80 ✔️ ±0   3m 25s ⏱️ ±0s
  1 suites ±0   17 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Dockerfile)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Docker Image)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Linux python 3.6)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (macOS python installed)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (macOS python 3.6)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Linux python installed)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi force-pushed the branch-improve-parse-error-logging branch from 3e98e73 to 76b5007 Compare July 5, 2022 16:32
@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Linux 22.04 python installed)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (Windows python installed)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Jul 5, 2022

Test Results (reference)

       69 files  ±    0         69 suites  ±0   23m 52s ⏱️ - 1m 0s
     302 tests +    3       302 ✔️ +    3      0 💤 ±0  0 ±0 
20 838 runs  +207  20 298 ✔️ +207  540 💤 ±0  0 ±0 

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi force-pushed the branch-improve-parse-error-logging branch from 76b5007 to 4c5ab88 Compare July 5, 2022 20:57
@EnricoMi EnricoMi closed this Jul 5, 2022
@EnricoMi EnricoMi reopened this Jul 5, 2022
@EnricoMi EnricoMi force-pushed the branch-improve-parse-error-logging branch from 4c5ab88 to 6c322af Compare July 5, 2022 21:00
@github-actions
Copy link

github-actions bot commented Jul 6, 2022

Test Results (Test Files)

     65 files  ±0         6 errors  323 suites  ±0   1h 6m 4s ⏱️ ±0s
1 626 tests ±0  1 351 ✔️ ±0    71 💤 ±0  197 ±0  7 🔥 ±0 
1 874 runs  ±0  1 543 ✔️ ±0  106 💤 ±0  217 ±0  8 🔥 ±0 

For more details on these parsing errors, failures and errors, see this check.

Results for commit 1fa65ba. ± Comparison against base commit 9dc979d.

♻️ This comment has been updated with latest results.

@EnricoMi EnricoMi merged commit dd6c641 into master-2.x Jul 6, 2022
@EnricoMi EnricoMi deleted the branch-improve-parse-error-logging branch July 6, 2022 10:33
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