test(nav): drop the two click-dispatch tests that flaked the suite

Removing the #showInfileTree and #searchNav click-dispatch specs added
in fa455de (and then tried to be patched up in 1fdcf51). Even with
no openFile and with explicit FindInFilesUI._closeFindBar cleanup,
triggering those clicks still leaked side effects across
closeTestWindow() + loadProjectInTestWindow() that shifted the
subsequent _validateNavForFiles tests' nav-history ordering, producing
deterministic "Active file to be test.css" timeouts only in full-suite
runs.

Commenting the two specs out was confirmed by the user to restore the
suite. The dispatch-on-click coverage these provided isn't worth the
flake surface; if it's wanted later it should live in a describe block
with isolated beforeAll/afterAll so it can't race the history tests.

Author: abose

test/spec/Extn-NavigationAndHistory-integ-test.js

@@ -344,52 +344,6 @@ define(function (require, exports, module) {
             await _expectNavButton(false, true, "nav forward disabled due to new file open");
         }
 
-        function recordCommands(fn) {
-            const executed = [];
-            const handler = function (event, id) { executed.push(id); };
-            CommandManager.on(CommandManager.EVENT_BEFORE_EXECUTE_COMMAND, handler);
-            try {
-                fn();
-            } finally {
-                CommandManager.off(CommandManager.EVENT_BEFORE_EXECUTE_COMMAND, handler);
-            }
-            return executed;
-        }
-
-        it("Should dispatch NAVIGATE_SHOW_IN_FILE_TREE when #showInfileTree is clicked", function () {
-            // Don't open a file here. `recordCommands` hooks EVENT_BEFORE_EXECUTE_COMMAND,
-            // which fires regardless of whether handleShowInTree's activeFile branch runs.
-            // Opening a file would leak into the session-restore state and shift the
-            // nav-history ordering the subsequent _validateNavForFiles tests depend on.
-            const $btn = $("#showInfileTree");
-            expect($btn.length).toBe(1);
-            const executed = recordCommands(function () {
-                $btn.trigger("click");
-            });
-            expect(executed).toContain(Commands.NAVIGATE_SHOW_IN_FILE_TREE);
-        }, 15000);
-
-        it("Should dispatch CMD_FIND_IN_FILES when #searchNav is clicked", async function () {
-            const $btn = $("#searchNav");
-            expect($btn.length).toBe(1);
-            const executed = recordCommands(function () {
-                $btn.trigger("click");
-            });
-            expect(executed).toContain(Commands.CMD_FIND_IN_FILES);
-            // The click opens the Find-in-Files bar; close it before the
-            // next test's beforeEach reloads, otherwise the modal bar leaks
-            // across the reload and subsequent navigation tests fail with
-            // "Active file to be ... timed out" because the new window's
-            // focus/key handling is stuck behind the stale bar.
-            const FindInFilesUI = testWindow.brackets.test.FindInFilesUI;
-            if (FindInFilesUI && FindInFilesUI._closeFindBar) {
-                FindInFilesUI._closeFindBar(true);
-            }
-            await awaitsFor(function () {
-                return $(".modal-bar").length === 0;
-            }, "find bar to close", 2000);
-        }, 15000);
-
         it("Should navigate back and forward between text files", async function () {
             await navigateResetStack();
             await _validateNavForFiles("test.css", "test.html", "test.js");