commit 7b02eaa837fabf0e3d0e810690e51d55506529ab
author Googler <tjgq@google.com> Tue Jun 18 07:25:08 2024 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Jun 18 07:26:43 2024 -0700
tree 5fe215f1261d4ceeab8af217810d6ec3963c6e46
parent 2562707390fed314e2fb56eb6f92afa964117d25

Use the generic term "config file" instead of the product-specific ".blazerc" in error messages.

Fixes #22685.

PiperOrigin-RevId: 644377301
Change-Id: I05e7e01f1b4f79c219aa974871efcbc41966966c

src/main/cpp/rc_file.cc

diff --git a/src/main/cpp/rc_file.cc b/src/main/cpp/rc_file.cc
index 2ad9d6d..01016e0 100644
--- a/src/main/cpp/rc_file.cc
+++ b/src/main/cpp/rc_file.cc
@@ -58,8 +58,8 @@
   string error_message;
   if (!blaze_util::ReadFile(filename, &contents, &error_message)) {
     blaze_util::StringPrintf(error_text,
-        "Unexpected error reading .blazerc file '%s': %s",
-        filename.c_str(), error_message.c_str());
+                             "Unexpected error reading config file '%s': %s",
+                             filename.c_str(), error_message.c_str());
     return ParseError::UNREADABLE_FILE;
   }
   const std::string canonical_filename =
@@ -106,7 +106,7 @@
            command == kCommandImport)) {
         blaze_util::StringPrintf(
             error_text,
-            "Invalid import declaration in .blazerc file '%s': '%s'"
+            "Invalid import declaration in config file '%s': '%s'"
             " (are you in your source checkout/WORKSPACE?)",
             canonical_filename.c_str(), line.c_str());
         return ParseError::INVALID_FORMAT;

src/test/cpp/rc_file_test.cc

diff --git a/src/test/cpp/rc_file_test.cc b/src/test/cpp/rc_file_test.cc
index ee8880b..1115154 100644
--- a/src/test/cpp/rc_file_test.cc
+++ b/src/test/cpp/rc_file_test.cc
@@ -794,7 +794,7 @@
   const std::vector<std::string> args = {"bazel", "build"};
   ParseOptionsAndCheckOutput(
       args, blaze_exit_code::INTERNAL_ERROR,
-      "Unexpected error reading .blazerc file '.*myimportedbazelrc'", "");
+      "Unexpected error reading config file '.*myimportedbazelrc'", "");
 }
 
 TEST_F(BlazercImportTest, BazelRcTryImportDoesNotFailForMissingFile) {

src/test/cpp/rc_options_test.cc

diff --git a/src/test/cpp/rc_options_test.cc b/src/test/cpp/rc_options_test.cc
index c7cec75..0a1bf28 100644
--- a/src/test/cpp/rc_options_test.cc
+++ b/src/test/cpp/rc_options_test.cc
@@ -406,10 +406,11 @@
   EXPECT_EQ(error, RcFile::ParseError::UNREADABLE_FILE);
   ASSERT_THAT(
       error_text,
-      MatchesRegex(kIsWindows
-        ? "Unexpected error reading \\.blazerc file '.*not_a_file\\.bazelrc':.*"
-        : "Unexpected error reading \\.blazerc file '.*not_a_file\\.bazelrc': "
-          "\\(error: 2\\): No such file or directory"));
+      MatchesRegex(kIsWindows ? "Unexpected error reading config file "
+                                "'.*not_a_file\\.bazelrc':.*"
+                              : "Unexpected error reading config file "
+                                "'.*not_a_file\\.bazelrc': "
+                                "\\(error: 2\\): No such file or directory"));
 }
 
 TEST_F(RcOptionsTest, ImportedFileDoesNotExist) {
@@ -422,12 +423,14 @@
       Parse("import_fake_file.bazelrc", &error, &error_text);
   EXPECT_EQ(error, RcFile::ParseError::UNREADABLE_FILE);
   if (kIsWindows) {
-    ASSERT_THAT(error_text, MatchesRegex(
-      "Unexpected error reading \\.blazerc file 'somefile':.*"));
+    ASSERT_THAT(
+        error_text,
+        MatchesRegex("Unexpected error reading config file 'somefile':.*"));
   } else {
-    ASSERT_EQ(error_text,
-      "Unexpected error reading .blazerc file 'somefile': (error: 2): No such "
-      "file or directory");
+    ASSERT_EQ(
+        error_text,
+        "Unexpected error reading config file 'somefile': (error: 2): No such "
+        "file or directory");
   }
 }
 
@@ -449,7 +452,7 @@
   EXPECT_EQ(error, RcFile::ParseError::INVALID_FORMAT);
   ASSERT_THAT(
       error_text,
-      MatchesRegex("Invalid import declaration in .blazerc file "
+      MatchesRegex("Invalid import declaration in config file "
                    "'.*bad_import.bazelrc': 'import somefile bar' \\(are you "
                    "in your source checkout/WORKSPACE\\?\\)"));
 }
@@ -463,7 +466,7 @@
   EXPECT_EQ(error, RcFile::ParseError::INVALID_FORMAT);
   ASSERT_THAT(
       error_text,
-      MatchesRegex("Invalid import declaration in .blazerc file "
+      MatchesRegex("Invalid import declaration in config file "
                    "'.*bad_import.bazelrc': 'try-import somefile bar' \\(are "
                    "you in your source checkout/WORKSPACE\\?\\)"));
 }