From 29b469f1c4d3c9ad20dff2126d475b215b0ffc64 Mon Sep 17 00:00:00 2001 From: Yury Samkevich Date: Wed, 19 Feb 2025 02:54:29 -0800 Subject: [PATCH] fix test_representative_config_flags_incorrectly_contain_run_args Summary: The test was introduced in D69508158 and never since worked on Windows. https://www.internalfb.com/intern/test/844425130309909?ref_report_id=0 It fails with: ``` %1 is not a valid Win32 application. (os error 193) ``` This diff is fixing the test for windows. Reviewed By: blackm00n Differential Revision: D69793544 fbshipit-source-id: db9fc4b24677224db0df75cef630ba66b643a723 --- .../core/log/test_representative_config_flags_data/prelude.bzl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/core/log/test_representative_config_flags_data/prelude.bzl b/tests/core/log/test_representative_config_flags_data/prelude.bzl index 3aea7b0457b03..e88e8b235c324 100644 --- a/tests/core/log/test_representative_config_flags_data/prelude.bzl +++ b/tests/core/log/test_representative_config_flags_data/prelude.bzl @@ -6,7 +6,7 @@ # of this source tree. def _action(ctx): - return [DefaultInfo(), RunInfo(ctx.attrs.src)] + return [DefaultInfo(), RunInfo(cmd_args("python3", ctx.attrs.src))] my_rule = rule( impl = _action,