Update Salesforce testCDCAndStreamingEvents test to wait for consumin…

[zbendhiba]

…g the right Document

Fixes #3103

[zbendhiba]

I don't know why the functional-extensions-tests-and-docs are failing. Do you think it's related to this PR, or do we have an issue ?

[aldettinger]

I don't know too. But I think it's not related. It happened on other PRs as well.

[ppalaga]

The documentation xref checks seem to have nothing to do with Camel Quarkus. I see only broken links in Camel and pointing to Camel. Can we safely ignore this, @zregvart?

For this PR I must say I do not understand how these changes fix the linked issue? You switched from a server side 10s await consumerTemplate.receiveBody() to test side 10s polling which sounds like no substantial change. How is that addressing the wrong document due to test ordering?

[ppalaga]

The documentation xref checks seem to have nothing to do with Camel Quarkus. I see only broken links in Camel and pointing to Camel. Can we safely ignore this, @zregvart?

Unping @zregvart, sorry for disturbing, it looks like there is a solution already: #3110

[zbendhiba]

The documentation xref checks seem to have nothing to do with Camel Quarkus. I see only broken links in Camel and pointing to Camel. Can we safely ignore this, @zregvart?

For this PR I must say I do not understand how these changes fix the linked issue? You switched from a server side 10s await consumerTemplate.receiveBody() to test side 10s polling which sounds like no substantial change. How is that addressing the wrong document due to test ordering?

The problem is not with the time, but what Account is consumed first. it's about testing the id of the account. If the consumers got the other account from another method first, the tests don't pass. So I wanted to poll multiple times until I get the right Account.
Is that make sense ?

[zbendhiba]

@ppalaga well I've done multiple tests locally. It did poll until it finds the right Account, but sometimes 1000s is not enough to poll the first one.
So I'm going to add more time to the consumer

[ppalaga]

Thanks for the explanation, @zbendhiba makes sense

[zbendhiba]

The netty failing tests seems not related to this

[ppalaga]

The netty failing tests seems not related to this

I do not remember to have seen anything like that before. I have restarted the tests to see whether it persists.

[zbendhiba]

thanks @ppalaga