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

fixup! test(MTE)! set default NetworkMode to NONE (#5041) #5043

Merged
merged 1 commit into from
Jun 11, 2022

Conversation

skaldarnar
Copy link
Member

Contains

PR #5041 introdcued a failing test due to a missing network mode setting on the ChunkRegionFutureTest.

This PR adds the networkMode = NetworkMode.LISTEN_SERVER setting for the integration environment to fix it.

How to test

Run the integration tests for the engine and ensure no failures:

gradlew :engine-tests:integrationTest

@skaldarnar skaldarnar added Category: Test/QA Requests, Issues and Changes targeting tests and quality assurance Type: Bug Issues reporting and PRs fixing problems labels Jun 11, 2022
Copy link
Member

@jdrueckert jdrueckert left a comment

Choose a reason for hiding this comment

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

Lgtm

@skaldarnar skaldarnar merged commit d452301 into develop Jun 11, 2022
@skaldarnar skaldarnar deleted the test/fix-failing-future-test branch June 11, 2022 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Test/QA Requests, Issues and Changes targeting tests and quality assurance Type: Bug Issues reporting and PRs fixing problems
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants