Skip to content

Convert all tests to use TIMED_TEST_SUITE_INITIALIZE/CLEANUP#621

Open
parth21999 wants to merge 3 commits intomasterfrom
paaggarwal/convert-to-timed-test-suite
Open

Convert all tests to use TIMED_TEST_SUITE_INITIALIZE/CLEANUP#621
parth21999 wants to merge 3 commits intomasterfrom
paaggarwal/convert-to-timed-test-suite

Conversation

@parth21999
Copy link
Copy Markdown
Member

Converts all test files from TEST_SUITE_INITIALIZE/CLEANUP to TIMED_TEST_SUITE_INITIALIZE/CLEANUP.

Changes:

  • Update c-pal submodule to latest master (with timed test suite support)
  • 65 test .c files: macro replacement
  • 49 PCH files: added c_pal/timed_test_suite.h before ENABLE_MOCKS
  • 17 int test .c files: added c_pal/timed_test_suite.h
  • UT CMakeLists: added real_process_watchdog + c_pal
  • Int CMakeLists: added c_pal where missing

Full Windows solution build verified locally - zero C/C++ errors.

- Update c-pal submodule to latest master (with timed test suite support)
- Replace TEST_SUITE_INITIALIZE/CLEANUP with TIMED_ versions in 65 test files
- Add c_pal/timed_test_suite.h to 49 PCH files and 17 int test files
- Add real_process_watchdog and c_pal to UT CMakeLists
- Add c_pal to int test CMakeLists

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@parth21999
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@parth21999
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

The process watchdog uses POSIX timer_create(SIGEV_THREAD) which spawns
a helper thread that valgrind flags as a possible memory leak during
timer_delete cleanup. This is a known false positive.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@parth21999
Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

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.

1 participant