Test command shouldn't update branding.

brave · Nov 19, 2022 · 5a93194 · 5a93194
1 parent cc79dee
commit 5a93194
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/build/commands/lib/build.js b/build/commands/lib/build.js
@@ -23,7 +23,8 @@ const build = (buildConfig = config.defaultBuildConfig, options) => {
   config.update(options)
   checkVersionsMatch()
 
-  util.touchOverriddenFilesAndUpdateBranding()
+  util.touchOverriddenFiles()
+  util.updateBranding()
 
   if (config.xcode_gen_target) {
     util.generateXcodeWorkspace()

diff --git a/build/commands/lib/test.js b/build/commands/lib/test.js
@@ -107,7 +107,7 @@ const test = (passthroughArgs, suite, buildConfig = config.defaultBuildConfig, o
   } else {
     config.buildTarget = suite
   }
-  util.touchOverriddenFilesAndUpdateBranding()
+  util.touchOverriddenFiles()
   util.buildTarget()
 
   // Filter out upstream tests that are known to fail for Brave

diff --git a/build/commands/lib/util.js b/build/commands/lib/util.js
@@ -405,7 +405,7 @@ const util = {
     }
   },
 
-  touchOverriddenFiles: () => {
+  touchOverriddenChromiumSrcFiles: () => {
     console.log('touch original files overridden by chromium_src...')
 
     // Return true when original file of |file| should be touched.
@@ -465,10 +465,9 @@ const util = {
     })
   },
 
-  touchOverriddenFilesAndUpdateBranding: () => {
-    util.touchOverriddenFiles()
+  touchOverriddenFiles: () => {
+    util.touchOverriddenChromiumSrcFiles()
     util.touchOverriddenVectorIconFiles()
-    util.updateBranding()
   },
 
   // Chromium compares pre-installed midl files and generated midl files from IDL during the build to check integrity.