Fix Integ tests

Signed-off-by: Vaibhav Agarwal <vaibhoag@amazon.com>

Author: vaibhoag

.github/actions/run-cypress-tests/action.yaml

@@ -131,6 +131,8 @@ runs:
         command: yarn run cypress run --config-file cypress.config.js
         wait-on: 'http://localhost:5601'
         browser: chrome
+      env:
+        NODE_OPTIONS: '--expose-gc'
     - name: Cypress tests
       uses: cypress-io/github-action@v5
       if: ${{ inputs.with-security == 'true' }}
@@ -139,6 +141,8 @@ runs:
         command: yarn run cypress run --config-file cypress.config.js --env SECURITY_ENABLED=true,openSearchUrl=https://localhost:9200,WAIT_FOR_LOADER_BUFFER_MS=500
         wait-on: 'http://localhost:5601'
         browser: chrome
+      env:
+        NODE_OPTIONS: '--expose-gc'
     # Screenshots are only captured on failure, will change this once we do visual regression tests
     - uses: actions/upload-artifact@v4
       if: failure()

.github/workflows/cypress-workflow.yml

@@ -8,8 +8,8 @@ on:
       - "*"
 env:
   OPENSEARCH_DASHBOARDS_VERSION: 'main'
-  CYPRESS_MEMORY_LIMIT: '40960'
-  NODE_OPTIONS: '--max-old-space-size=40960'
+  CYPRESS_MEMORY_LIMIT: '65536'  # Increase to 64GB
+  NODE_OPTIONS: '--max-old-space-size=65536'
 jobs:
   tests:
     name: Run Cypress E2E tests

cypress/e2e/plugins/index-management-dashboards-plugin/split_index.js

@@ -16,13 +16,28 @@ describe("Split Index", () => {
     cy.wait(6000);
   });
 
+  after(() => {
+    if (global.gc) global.gc();
+  });
+
   describe("can be created and updated", () => {
     beforeEach(() => {
       // Visit ISM OSD
       cy.visit(`${BASE_PATH}/app/${IM_PLUGIN_NAME}#/indices`);
       cy.contains("Rows per page", { timeout: 60000 });
     });
 
+    afterEach(() => {
+      // Force garbage collection after each test
+      if (global.gc) global.gc();
+      // Clear any Monaco editor instances
+      cy.window().then((win) => {
+        if (win.monaco && win.monaco.editor) {
+          win.monaco.editor.getModels().forEach((model) => model.dispose());
+        }
+      });
+    });
+
     let splitNumber = 2;
     let replicaNumber = 1;
     it("Create an index successfully", () => {
@@ -110,12 +125,18 @@ describe("Split Index", () => {
         .get('[aria-controls="accordionForCreateIndexSettings"]')
         .click()
         .end()
+        .get('[data-test-subj="codeEditorContainer"]')
+        .should("be.visible")
+        .wait(1000)
         .get('[data-test-subj="codeEditorContainer"] textarea')
         .focus()
+        .wait(500)
         // Need to remove the default {} in advanced setting
         .clear()
+        .wait(500)
         .type(`{"index.number_of_shards": "${splitNumber}", "index.number_of_replicas": "${replicaNumber}"}`, {
           parseSpecialCharSequences: false,
+          delay: 50,
         })
         .end()
         .get('[data-test-subj="splitButton"]', { timeout: 10000 })

public/components/MonacoJSONEditor/MonacoJSONEditor.tsx

@@ -74,6 +74,10 @@ const MonacoJSONEditor = forwardRef(
     useEffect(() => {
       return () => {
         onClickOutsideHandler.current();
+        // Cleanup Monaco editor to prevent memory leaks
+        if (editorRef.current) {
+          editorRef.current.dispose();
+        }
       };
     }, []);
 
@@ -127,6 +131,17 @@ const MonacoJSONEditor = forwardRef(
             editorDidMount={(editor) => {
               editorRef.current = editor;
               setIsReady(true);
+              // Add memory management for large JSON files
+              editor.updateOptions({
+                wordWrap: "on",
+                minimap: { enabled: false },
+                scrollBeyondLastLine: false,
+                renderLineHighlight: "none",
+                occurrencesHighlight: false,
+                selectionHighlight: false,
+                overviewRulerBorder: false,
+                hideCursorInOverviewRuler: true,
+              });
             }}
           />
         </div>

public/components/SwitchableEditor/SwitchableEditor.tsx

@@ -46,7 +46,7 @@ const SwitchableEditor = forwardRef(
         ) : null}
         {checked ? (
           diagnosticsOptions ? (
-            <MonacoJSONEditor path={path} {...others} ref={ref} />
+            <MonacoJSONEditor path={path} {...others} ref={editorRef} />
           ) : (
             <JSONDiffEditor {...others} ref={editorRef} />
           )