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

re-enable runall tests under CTest #120

Open
derekbruening opened this issue Nov 27, 2014 · 2 comments
Open

re-enable runall tests under CTest #120

derekbruening opened this issue Nov 27, 2014 · 2 comments

Comments

@derekbruening
Copy link
Contributor

From derek.br...@gmail.com on April 12, 2009 23:12:23

splitting from issue #65 This case covers writing a CMake script to replace the old runall shell script.

Now that we have 64-bit tools ( issue #28 ) we should be able to have both
32-bit and 64-bit runall tests.

Xref issue #111 where we want to support runall tests in parallel.

Original issue: http://code.google.com/p/dynamorio/issues/detail?id=120

@derekbruening
Copy link
Contributor Author

From derek.br...@gmail.com on October 08, 2009 11:20:23

in r226 :

  • added suite/tests/runall.cmake to drive the testing of nudges.
    this is also a step toward re-enabling the runall tests on Windows ( Issue 120 ).

@derekbruening
Copy link
Contributor Author

From bruen...@google.com on March 06, 2014 13:44:01

To get started on this, see all of the "issue #120" comments in suite/tests/CMakeLists.txt.

Owner: ---
Labels: GoodContrib

derekbruening added a commit that referenced this issue Feb 20, 2018
Changes the libutil nudge code to use NtCreateThreadEx on Win8+.

For a test, enables the client.nudge_test test, part of i#120: enable
.runall tests on Windows.  Here I added a new app win32.infloop to parallel
linux.infloop, rather than launching calc.exe or notepad.exe like we did in
the past (not yet tackling systemwide injection here).  It has a MessageBox
with a title containing the pid, allowing us to use tools/closewnd with a
unique target name for a clean and race-free exit.  win32.infloop also has
a 3-minute timeout to avoid leaving stale processes behind in case of
issues closing it externally.

Revamped runall.cmake to work on Windows, using "ping" to sleep,
updating nudge and close commands, removing stale pid files, etc.

Ported suite code to use run_in_bg on Windows as well as Linux,
but with the pidfile coming from drrun.

Fixes #1309
Fixes #1432
Issue: #120, #1835
derekbruening added a commit that referenced this issue Feb 20, 2018
Changes the libutil nudge code to use NtCreateThreadEx on Win8+.

For a test, enables the client.nudge_test test, part of i#120: enable
.runall tests on Windows.  Here I added a new app win32.infloop to parallel
linux.infloop, rather than launching calc.exe or notepad.exe like we did in
the past (not yet tackling systemwide injection here).  It has a MessageBox
with a title containing the pid, allowing us to use tools/closewnd with a
unique target name for a clean and race-free exit.  win32.infloop also has
a 3-minute timeout to avoid leaving stale processes behind in case of
issues closing it externally.

Revamped runall.cmake to work on Windows, using "ping" to sleep,
updating nudge and close commands, removing stale pid files, etc.

Ported suite code to use run_in_bg on Windows as well as Linux,
but with the pidfile coming from drrun.

Fixes #1309
Fixes #1432
Issue: #120, #1835
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant