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

Adds hasAnObservation assertion that allows doing arbitrary assertions + clear method #3679

Merged
merged 4 commits into from
Mar 9, 2023

Conversation

marcingrzejszczak
Copy link
Contributor

fixes gh-3674

@marcingrzejszczak marcingrzejszczak changed the title Adds hasAnObservation assertion that allows doing arbitrary assertions Adds hasAnObservation assertion that allows doing arbitrary assertions + clear method Mar 9, 2023
.atMost(Duration.ofMillis(50))
.untilAsserted(() -> BDDAssertions.then(registry.getContexts()).hasSize(3));
Awaitility.await().pollDelay(Duration.ofMillis(10)).atMost(Duration.ofMillis(50)).untilAsserted(() -> {
System.out.println("Registry size [" + registry.getContexts().size() + "]");
Copy link
Contributor

Choose a reason for hiding this comment

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

System.out :)

Copy link
Member

Choose a reason for hiding this comment

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

👍🏼 🚓

@jonatan-ivanov jonatan-ivanov merged commit 6ef2ec4 into main Mar 9, 2023
@jonatan-ivanov jonatan-ivanov deleted the issues_#3674 branch March 9, 2023 19:02
izeye added a commit to izeye/micrometer that referenced this pull request Mar 11, 2023
@izeye izeye mentioned this pull request Mar 11, 2023
shakuzen pushed a commit that referenced this pull request Mar 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't re-initialise TestObservationRegistry between tests
3 participants