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

Remove TestUtils::invoke_test_if and some extra macros from tests #1831

Merged
merged 6 commits into from
Sep 16, 2024

Conversation

SergeyKopienko
Copy link
Contributor

@SergeyKopienko SergeyKopienko commented Sep 10, 2024

In this PR we remove TestUtils::invoke_test_if from tests :

  • starting from C++17 we can use if constexpr instead.

@SergeyKopienko SergeyKopienko marked this pull request as ready for review September 10, 2024 13:38
@SergeyKopienko SergeyKopienko added this to the 2022.7.0 milestone Sep 10, 2024
@SergeyKopienko SergeyKopienko changed the title Remove TestUtils::invoke_test_if from tests Remove TestUtils::invoke_test_if and some extra macros from tests Sep 10, 2024
@akukanov akukanov removed this from the 2022.7.0 milestone Sep 10, 2024
@akukanov
Copy link
Contributor

No reason to expedite this patch; removed from the 2022.7 milestone.

Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
…sed anymore

Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
Signed-off-by: Sergey Kopienko <sergey.kopienko@intel.com>
@SergeyKopienko SergeyKopienko force-pushed the dev/skopienko/remove_invoke_test_if branch 2 times, most recently from 7b2c0e7 to f8f6e04 Compare September 16, 2024 07:47
@SergeyKopienko SergeyKopienko merged commit f8b2122 into main Sep 16, 2024
22 checks passed
@SergeyKopienko SergeyKopienko deleted the dev/skopienko/remove_invoke_test_if branch September 16, 2024 08:43
This pull request was closed.
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.

3 participants