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

Disable add_output_performance test #24376

Merged

Conversation

olpipi
Copy link
Contributor

@olpipi olpipi commented May 6, 2024

Details:

  • Disable add_output_performance test

Tickets:

@olpipi olpipi requested a review from a team as a code owner May 6, 2024 08:09
@github-actions github-actions bot added the category: Core OpenVINO Core (aka ngraph) label May 6, 2024
@@ -1147,6 +1147,8 @@ TEST(model, add_output_port_to_result) {
}

TEST(model, add_output_performance) {
// skip this test due to CVS-140440
GTEST_SKIP();
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
GTEST_SKIP();
SKIP_IF_CURRENT_TEST_IS_DISABLED();

Copy link
Contributor Author

@olpipi olpipi May 6, 2024

Choose a reason for hiding this comment

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

SKIP_IF_CURRENT_TEST_IS_DISABLED is declared and defined in func test utils.
I don't want to add dependency to func test utils to unit test target.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
GTEST_SKIP();
GTEST_SKIP() << "CVS-140440";

Copy link
Contributor

@iefode iefode left a comment

Choose a reason for hiding this comment

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

Add test to skipped test list

@@ -1147,6 +1147,8 @@ TEST(model, add_output_port_to_result) {
}

TEST(model, add_output_performance) {
// skip this test due to CVS-140440
GTEST_SKIP();
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
GTEST_SKIP();
GTEST_SKIP() << "CVS-140440";

@p-durandin p-durandin enabled auto-merge May 6, 2024 11:13
@p-durandin p-durandin added this pull request to the merge queue May 6, 2024
Merged via the queue into openvinotoolkit:master with commit 4947369 May 6, 2024
114 of 115 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Core OpenVINO Core (aka ngraph)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants