test: integ for escape key should not toggle panel if code hints popup visible

Author: abose

src/features/ParameterHintsManager.js

@@ -231,11 +231,11 @@ define(function (require, exports, module) {
      *
      */
     function dismissHint(editor) {
+        popupShown = false;
         if (hintState.visible) {
             $hintContainer.hide();
             $hintContent.empty();
             hintState = {};
-            popupShown = false;
 
             if (editor) {
                 editor.off("cursorActivity.ParameterHinting", handleCursorActivity);

test/spec/MainViewManager-integ-test.js

@@ -19,7 +19,7 @@
  *
  */
 
-/*global describe, beforeEach, beforeAll, afterAll, afterEach, it, expect, awaitsForDone, spyOn, jasmine, Phoenix */
+/*global describe, beforeEach, beforeAll, afterAll, it, expect, awaitsForDone, spyOn, jasmine, awaitsFor */
 
 define(function (require, exports, module) {
 
@@ -1009,6 +1009,28 @@ define(function (require, exports, module) {
                 expect(WorkspaceManager.removeEscapeKeyEventHandler("x")).toBeTrue();
             });
 
+            it("should not dismiss any panel if codehints are visible", async function () {
+                panel1.show();
+                panel2.hide();
+                expect(panel1.isVisible()).toBeTrue();
+
+                expect(MainViewManager.getActivePaneId()).toEqual("first-pane");
+                promise = MainViewManager._open(MainViewManager.FIRST_PANE, FileSystem.getFileForPath(testPath + "/test.js"));
+                await awaitsForDone(promise, "MainViewManager.doOpen");
+                let editor = EditorManager.getActiveEditor();
+                editor.setCursorPos(0, 0);
+                await awaitsForDone(CommandManager.execute(Commands.SHOW_CODE_HINTS));
+                await awaitsFor(function () {
+                    return testWindow.$(".codehint-menu").is(":visible");
+                }, "codehints to be shown");
+
+                SpecRunnerUtils.simulateKeyEvent(KeyEvent.DOM_VK_ESCAPE, "keydown", _$("#editor-holder")[0]);
+                await awaitsFor(function () {
+                    return !testWindow.$(".codehint-menu").is(":visible");
+                }, "codehints to be dismissed on escape press");
+                expect(panel1.isVisible()).toBeTrue();
+            });
+
             it("should escape close bottom panel one by one", async function () {
                 panel1.show();
                 expect(panel1.isVisible()).toBeTrue();