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

SourceWithContextSpec test that fails intermittently #1610

Closed
pjfanning opened this issue Dec 17, 2024 · 2 comments
Closed

SourceWithContextSpec test that fails intermittently #1610

pjfanning opened this issue Dec 17, 2024 · 2 comments
Labels
flaky Related to flaky tests
Milestone

Comments

@pjfanning
Copy link
Contributor

pjfanning commented Dec 17, 2024

  • Was part of a 'Nightly Build' but one that was temporarily enabled to run for PRs.
  • Failed in a few of the matrix of runs but passed in most runs
[12-17 12:13:48.607] [info] - must Apply a viaFlow with optional elements using unsafeOptionalVia *** FAILED *** (4 seconds, 24 milliseconds)
[12-17 12:13:48.607] [info]   java.lang.AssertionError: assertion failed: expected OnNext((Some(1),1)), found OnNext((None,2))
[12-17 12:13:48.607] [info]   at scala.Predef$.assert(Predef.scala:223)
[12-17 12:13:48.607] [info]   at org.apache.pekko.testkit.TestKitBase.expectMsg_internal(TestKit.scala:473)
[12-17 12:13:48.607] [info]   at org.apache.pekko.testkit.TestKitBase.expectMsg(TestKit.scala:449)
[12-17 12:13:48.607] [info]   at org.apache.pekko.testkit.TestKitBase.expectMsg$(TestKit.scala:449)
[12-17 12:13:48.607] [info]   at org.apache.pekko.testkit.TestKit.expectMsg(TestKit.scala:982)
[12-17 12:13:48.607] [info]   at org.apache.pekko.stream.testkit.TestSubscriber$ManualProbe.$anonfun$expectNextN$1(StreamTestKit.scala:585)
[12-17 12:13:48.607] [info]   at scala.collection.Iterator.foreach(Iterator.scala:943)
[12-17 12:13:48.607] [info]   at scala.collection.Iterator.foreach$(Iterator.scala:943)
[12-17 12:13:48.607] [info]   at scala.collection.AbstractIterator.foreach(Iterator.scala:1431)
[12-17 12:13:48.608] [info]   at scala.collection.IterableLike.foreach(IterableLike.scala:74)
[12-17 12:13:48.608] [info]   at scala.collection.IterableLike.foreach$(IterableLike.scala:73)
[12-17 12:13:48.608] [info]   at scala.collection.AbstractIterable.foreach(Iterable.scala:56)
[12-17 12:13:48.608] [info]   at org.apache.pekko.stream.testkit.TestSubscriber$ManualProbe.expectNextN(StreamTestKit.scala:585)
[12-17 12:13:48.608] [info]   at org.apache.pekko.stream.testkit.TestSubscriber$ManualProbe.expectNext(StreamTestKit.scala:554)
[12-17 12:13:48.608] [info]   at org.apache.pekko.stream.scaladsl.SourceWithContextSpec.$anonfun$new$55(SourceWithContextSpec.scala:251)
@pjfanning pjfanning added the flaky Related to flaky tests label Dec 17, 2024
@He-Pin
Copy link
Member

He-Pin commented Dec 17, 2024

I will look into this weekend

@He-Pin
Copy link
Member

He-Pin commented Dec 17, 2024

image
I looked into it, the current implementation is using a Merge which will not keep the orgin order.
@mdedetrich cc

So I think the current implementation is broken.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky Related to flaky tests
Projects
None yet
Development

No branches or pull requests

2 participants