fix lint + unit tests for new API #637

README.md

@@ -10,8 +10,8 @@ https://terminal.jcubic.pl
 
 [![npm](https://img.shields.io/badge/npm-DEV-blue.svg)](https://www.npmjs.com/package/jquery.terminal)
 ![bower](https://img.shields.io/badge/bower-DEV-yellow.svg)
-[![travis](https://travis-ci.org/jcubic/jquery.terminal.svg?branch=devel&8a8437a9fee25cdade9be0a97dda7f6e15f4bf4e)](https://travis-ci.org/jcubic/jquery.terminal)
-[![Coverage Status](https://coveralls.io/repos/github/jcubic/jquery.terminal/badge.svg?branch=devel&829978fd999563a22db4e88d604c0e15)](https://coveralls.io/github/jcubic/jquery.terminal?branch=devel)
+[![travis](https://travis-ci.org/jcubic/jquery.terminal.svg?branch=devel&e8b59ab0777ee3319751b83e75f27dac491ba698)](https://travis-ci.org/jcubic/jquery.terminal)
+[![Coverage Status](https://coveralls.io/repos/github/jcubic/jquery.terminal/badge.svg?branch=devel&634dd6bdb563cac4da5f3e6ae04ce222)](https://coveralls.io/github/jcubic/jquery.terminal?branch=devel)
 ![downloads](https://img.shields.io/npm/dm/jquery.terminal.svg?style=flat)
 [![](https://data.jsdelivr.com/v1/package/npm/jquery.terminal/badge?style=rounded)](https://www.jsdelivr.com/package/npm/jquery.terminal)
 [![LICENSE MIT](https://img.shields.io/badge/license-MIT-blue.svg)](https://github.com/jcubic/jquery.terminal/blob/master/LICENSE)

__tests__/__snapshots__/terminal.spec.js.snap

@@ -1,5 +1,20 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[` 1`] = `
+Object {
+  "char": Object {
+    "height": 0,
+    "width": 0,
+  },
+  "cols": 100,
+  "rows": 25,
+  "terminal": Object {
+    "height": 0,
+    "width": 0,
+  },
+}
+`;
+
 exports[`Terminal plugin cmd plugin should move cursor 1`] = `
 Array [
   5,

__tests__/terminal.spec.js

@@ -2964,6 +2964,17 @@ describe('Terminal plugin', function() {
                 expect(term.history().data()).toEqual([]);
             });
         });
+        describe('geometry', function() {
+            var term = $('<div/>').css({
+                width: 800,
+                height: 600
+            }).appendTo('body').terminal($.noop, {
+                numChars: 100,
+                numRows: 25
+            });
+            expect(term.geometry()).toMatchSnapshot();
+            term.remove();
+        });
         describe('exit', function() {
             it('should add exit command', function() {
                 var term = $('<div/>').terminal($.noop, {

js/jquery.terminal-2.20.2.js

@@ -41,7 +41,7 @@
  *
  * broken image by Sophia Bai from the Noun Project (CC-BY)
  *
- * Date: Wed, 03 Feb 2021 15:24:28 +0000
+ * Date: Wed, 03 Feb 2021 21:07:55 +0000
  */
 /* global define, Map */
 /* eslint-disable */
@@ -4486,7 +4486,7 @@
     // -------------------------------------------------------------------------
     $.terminal = {
         version: 'DEV',
-        date: 'Wed, 03 Feb 2021 15:24:28 +0000',
+        date: 'Wed, 03 Feb 2021 21:07:55 +0000',
         // colors from https://www.w3.org/wiki/CSS/Properties/color/keywords
         color_names: [
             'transparent', 'currentcolor', 'black', 'silver', 'gray', 'white',
@@ -5747,8 +5747,8 @@
                             return '';
                         }
                         var quote = string[0];
-                        var re = new RegExp("(^|(?:\\\\\\\\(?:\\\\\\\\)*))" + quote, "g");
-                        string = string.replace(re, '$1').replace(new RegExp(quote + '$'), '');
+                        var re = new RegExp("(\\\\\\\\(?:\\\\\\\\)*)" + quote, "g");
+                        string = string.replace(re, '$1').replace(/^['"]|['"]$/g, '');
                         if (quote === "'") {
                             string = string.replace(/"/g, '\\"');
                         }

js/jquery.terminal-src.js

@@ -5747,8 +5747,8 @@
                             return '';
                         }
                         var quote = string[0];
-                        var re = new RegExp("(^|(?:\\\\\\\\(?:\\\\\\\\)*))" + quote, "g");
-                        string = string.replace(re, '$1').replace(new RegExp(quote + '$'), '');
+                        var re = new RegExp("(\\\\\\\\(?:\\\\\\\\)*)" + quote, "g");
+                        string = string.replace(re, '$1').replace(/^['"]|['"]$/g, '');
                         if (quote === "'") {
                             string = string.replace(/"/g, '\\"');
                         }

js/jquery.terminal.js

@@ -41,7 +41,7 @@
  *
  * broken image by Sophia Bai from the Noun Project (CC-BY)
  *
- * Date: Wed, 03 Feb 2021 15:24:28 +0000
+ * Date: Wed, 03 Feb 2021 21:07:55 +0000
  */
 /* global define, Map */
 /* eslint-disable */
@@ -4486,7 +4486,7 @@
     // -------------------------------------------------------------------------
     $.terminal = {
         version: 'DEV',
-        date: 'Wed, 03 Feb 2021 15:24:28 +0000',
+        date: 'Wed, 03 Feb 2021 21:07:55 +0000',
         // colors from https://www.w3.org/wiki/CSS/Properties/color/keywords
         color_names: [
             'transparent', 'currentcolor', 'black', 'silver', 'gray', 'white',
@@ -5747,8 +5747,8 @@
                             return '';
                         }
                         var quote = string[0];
-                        var re = new RegExp("(^|(?:\\\\\\\\(?:\\\\\\\\)*))" + quote, "g");
-                        string = string.replace(re, '$1').replace(new RegExp(quote + '$'), '');
+                        var re = new RegExp("(\\\\\\\\(?:\\\\\\\\)*)" + quote, "g");
+                        string = string.replace(re, '$1').replace(/^['"]|['"]$/g, '');
                         if (quote === "'") {
                             string = string.replace(/"/g, '\\"');
                         }