From fb38bd13cd01caf549dfd6afcea973dcff03395c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C3=B6rg=20Breitbart?= <jerch@rockborn.de>
Date: Wed, 24 Jul 2019 12:51:24 +0200
Subject: [PATCH] fix general purpose test script

---
 bin/test_mousemodes.js        | 77 +++++++++++++++++++++++------------
 test/api/MouseTracking.api.ts |  7 ++--
 2 files changed, 54 insertions(+), 30 deletions(-)

diff --git a/bin/test_mousemodes.js b/bin/test_mousemodes.js
index 8188a3cdc..976a38e00 100644
--- a/bin/test_mousemodes.js
+++ b/bin/test_mousemodes.js
@@ -48,43 +48,66 @@ stdin.addListener('data', function(data) {
   }
 });
 
-// basic button codes (modifier keys are added on top)
-const BUTTONS = {
-  0: ['left', 'press'],
-  1: ['middle', 'press'],
-  2: ['right', 'press'],
-  3: ['', 'release'],
-  32: ['left', 'move'],
-  33: ['middle', 'move'],
-  34: ['right', 'move'],
-  35: ['', 'move'],
-  64: ['wheel', 'up'],
-  65: ['wheel', 'down']
+// button definitions
+const buttons = {
+  '<none>':  -1,
+  left:       0,
+  middle:     1,
+  right:      2,
+  released:   3,
+  wheelUp:    4,
+  wheelDown:  5,
+  wheelLeft:  6,
+  wheelRight: 7,
+  aux1:       8,
+  aux2:       9,
+  aux3:      10,
+  aux4:      11,
+  aux5:      12,
+  aux6:      13,
+  aux7:      14,
+  aux8:      15
 };
+const reverseButtons = {};
+for (const el in buttons) {
+  reverseButtons[buttons[el]] = el;
+}
 
+// extract button data from buttonCode
 function evalButtonCode(code) {
-  // 2 bits: 0 - left, 1 - middle, 2 - right, 3 - release
-  // higher bits: 4 - shift, 8 - meta, 16 - control
-  const modifier = {shift: !!(code & 4), meta: !!(code & 8), control: !!(code & 16)};
-  const wheel = code & 64;
-  let action;
-  let button;
-  if (wheel) {
-    action = (code & 1) ? 'down' : 'up';
-    button = 'wheel';
-  } else {
-    action = code & 32 ? 'move' : code === 3 ? 'release' : 'press';
-    code &= 3; // TODO: more than 3 buttons + wheel
-    button = code === 0 ? 'left' : code === 1 ? 'middle' : code === 2 ? 'right' : '<none>';
+  // more than 15 buttons are not supported
+  if (code > 255) {
+    return {button: 'invalid', action: 'invalid', modifier: {}};
   }
-  return {button, action, modifier};
+  const modifier = {shift: !!(code & 4), meta: !!(code & 8), control: !!(code & 16)};
+  const move = code & 32;
+  let button = code & 3;
+  if (code & 128) {
+    button |= 8;
+  }
+  if (code & 64) {
+    button |= 4
+  }
+  let actionS = 'press';
+  let buttonS = reverseButtons[button];
+  if (button === 3) {
+    buttonS = '<none>';
+    actionS = 'release';
+  }
+  if (move) {
+    actionS = 'move';
+  } else if (4 <= button && button <= 7) {
+    buttonS = 'wheel';
+    actionS = button === 4 ? 'up' : button === 5 ? 'down' : button === 6 ? 'left' : 'right';
+  }
+  return {button: buttonS, action: actionS, modifier};
 }
 
 // protocols
 const PROTOCOLS = {
   '9    (X10: press only)': '\x1b[?9h',
   '1000 (VT200: press, release, wheel)': '\x1b[?1000h',
-  // '1001 (VT200 highlight)': '\x1b[?1001h',  // handle of backreport not implemented
+  // '1001 (VT200 highlight)': '\x1b[?1001h',  // handle of backreport - not implemented
   '1002 (press, release, move on pressed, wheel)': '\x1b[?1002h',
   '1003 (press, relase, move, wheel)': '\x1b[?1003h'
 }
diff --git a/test/api/MouseTracking.api.ts b/test/api/MouseTracking.api.ts
index 807168896..266edff7e 100644
--- a/test/api/MouseTracking.api.ts
+++ b/test/api/MouseTracking.api.ts
@@ -138,11 +138,12 @@ function evalButtonCode(code: number): any {
   }
   let actionS = 'press';
   let buttonS = reverseButtons[button];
-  if (move) {
-    actionS = 'move';
-  } else if (button === 3) {
+  if (button === 3) {
     buttonS = '<none>';
     actionS = 'release';
+  }
+  if (move) {
+    actionS = 'move';
   } else if (4 <= button && button <= 7) {
     buttonS = 'wheel';
     actionS = button === 4 ? 'up' : button === 5 ? 'down' : button === 6 ? 'left' : 'right';