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

feat: structured parsers and diagnostic messages #75

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

rcarriga
Copy link
Owner

Implements the requirements for #73 to show diagnsotic messages beside
the lines that cause an error. A parser will need to be implemented for
each runner.

Structured parsing is also required for issue #70 so ultest can parse
results of multiple files.

@rcarriga rcarriga force-pushed the feat/output-parser/structured-parsers branch 3 times, most recently from 1b7cbee to 3eada6b Compare October 24, 2021 09:51
@rcarriga rcarriga force-pushed the feat/output-parser/structured-parsers branch 2 times, most recently from 6cbed56 to 24706c7 Compare October 25, 2021 16:26
@rcarriga rcarriga force-pushed the feat/output-parser/structured-parsers branch 2 times, most recently from aaa337a to efd96a0 Compare November 28, 2021 17:13
Implements the requirements for #73 to show diagnsotic messages beside
the lines that cause an error. A parser will need to be implemented for
each runner.

Structured parsing is also required for issue #70 so ultest can parse
results of multiple files.
BREAKING CHANGE: Process functions accept lists

In preparation for suite testing, the process functions have changed to
accept lists instead of single tests/results. This greatly improves the
performance of the summary and diagnostics.
Allows ultest to determine the suitable stack trace to use such as
for hypothesis, the second trace should be used.
@rcarriga rcarriga force-pushed the feat/output-parser/structured-parsers branch from efd96a0 to f96880b Compare December 4, 2021 16:06
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.

1 participant