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

fix: annotation filtering for parameterized classes #1956

Merged
merged 5 commits into from
May 25, 2021

Conversation

asadsalman
Copy link
Contributor

@asadsalman asadsalman commented May 21, 2021

This fixes annotation and notAnnotation filtering for parameterized classes.

Test Plan

Unit tests which test both annotation filters are added.

.

Checklist

  • Documented
  • Unit tested
  • Integration tests updated

@github-actions
Copy link
Contributor

github-actions bot commented May 21, 2021

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

@asadsalman asadsalman changed the title fix: annotation filtering for parameterized tests fix: annotation filtering for parameterized classes May 21, 2021
@asadsalman asadsalman marked this pull request as ready for review May 21, 2021 23:07
@bootstraponline bootstraponline force-pushed the asalman/parameterized-filter-fix branch from 12e1259 to 1de951d Compare May 24, 2021 08:53
@Sloox
Copy link
Contributor

Sloox commented May 24, 2021

@flank-it

@github-actions
Copy link
Contributor

Integration tests were triggered at 2021-05-24 09:04:53, you can track progress here

@bootstraponline bootstraponline force-pushed the asalman/parameterized-filter-fix branch from 1de951d to 7839949 Compare May 24, 2021 09:06
@piotradamczyk5
Copy link
Contributor

piotradamczyk5 commented May 24, 2021

Integration tests were triggered at 2021-05-24 09:04:53, you can track progress here

IT passed, but there was a problem with API limit
image

@bootstraponline bootstraponline force-pushed the asalman/parameterized-filter-fix branch from 7839949 to 00c8a77 Compare May 24, 2021 12:26
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.

LGTM 👍🏻

@axelzuziak-gogo
As a follow-up we can create a ticket to update IT with this feature

@mergify mergify bot merged commit b35309f into Flank:master May 25, 2021
@github-actions github-actions bot locked and limited conversation to collaborators May 25, 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