This message was deleted WireMock #general

Join Slack

This message was deleted.

# general

Slackbot

03/13/2023, 1:39 PM

This message was deleted.

Paul Petershagen

03/14/2023, 3:35 AM

Hi, I obviously can only look at your test set up, I don't know if your WireMock assertions make sense, but: You should definitely change the await() statement to something like:

Copy code

Awaitility.await().pollDelay(Durations.TWO_SECONDS).untilAsserted(() -> {
       
        wireMockServer.verify(1, getRequestedFor(urlMatching(
                String.format("{url}", userId))));

        wireMockServer.verify(0, postRequestedFor(urlMatching(
                "{url}"))); 
});

It will wait until the assertions don't throw an exception. You might also want to add an

.atMost(...)

, to change the max runtime for the

await()

. But if in that time frame the unit under test sends two requests, then that is its behavior, then double check the prod code is doing the right thing.

Paul Petershagen

03/14/2023, 3:39 AM

BTW can you check the

urlMatching

statement if that is right, not sure if you have modified it for this post here.

Sheetal Jain

03/14/2023, 6:24 AM

Thanks, @Paul Petershagen, I modified the

urlMatching

. Let me make the changes that you have suggested then I'll let you know.

Sheetal Jain

03/14/2023, 6:31 AM

Can you please explain to me what is the reason for changing

await() statement

to what you have suggested? Like where is the problem.

Sheetal Jain

03/14/2023, 6:37 AM

I tried what you have suggested, it is still giving me error

Copy code

org.awaitility.core.ConditionTimeoutException: Assertion condition defined as a lambda expression in co.tala.atlas.promotions.integration.kafka.consumer.SignUpEventConsumerTest Expected exactly 1 requests matching the following pattern but received 2:

Paul Petershagen

03/14/2023, 6:41 AM

The idea beind await().until(..) statement is to wait until a test or scenario condition is fulfilled. But in your case you always return true, means the await() waits only once. If your service finishes up after 21 seconds, it won't honor / notice it.

Paul Petershagen

03/14/2023, 6:43 AM

The error says it retrieved 2 GET requests, but you wanted to verify there is only one. See logs for the request / response dumps. Also would also start debugging your service in regards to the 2 reqeusts, it that is not expected.

Sheetal Jain

03/14/2023, 6:45 AM

okay, does it requires any change in this test case or I need to check other files or classes?

Paul Petershagen

03/14/2023, 6:48 AM

Look into the production process flow, which is supposed to be tested by this test.

Sheetal Jain

03/14/2023, 6:48 AM

okay, Thanks for the help @Paul Petershagen.

🙏 1

Sheetal Jain

03/22/2023, 8:57 AM

This issue is not solved yet. Can anyone please help me with this?

Sheetal Jain

03/22/2023, 10:06 AM

Copy code

com.github.tomakehurst.wiremock.client.VerificationException: Expected exactly 1 requests matching the following pattern but received 2:
    {
      "urlPattern" : "/users/customer/profiles/844424930132106",
      "method" : "GET"
    }
        at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:538)
        at com.github.tomakehurst.wiremock.client.WireMock.verifyThat(WireMock.java:521)
        at com.github.tomakehurst.wiremock.WireMockServer.verify(WireMockServer.java:271)
        at

Sheetal Jain

03/22/2023, 12:52 PM

@Tom can you please check?

462 Views

Open in Slack

Previous Next