From cfe81dfbf428db08df324349de0c72c58468abd6 Mon Sep 17 00:00:00 2001 From: Ben Sander Date: Fri, 12 May 2017 17:02:31 -0500 Subject: [PATCH] Update tests README [ROCm/hip commit: c67b828a5aae908ec3608f1be46919ebaab67fd7] --- projects/hip/tests/README.md | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/projects/hip/tests/README.md b/projects/hip/tests/README.md index 223bd149dc..cb41cc10cd 100644 --- a/projects/hip/tests/README.md +++ b/projects/hip/tests/README.md @@ -59,5 +59,9 @@ Find the test and commandline that fail: grep -IR hipMemcpy-modes -IR ../tests/ ../tests/src/runtimeApi/memory/hipMemcpy.cpp: * RUN_NAMED: %t hipMemcpy-modes --tests 0x1 +# Guidelines for adding new tests +- Prefer to enhance an existing test as opposed to writing a new one. Tests have overhead to start and many small tests spend precious test time on startup and initialization issues. +- Make the test run standalone without requirement for command-line arguments. THis makes it easier to debug since the name of the test is shown in the test report and if you know the name of the test you can the run the test. +- For long-running tests or tests with multiple phases, consider using the --tests option as an optional mechanism to allow debuggers to start with the failing subset of the test.