Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: added support for JUnit Theory tests #1968

Merged
merged 4 commits into from
May 26, 2021

Conversation

asadsalman
Copy link
Contributor

@asadsalman asadsalman commented May 25, 2021

JUnit Theories were not parsed by Flank perviously. This PR adds support for them.

Test Plan

How do we know the code works?
Added test.

Checklist

  • Documented
  • Unit tested
  • Integration tests updated

@github-actions
Copy link
Contributor

github-actions bot commented May 25, 2021

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

Copy link
Contributor

@pawelpasterz pawelpasterz left a comment

Choose a reason for hiding this comment

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

Looking good, just small change request 👍🏻

@pawelpasterz
Copy link
Contributor

Follow-up: create IT for this case
cc: @axelzuziak-gogo

@Sloox
Copy link
Contributor

Sloox commented May 26, 2021

@flank-it

@github-actions
Copy link
Contributor

github-actions bot commented May 26, 2021

Integration tests succeed for all OSes ✅
Windows Build scan:
MacOS Build scan: https://gradle.com/s/6z2jwlakbozqw
Linux Build scan: https://gradle.com/s/uvoga42r32kie
Workflow run https://github.com/Flank/flank/actions/runs/878248758

@bootstraponline bootstraponline force-pushed the asalman/theory-support branch from b25a402 to 74dd6a1 Compare May 26, 2021 11:37
@bootstraponline bootstraponline force-pushed the asalman/theory-support branch from 74dd6a1 to 41163ca Compare May 26, 2021 14:21
@mergify mergify bot merged commit 4b5b92e into Flank:master May 26, 2021
@github-actions github-actions bot locked and limited conversation to collaborators May 26, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants