Review comments fix:1

Author: maheshrajus

tez-ui/README.md

@@ -77,7 +77,39 @@ You will need the following things properly installed on your computer.
 
 ### Running Tests
 
-* `yarn test`
+Tests run in **headless Chrome**
+Google Chrome must be installed on your machine.
+
+**Step 1: Set `CHROME_BIN` to your Chrome binary**
+
+On Linux:
+```bash
+export CHROME_BIN=google-chrome-stable
+```
+On macOS:
+```bash
+export CHROME_BIN="/Applications/Google Chrome.app/Contents/MacOS/Google Chrome"
+```
+
+**Step 2: Run the tests**
+
+Via Maven (from the `tez-ui` directory):
+```bash
+mvn clean install
+```
+If `CHROME_BIN` is not set in your environment, you can also pass the Chrome path
+directly as a Maven property:
+```bash
+mvn clean install -Dchrome.bin="/path/to/Google Chrome"
+```
+Via Yarn (from inside `src/main/webapp`):
+```bash
+yarn test
+```
+To skip tests:
+```bash
+mvn clean package -DskipTests
+```
 
 ### Building
 

tez-ui/pom.xml

@@ -39,6 +39,13 @@
     <allow-root-build>--allow-root=false</allow-root-build>
 
     <skipTests>false</skipTests>
+
+    <!--
+      Chrome binary used by testem for headless UI tests.
+      Override on the command line: -Dchrome.bin="/path/to/chrome"
+      Falls back to google-chrome-stable when not overridden.
+    -->
+    <chrome.bin>google-chrome-stable</chrome.bin>
   </properties>
 
   <profiles>
@@ -91,6 +98,9 @@
                   <argument>run</argument>
                   <argument>test:mvn</argument>
                 </arguments>
+                <environmentVariables>
+                  <CHROME_BIN>${chrome.bin}</CHROME_BIN>
+                </environmentVariables>
               </configuration>
             </execution>
           </executions>
@@ -149,6 +159,9 @@
                   <argument>run</argument>
                   <argument>test:mvn</argument>
                 </arguments>
+                <environmentVariables>
+                  <CHROME_BIN>${chrome.bin}</CHROME_BIN>
+                </environmentVariables>
               </configuration>
             </execution>
           </executions>

tez-ui/src/main/webapp/package.json

@@ -63,6 +63,7 @@
     "em-tgraph": "0.0.14"
   },
   "resolutions": {
+    "moment": "2.30.1",
     "**/form-data/async": "2.6.4",
     "**/mkdirp/minimist": "1.2.6",
     "**/optimist/minimist": "1.2.6",

tez-ui/src/main/webapp/testem.json

@@ -4,6 +4,12 @@
   "disable_watching": true,
   "browser_start_timeout": 120,
   "timeout": 300,
+  "launchers": {
+    "Chrome": {
+      "command": "\"${CHROME_BIN:-google-chrome-stable}\" --headless --disable-gpu --no-sandbox --disable-dev-shm-usage --no-default-browser-check --no-first-run --ignore-certificate-errors --test-type <url>",
+      "protocol": "browser"
+    }
+  },
   "launch_in_ci": [
     "Chrome"
   ],

tez-ui/src/main/webapp/yarn.lock

@@ -3478,9 +3478,9 @@ moment-timezone@^0.3.0:
   dependencies:
     moment ">= 2.6.0"
 
-"moment@>= 2.29.4":
-  version "2.29.4"
-  resolved "https://registry.yarnpkg.com/moment/-/moment-2.29.4.tgz#3dbe052889fe7c1b2ed966fcb3a77328964ef108"
+moment@2.30.1, "moment@>= 2.6.0":
+  version "2.30.1"
+  resolved "https://registry.yarnpkg.com/moment/-/moment-2.30.1.tgz#f8c91c07b7a786e30c59926df530b4eac96974ae"
 
 morgan@^1.5.2:
   version "1.8.1"