Skip to content

Fix two testNuma tests #733

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

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Sep 16, 2024

Description

Fix two testNuma tests:

  • checkModeInterleave and
  • checkModeInterleaveCustomPartSize

It fixes the Coverity issue no. 468463.

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau requested a review from a team as a code owner September 16, 2024 08:04
@ldorau ldorau force-pushed the Add_asserts_to_testNuma_checkModeInterleaveCustomPartSize branch from 836d862 to 187876b Compare September 16, 2024 08:04
@ldorau ldorau requested a review from lplewa September 16, 2024 09:43
@ldorau
Copy link
Contributor Author

ldorau commented Sep 16, 2024

@lplewa please review

@ldorau ldorau force-pushed the Add_asserts_to_testNuma_checkModeInterleaveCustomPartSize branch from 187876b to fe486c7 Compare September 16, 2024 11:52
Fix two testNuma tests:
- checkModeInterleave and
- checkModeInterleaveCustomPartSize

It fixes the Coverity issue no. 468463.

Signed-off-by: Lukasz Dorau <[email protected]>
@ldorau ldorau force-pushed the Add_asserts_to_testNuma_checkModeInterleaveCustomPartSize branch from fe486c7 to 17d13b7 Compare September 16, 2024 12:48
@ldorau ldorau changed the title Add asserts to testNuma checkModeInterleaveCustomPartSize Fix two testNuma tests Sep 16, 2024
@ldorau ldorau requested a review from bratpiorka September 16, 2024 12:50
@PatKamin
Copy link
Contributor

The fix looks great now, thanks!

@ldorau ldorau merged commit 4234394 into oneapi-src:main Sep 16, 2024
70 checks passed
@ldorau ldorau deleted the Add_asserts_to_testNuma_checkModeInterleaveCustomPartSize branch September 16, 2024 16:22
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.

4 participants