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

fix not working JournalDaoStreamMessagesMemoryTest #831

Merged
merged 3 commits into from
May 7, 2024

Conversation

Roiocam
Copy link
Contributor

@Roiocam Roiocam commented Apr 26, 2024

JournalDaoStreamMessagesMemoryTest didn't work in the newDao because it uses legacyDao, we could create dao instance via reflection to make it works.

@Roiocam
Copy link
Contributor Author

Roiocam commented Apr 26, 2024

@johanandren @octonato would you like to take a look?

@Roiocam Roiocam marked this pull request as draft April 26, 2024 03:11
@Roiocam Roiocam marked this pull request as ready for review April 26, 2024 05:47
@octonato
Copy link
Member

octonato commented May 6, 2024

Was this test not being run? How was it passing before?

@johanandren
Copy link
Member

@octonato if (newDao) pending

Copy link
Member

@johanandren johanandren left a comment

Choose a reason for hiding this comment

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

LGTM, I verified in test runs that it is executing and pass (takes 42s on oracle though 😱)

@johanandren johanandren merged commit 646c051 into akka:main May 7, 2024
10 checks passed
@johanandren
Copy link
Member

I see I missed one thing though, we lost the class H2JournalDaoStreamMessagesMemoryTest, I'll follow up and re-introduce

johanandren added a commit that referenced this pull request May 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants