Cold-Air-Networks/mirror-opencl-cts
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Make CL_CONFORMANCE_RESULTS_FILENAME Bazel-aware (#2629)
Some checks failed
Presubmit / Build ubuntu-22.04 gl (push) Failing after 1s
Details
Presubmit / Build ubuntu-22.04 aarch64 debug (push) Failing after 1s
Details
Presubmit / Build ubuntu-22.04 android-aarch64 (push) Failing after 1s
Details
Presubmit / Check code format (push) Failing after 1s
Details
Presubmit / Build macos-latest (push) Has been cancelled
Details
Presubmit / Build windows-11-arm (push) Has been cancelled
Details
Presubmit / Build windows-latest (push) Has been cancelled
Details

Browse source 
Bazel requires that test outputs are put in a specific directory given
by $TEST_UNDECLARED_OUTPUTS_DIR when running a test through Bazel test

This patch checks for the environment variable $BAZEL_TEST, which Bazel
sets when running tests, and prepends the specified directory to the
user-provided path.

The behaviour when running outside of a bazel test environment is
unchanged
This commit is contained in:
Ole Strohm 2026-03-17 16:54:32 +00:00 committed by GitHub
parent 0338fd9d39
commit 4a923c074b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 24 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
test_common/harness/testHarness.cpp
View file

@ -14,6 +14,7 @@
// limitations under the License.
//
#include "testHarness.h"
#include "stringHelpers.h"
#include "compat.h"
#include <algorithm>
#include <stdio.h>
@ -21,6 +22,7 @@
#include <string.h>
#include <cassert>
#include <deque>
#include <filesystem>
#include <mutex>
#include <set>
#include <stdexcept>
@ -33,6 +35,8 @@
#include "imageHelpers.h"
#include "parseParameters.h"
namespace fs = std::filesystem;
#if !defined(_WIN32)
#include <sys/utsname.h>
#include <unistd.h>
@ -95,11 +99,25 @@ static int saveResultsToJson(const char *suiteName, test_definition testList[],
return EXIT_SUCCESS;
}
FILE *file = fopen(fileName, "w");
fs::path file_path(fileName);
// When running under Bazel test, prepend the Bazel output directory to
// the provided path
if (nullptr != getenv("BAZEL_TEST"))
{
char *bazel_output_dir = getenv("TEST_UNDECLARED_OUTPUTS_DIR");
if (nullptr != bazel_output_dir)
{
file_path = fs::path(bazel_output_dir) / file_path;
}
}
auto file_path_str = to_string(file_path.u8string());
FILE *file = fopen(file_path_str.c_str(), "w");
if (NULL == file)
{
log_error("ERROR: Failed to open '%s' for writing results.\n",
fileName);
file_path_str.c_str());
return EXIT_FAILURE;
}
@ -128,7 +146,8 @@ static int saveResultsToJson(const char *suiteName, test_definition testList[],
int ret = fclose(file) ? EXIT_FAILURE : EXIT_SUCCESS;
log_info("Saving results to %s: %s!\n", fileName, save_map[ret]);
log_info("Saving results to %s: %s!\n", file_path_str.c_str(),
save_map[ret]);
return ret;
}
@ -309,6 +328,8 @@ int runTestHarnessWithCheck(int argc, const char *argv[], int testNum,
"CL_CONFORMANCE_RESULTS_FILENAME (currently '%s')\n",
fileName != NULL ? fileName : "<undefined>");
log_info("\t to save results to JSON file.\n");
log_info("\t When running in Bazel test this is relative to "
"$TEST_UNDECLARED_OUTPUTS_DIR.\n");
log_info("\n");
log_info("Test names:\n");