From af26d983e5e20c64f0f73d22ebd998f54dff1e8a Mon Sep 17 00:00:00 2001
From: Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com>
Date: Wed, 26 Jul 2017 13:45:44 +0100
Subject: [PATCH] Fixup tests following label changes.

---
 .../sqleditor/keyboard_shortcuts_spec.js      | 37 ++++++++++---------
 1 file changed, 20 insertions(+), 17 deletions(-)

diff --git a/web/regression/javascript/sqleditor/keyboard_shortcuts_spec.js b/web/regression/javascript/sqleditor/keyboard_shortcuts_spec.js
index d64a50432..9f89482a2 100644
--- a/web/regression/javascript/sqleditor/keyboard_shortcuts_spec.js
+++ b/web/regression/javascript/sqleditor/keyboard_shortcuts_spec.js
@@ -14,9 +14,9 @@ describe('the keyboard shortcuts', () => {
     F5_KEY = 116,
     F7_KEY = 118,
     F8_KEY = 119,
-    COMMA_KEY = 188,
     PERIOD_KEY = 190,
-    FWD_SLASH_KEY = 191;
+    FWD_SLASH_KEY = 191,
+    isMacSystem = window.navigator.platform.search('Mac') != -1;
 
   let sqlEditorControllerSpy;
   let event;
@@ -164,12 +164,13 @@ describe('the keyboard shortcuts', () => {
     });
   });
 
-  describe('Shift+CTRL+Comma', () => {
+  describe('inlineComment', () => {
     describe('when there is not a query already running', () => {
       beforeEach(() => {
+        event.metaKey = isMacSystem;
         event.shiftKey = true;
-        event.ctrlKey = true;
-        event.which = COMMA_KEY;
+        event.ctrlKey = !isMacSystem;
+        event.which = FWD_SLASH_KEY;
         keyboardShortcuts.processEvent(sqlEditorControllerSpy, event);
       });
 
@@ -182,9 +183,9 @@ describe('the keyboard shortcuts', () => {
 
     describe('when the query is already running', () => {
       it('does nothing', () => {
-        event.shiftKey = true;
-        event.ctrlKey = true;
-        event.which = COMMA_KEY;
+        event.shiftKey = isMacSystem;
+        event.ctrlKey = !isMacSystem;
+        event.which = FWD_SLASH_KEY;
         sqlEditorControllerSpy.isQueryRunning.and.returnValue(true);
 
         keyboardShortcuts.processEvent(sqlEditorControllerSpy, event);
@@ -194,11 +195,12 @@ describe('the keyboard shortcuts', () => {
     });
   });
 
-  describe('Shift+CTRL+Period', () => {
+  describe('inlineUncomment', () => {
     describe('when there is not a query already running', () => {
       beforeEach(() => {
+        event.metaKey = isMacSystem;
         event.shiftKey = true;
-        event.ctrlKey = true;
+        event.ctrlKey = !isMacSystem;
         event.which = PERIOD_KEY;
         keyboardShortcuts.processEvent(sqlEditorControllerSpy, event);
       });
@@ -212,9 +214,10 @@ describe('the keyboard shortcuts', () => {
 
     describe('when the query is already running', () => {
       it('does nothing', () => {
+        event.metaKey = isMacSystem;
         event.shiftKey = true;
-        event.ctrlKey = true;
-        event.which = COMMA_KEY;
+        event.ctrlKey = !isMacSystem;
+        event.which = PERIOD_KEY;
         sqlEditorControllerSpy.isQueryRunning.and.returnValue(true);
 
         keyboardShortcuts.processEvent(sqlEditorControllerSpy, event);
@@ -224,11 +227,11 @@ describe('the keyboard shortcuts', () => {
     });
   });
 
-  describe('Shift+CTRL+/', () => {
+  describe('blockComment', () => {
     describe('when there is not a query already running', () => {
       beforeEach(() => {
-        event.shiftKey = true;
-        event.ctrlKey = true;
+        event.metaKey = isMacSystem;
+        event.ctrlKey = !isMacSystem;
         event.which = FWD_SLASH_KEY;
         keyboardShortcuts.processEvent(sqlEditorControllerSpy, event);
       });
@@ -242,8 +245,8 @@ describe('the keyboard shortcuts', () => {
 
     describe('when the query is already running', () => {
       it('does nothing', () => {
-        event.shiftKey = true;
-        event.ctrlKey = true;
+        event.metaKey = isMacSystem;
+        event.ctrlKey = !isMacSystem;
         event.which = FWD_SLASH_KEY;
         sqlEditorControllerSpy.isQueryRunning.and.returnValue(true);