js: code cleanup (eslint)

--
* trying to stick to eslint from now on for readability

* As some attribution was lost in previous git confusions, I added my
  name into some of the licence headers

11 files changed, 245 insertions, 168 deletions

diff --git a/lang/js/BrowserTestExtension/popup.js b/lang/js/BrowserTestExtension/popup.js
index 12beb1eb..794620b6 100644
--- a/lang/js/BrowserTestExtension/popup.js
+++ b/lang/js/BrowserTestExtension/popup.js
@@ -16,27 +16,13 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
- */
-/* gpgme.js - Javascript integration for gpgme
- * Copyright (C) 2018 Bundesamt für Sicherheit in der Informationstechnik
- *
- * This file is part of GPGME.
- *
- * GPGME is free software; you can redistribute it and/or modify it
- * under the terms of the GNU Lesser General Public License as
- * published by the Free Software Foundation; either version 2.1 of
- * the License, or (at your option) any later version.
  *
- * GPGME is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this program; if not, see <http://www.gnu.org/licenses/>.
- * SPDX-License-Identifier: LGPL-2.1+
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
 
+/* global chrome */
+
 document.addEventListener('DOMContentLoaded', function() {
     chrome.tabs.create({
         url: './index.html'
diff --git a/lang/js/BrowserTestExtension/runbrowsertest.js b/lang/js/BrowserTestExtension/runbrowsertest.js
index 39bc3fb9..c46eb120 100644
--- a/lang/js/BrowserTestExtension/runbrowsertest.js
+++ b/lang/js/BrowserTestExtension/runbrowsertest.js
@@ -16,6 +16,11 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
 
+/* global mocha */
+
 mocha.run();
diff --git a/lang/js/BrowserTestExtension/rununittests.js b/lang/js/BrowserTestExtension/rununittests.js
index f85ed8b5..df31589e 100644
--- a/lang/js/BrowserTestExtension/rununittests.js
+++ b/lang/js/BrowserTestExtension/rununittests.js
@@ -16,6 +16,12 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
+
+/* global Gpgmejs_test, mocha*/
+
 Gpgmejs_test.unittests();
 mocha.run();
diff --git a/lang/js/BrowserTestExtension/setup_testing.js b/lang/js/BrowserTestExtension/setup_testing.js
index 7f70d347..52aeac58 100644
--- a/lang/js/BrowserTestExtension/setup_testing.js
+++ b/lang/js/BrowserTestExtension/setup_testing.js
@@ -16,7 +16,13 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
+
+/* global mocha, chai */
+
 mocha.setup('bdd');
-var expect = chai.expect;
+const expect = chai.expect; //eslint-disable-line no-unused-vars
 chai.config.includeStack = true;
\ No newline at end of file
diff --git a/lang/js/BrowserTestExtension/tests/KeyImportExport.js b/lang/js/BrowserTestExtension/tests/KeyImportExport.js
index e6eb5a30..6b298049 100644
--- a/lang/js/BrowserTestExtension/tests/KeyImportExport.js
+++ b/lang/js/BrowserTestExtension/tests/KeyImportExport.js
@@ -16,6 +16,9 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
 
 /* global describe, it, expect, Gpgmejs, ImportablePublicKey */
diff --git a/lang/js/BrowserTestExtension/tests/encryptDecryptTest.js b/lang/js/BrowserTestExtension/tests/encryptDecryptTest.js
index e5c2f749..a84be27c 100644
--- a/lang/js/BrowserTestExtension/tests/encryptDecryptTest.js
+++ b/lang/js/BrowserTestExtension/tests/encryptDecryptTest.js
@@ -1,4 +1,3 @@
-
 /* gpgme.js - Javascript integration for gpgme
  * Copyright (C) 2018 Bundesamt für Sicherheit in der Informationstechnik
  *
@@ -17,23 +16,31 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
 
+/* global describe, it, expect, Gpgmejs */
+/* global inputvalues, encryptedData, bigString, bigBoringString */
+
 describe('Encryption and Decryption', function () {
     it('Successful encrypt and decrypt simple string', function (done) {
         let prm = Gpgmejs.init();
         prm.then(function (context) {
             context.encrypt(
                 inputvalues.encrypt.good.data,
-                inputvalues.encrypt.good.fingerprint).then(function (answer) {
+                inputvalues.encrypt.good.fingerprint).then(
+                function (answer) {
                     expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
+                    expect(answer.data).to.be.a('string');
                     expect(answer.data).to.include('BEGIN PGP MESSAGE');
                     expect(answer.data).to.include('END PGP MESSAGE');
                     context.decrypt(answer.data).then(function (result) {
                         expect(result).to.not.be.empty;
                         expect(result.data).to.be.a('string');
-                        expect(result.data).to.equal(inputvalues.encrypt.good.data);
+                        expect(result.data).to.equal(
+                            inputvalues.encrypt.good.data);
                         done();
                     });
                 });
@@ -51,7 +58,7 @@ describe('Encryption and Decryption', function () {
                     expect(result.data).to.equal(
                         '¡Äußerste µ€ før ñoquis@hóme! Добрый день\n');
                     done();
-            });
+                });
         });
     }).timeout(3000);
 
@@ -64,7 +71,7 @@ describe('Encryption and Decryption', function () {
                     inputvalues.encrypt.good.fingerprint).then(
                     function (answer) {
                         expect(answer).to.not.be.empty;
-                        expect(answer.data).to.be.a("string");
+                        expect(answer.data).to.be.a('string');
                         expect(answer.data).to.include(
                             'BEGIN PGP MESSAGE');
                         expect(answer.data).to.include(
@@ -79,39 +86,41 @@ describe('Encryption and Decryption', function () {
                             });
                     });
             });
-    }).timeout(5000);
+        }).timeout(5000);
 
     for (let j = 0; j < inputvalues.encrypt.good.data_nonascii_32.length; j++){
-        it('Roundtrip with >1MB non-ascii input meeting default chunksize (' + (j + 1) + '/' + inputvalues.encrypt.good.data_nonascii_32.length + ')',
-            function (done) {
-                let input = inputvalues.encrypt.good.data_nonascii_32[j];
-                expect(input).to.have.length(32);
-                let prm = Gpgmejs.init();
-                prm.then(function (context) {
-                    let data = '';
-                    for (let i=0; i < 34 * 1024; i++){
-                        data += input;
-                    }
-                    context.encrypt(data,
-                        inputvalues.encrypt.good.fingerprint).then(
-                        function (answer) {
-                            expect(answer).to.not.be.empty;
-                            expect(answer.data).to.be.a("string");
-                            expect(answer.data).to.include(
-                                'BEGIN PGP MESSAGE');
-                            expect(answer.data).to.include(
-                                'END PGP MESSAGE');
-                            context.decrypt(answer.data).then(
-                                function (result) {
-                                    expect(result).to.not.be.empty;
-                                    expect(result.data).to.be.a('string');
-                                    expect(result.data).to.equal(data);
-                                    done();
-                                });
-                        });
-                });
+        it('Roundtrip with >1MB non-ascii input meeting default chunksize (' +
+            (j + 1) + '/'
+            + inputvalues.encrypt.good.data_nonascii_32.length + ')',
+        function (done) {
+            let input = inputvalues.encrypt.good.data_nonascii_32[j];
+            expect(input).to.have.length(32);
+            let prm = Gpgmejs.init();
+            prm.then(function (context) {
+                let data = '';
+                for (let i=0; i < 34 * 1024; i++){
+                    data += input;
+                }
+                context.encrypt(data,
+                    inputvalues.encrypt.good.fingerprint).then(
+                    function (answer) {
+                        expect(answer).to.not.be.empty;
+                        expect(answer.data).to.be.a('string');
+                        expect(answer.data).to.include(
+                            'BEGIN PGP MESSAGE');
+                        expect(answer.data).to.include(
+                            'END PGP MESSAGE');
+                        context.decrypt(answer.data).then(
+                            function (result) {
+                                expect(result).to.not.be.empty;
+                                expect(result.data).to.be.a('string');
+                                expect(result.data).to.equal(data);
+                                done();
+                            });
+                    });
+            });
         }).timeout(3000);
-    };
+    }
 
     it('Random data, as string', function (done) {
         let data = bigString(1000);
@@ -121,7 +130,7 @@ describe('Encryption and Decryption', function () {
                 inputvalues.encrypt.good.fingerprint).then(
                 function (answer) {
                     expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
+                    expect(answer.data).to.be.a('string');
                     expect(answer.data).to.include(
                         'BEGIN PGP MESSAGE');
                     expect(answer.data).to.include(
@@ -143,10 +152,10 @@ describe('Encryption and Decryption', function () {
         let prm = Gpgmejs.init();
         prm.then(function (context) {
             context.encrypt(b64data,
-                inputvalues.encrypt.good.fingerprint,).then(
+                inputvalues.encrypt.good.fingerprint).then(
                 function (answer) {
                     expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
+                    expect(answer.data).to.be.a('string');
                     expect(answer.data).to.include(
                         'BEGIN PGP MESSAGE');
                     expect(answer.data).to.include(
@@ -171,7 +180,7 @@ describe('Encryption and Decryption', function () {
                 inputvalues.encrypt.good.fingerprint, true).then(
                 function (answer) {
                     expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
+                    expect(answer.data).to.be.a('string');
                     expect(answer.data).to.include(
                         'BEGIN PGP MESSAGE');
                     expect(answer.data).to.include(
@@ -196,7 +205,7 @@ describe('Encryption and Decryption', function () {
                 inputvalues.encrypt.good.fingerprint).then(
                 function (answer) {
                     expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
+                    expect(answer.data).to.be.a('string');
 
                     expect(answer.data).to.include(
                         'BEGIN PGP MESSAGE');
diff --git a/lang/js/BrowserTestExtension/tests/encryptTest.js b/lang/js/BrowserTestExtension/tests/encryptTest.js
index a16f993c..2cb4e58b 100644
--- a/lang/js/BrowserTestExtension/tests/encryptTest.js
+++ b/lang/js/BrowserTestExtension/tests/encryptTest.js
@@ -16,7 +16,14 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
+
+/* global describe, it, expect, Gpgmejs */
+/* global inputvalues, fixedLengthString */
+
 describe('Encryption', function () {
     it('Successful encrypt', function (done) {
         let prm = Gpgmejs.init();
@@ -24,12 +31,12 @@ describe('Encryption', function () {
             context.encrypt(
                 inputvalues.encrypt.good.data,
                 inputvalues.encrypt.good.fingerprint).then(function (answer) {
-                    expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
-                    expect(answer.data).to.include('BEGIN PGP MESSAGE');
-                    expect(answer.data).to.include('END PGP MESSAGE');
-                    done();
-                });
+                expect(answer).to.not.be.empty;
+                expect(answer.data).to.be.a('string');
+                expect(answer.data).to.include('BEGIN PGP MESSAGE');
+                expect(answer.data).to.include('END PGP MESSAGE');
+                done();
+            });
         });
     });
 
@@ -40,12 +47,12 @@ describe('Encryption', function () {
             context.encrypt(
                 data,
                 inputvalues.encrypt.good.fingerprint).then(function (answer) {
-                    expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
-                    expect(answer.data).to.include('BEGIN PGP MESSAGE');
-                    expect(answer.data).to.include('END PGP MESSAGE');
-                    done();
-                });
+                expect(answer).to.not.be.empty;
+                expect(answer.data).to.be.a('string');
+                expect(answer.data).to.include('BEGIN PGP MESSAGE');
+                expect(answer.data).to.include('END PGP MESSAGE');
+                done();
+            });
         });
     }).timeout(10000);
 
@@ -56,12 +63,12 @@ describe('Encryption', function () {
             context.encrypt(
                 data,
                 inputvalues.encrypt.good.fingerprint).then(function (answer) {
-                    expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
-                    expect(answer.data).to.include('BEGIN PGP MESSAGE');
-                    expect(answer.data).to.include('END PGP MESSAGE');
-                    done();
-                });
+                expect(answer).to.not.be.empty;
+                expect(answer.data).to.be.a('string');
+                expect(answer.data).to.include('BEGIN PGP MESSAGE');
+                expect(answer.data).to.include('END PGP MESSAGE');
+                done();
+            });
         });
     }).timeout(20000);
 
@@ -72,12 +79,12 @@ describe('Encryption', function () {
             context.encrypt(
                 data,
                 inputvalues.encrypt.good.fingerprint).then(function (answer) {
-                    expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a("string");
-                    expect(answer.data).to.include('BEGIN PGP MESSAGE');
-                    expect(answer.data).to.include('END PGP MESSAGE');
-                    done();
-                });
+                expect(answer).to.not.be.empty;
+                expect(answer.data).to.be.a('string');
+                expect(answer.data).to.include('BEGIN PGP MESSAGE');
+                expect(answer.data).to.include('END PGP MESSAGE');
+                done();
+            });
         });
     }).timeout(20000);
 
@@ -87,12 +94,12 @@ describe('Encryption', function () {
             context.encrypt(
                 inputvalues.encrypt.good.data,
                 null).then(function (answer) {
-                    expect(answer).to.be.undefined;
-                }, function(error){
-                    expect(error).to.be.an('Error');
-                    expect(error.code).to.equal('MSG_INCOMPLETE');
-                    done();
-                });
+                expect(answer).to.be.undefined;
+            }, function(error){
+                expect(error).to.be.an('Error');
+                expect(error.code).to.equal('MSG_INCOMPLETE');
+                done();
+            });
         });
     });
 
@@ -101,12 +108,12 @@ describe('Encryption', function () {
         prm.then(function (context) {
             context.encrypt(
                 null, inputvalues.encrypt.good.keyid).then(function (answer) {
-                    expect(answer).to.be.undefined;
-                }, function (error) {
-                    expect(error).to.be.an.instanceof(Error);
-                    expect(error.code).to.equal('MSG_INCOMPLETE');
-                    done();
-                });
+                expect(answer).to.be.undefined;
+            }, function (error) {
+                expect(error).to.be.an.instanceof(Error);
+                expect(error.code).to.equal('MSG_INCOMPLETE');
+                done();
+            });
         });
     });
 
@@ -116,15 +123,15 @@ describe('Encryption', function () {
             context.encrypt(
                 inputvalues.encrypt.good.data,
                 inputvalues.encrypt.bad.fingerprint).then(function (answer) {
-                    expect(answer).to.be.undefined;
-                }, function(error){
-                    expect(error).to.be.an('Error');
-                    expect(error.code).to.not.be.undefined;
-                    expect(error.code).to.equal('GNUPG_ERROR');
-                    done();
-                });
+                expect(answer).to.be.undefined;
+            }, function(error){
+                expect(error).to.be.an('Error');
+                expect(error.code).to.not.be.undefined;
+                expect(error.code).to.equal('GNUPG_ERROR');
+                done();
+            });
         });
-    }).timeout(5000);;
+    }).timeout(5000);
 
     it('Overly large message ( > 65MB) is rejected', function (done) {
         let prm = Gpgmejs.init();
@@ -132,15 +139,15 @@ describe('Encryption', function () {
             context.encrypt(
                 fixedLengthString(65),
                 inputvalues.encrypt.good.fingerprint).then(function (answer) {
-                    expect(answer).to.be.undefined;
-                }, function(error){
-                    expect(error).to.be.an.instanceof(Error);
-                    // expect(error.code).to.equal('GNUPG_ERROR');
-                    // TODO: there is a 64 MB hard limit at least in chrome at:
-                    // chromium//extensions/renderer/messaging_util.cc:
-                    // kMaxMessageLength
-                    done();
-                });
+                expect(answer).to.be.undefined;
+            }, function(error){
+                expect(error).to.be.an.instanceof(Error);
+                // expect(error.code).to.equal('GNUPG_ERROR');
+                // TODO: there is a 64 MB hard limit at least in chrome at:
+                // chromium//extensions/renderer/messaging_util.cc:
+                // kMaxMessageLength
+                done();
+            });
         });
     }).timeout(8000);
 
diff --git a/lang/js/BrowserTestExtension/tests/inputvalues.js b/lang/js/BrowserTestExtension/tests/inputvalues.js
index 9dc13324..024aad25 100644
--- a/lang/js/BrowserTestExtension/tests/inputvalues.js
+++ b/lang/js/BrowserTestExtension/tests/inputvalues.js
@@ -16,9 +16,12 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
 
-var inputvalues = {// eslint-disable-line no-unused-vars
+const inputvalues = {// eslint-disable-line no-unused-vars
     encrypt: {
         good:{
             data : 'Hello World.',
@@ -131,7 +134,7 @@ function slightlyLessBoringString(megabytes, set){
 }
 
 // Data encrypted with testKey
-var encryptedData =// eslint-disable-line no-unused-vars
+const encryptedData =// eslint-disable-line no-unused-vars
     '-----BEGIN PGP MESSAGE-----\n' +
     '\n' +
     'hQEMA6B8jfIUScGEAQgAlANd3uyhmhYLzVcfz4LEqA8tgUC3n719YH0iuKEzG/dv\n' +
@@ -146,7 +149,7 @@ var encryptedData =// eslint-disable-line no-unused-vars
     '=zap6\n' +
     '-----END PGP MESSAGE-----\n';
 
-var ImportablePublicKey = {// eslint-disable-line no-unused-vars
+const ImportablePublicKey = {// eslint-disable-line no-unused-vars
     fingerprint: '78034948BA7F5D0E9BDB67E4F63790C11E60278A',
     key:'-----BEGIN PGP PUBLIC KEY BLOCK-----\n' +
     '\n' +
diff --git a/lang/js/BrowserTestExtension/tests/longRunningTests.js b/lang/js/BrowserTestExtension/tests/longRunningTests.js
index 5c588f27..eefe126d 100644
--- a/lang/js/BrowserTestExtension/tests/longRunningTests.js
+++ b/lang/js/BrowserTestExtension/tests/longRunningTests.js
@@ -1,39 +1,78 @@
+/* gpgme.js - Javascript integration for gpgme
+ * Copyright (C) 2018 Bundesamt für Sicherheit in der Informationstechnik
+ *
+ * This file is part of GPGME.
+ *
+ * GPGME is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU Lesser General Public License as
+ * published by the Free Software Foundation; either version 2.1 of
+ * the License, or (at your option) any later version.
+ *
+ * GPGME is distributed in the hope that it will be useful, but
+ * WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this program; if not, see <http://www.gnu.org/licenses/>.
+ * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
+ */
+/* global describe, it, expect, Gpgmejs */
+/* global bigString, inputvalues */
+
 describe('Long running Encryption/Decryption', function () {
     for (let i=0; i < 100; i++) {
-        it('Successful encrypt/decrypt completely random data ' + (i+1) + '/100', function (done) {
+        it('Successful encrypt/decrypt completely random data ' +
+            (i+1) + '/100', function (done) {
             let prm = Gpgmejs.init();
             let data = bigString(2*1024*1024);
-                prm.then(function (context) {
-                    context.encrypt(data,
-                        inputvalues.encrypt.good.fingerprint).then(
-                            function (answer){
-                                expect(answer).to.not.be.empty;
-                                expect(answer.data).to.be.a("string");
-                                expect(answer.data).to.include(
-                                    'BEGIN PGP MESSAGE');
-                                expect(answer.data).to.include(
-                                    'END PGP MESSAGE');
-                                context.decrypt(answer.data).then(
-                                    function(result){
-                                        expect(result).to.not.be.empty;
-                                        expect(result.data).to.be.a('string');
-                                        if (result.data.length !== data.length) {
-                                            console.log('diff: ' + (result.data.length - data.length));
-                                            for (let i=0; i < result.data.length; i++){
-                                                if (result.data[i] !== data[i]){
-                                                    console.log('position: ' + i);
-                                                    console.log('result : '+ result.data.charCodeAt(i) + result.data[i-2] + result.data[i-1] + result.data[i] + result.data[i+1] + result.data[i+2]);
-                                                    console.log('original: ' + data.charCodeAt(i) + data[i-2] + data[i-1] + data[i] + data[i+1] + data[i+2]);
-                                                    break;
-                                                }
-                                            }
+            prm.then(function (context) {
+                context.encrypt(data,
+                    inputvalues.encrypt.good.fingerprint).then(
+                    function (answer){
+                        expect(answer).to.not.be.empty;
+                        expect(answer.data).to.be.a('string');
+                        expect(answer.data).to.include(
+                            'BEGIN PGP MESSAGE');
+                        expect(answer.data).to.include(
+                            'END PGP MESSAGE');
+                        context.decrypt(answer.data).then(
+                            function(result){
+                                expect(result).to.not.be.empty;
+                                expect(result.data).to.be.a('string');
+                                if (result.data.length !== data.length) {
+                                    // console.log('diff: ' +
+                                    // (result.data.length - data.length));
+                                    for (let i=0; i < result.data.length; i++){
+                                        if (result.data[i] !== data[i]){
+                                            // console.log('position: ' + i);
+                                            // console.log('result : ' +
+                                            // result.data.charCodeAt(i) +
+                                            // result.data[i-2] +
+                                            // result.data[i-1] +
+                                            // result.data[i] +
+                                            // result.data[i+1] +
+                                            // result.data[i+2]);
+                                            // console.log('original: ' +
+                                            // data.charCodeAt(i) +
+                                            // data[i-2] +
+                                            // data[i-1] +
+                                            // data[i] +
+                                            // data[i+1] +
+                                            // data[i+2]);
+                                            break;
                                         }
-                                        expect(result.data).to.equal(data);
-                                        done();
-                                });
-                        });
-                });
+                                    }
+                                }
+                                expect(result.data).to.equal(data);
+                                done();
+                            });
+                    });
+            });
         }).timeout(8000);
-    };
+    }
 
 });
diff --git a/lang/js/BrowserTestExtension/tests/signTest.js b/lang/js/BrowserTestExtension/tests/signTest.js
index 2e5edb30..ffd2d5de 100644
--- a/lang/js/BrowserTestExtension/tests/signTest.js
+++ b/lang/js/BrowserTestExtension/tests/signTest.js
@@ -16,7 +16,14 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
+
+/* global describe, it, expect, Gpgmejs */
+/* global bigString, inputvalues */
+
 describe('Signing', function () {
     it('Sign a message', function (done) {
         let prm = Gpgmejs.init();
@@ -25,12 +32,12 @@ describe('Signing', function () {
             context.sign(
                 data,
                 inputvalues.encrypt.good.fingerprint).then(function (answer) {
-                    expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a('string');
-                    expect(answer.data).to.include('BEGIN PGP SIGNATURE');
-                    expect(answer.data).to.include('END PGP SIGNATURE');
-                    expect(answer.data).to.include(data);
-                    done();
+                expect(answer).to.not.be.empty;
+                expect(answer.data).to.be.a('string');
+                expect(answer.data).to.include('BEGIN PGP SIGNATURE');
+                expect(answer.data).to.include('END PGP SIGNATURE');
+                expect(answer.data).to.include(data);
+                done();
             });
         });
     });
@@ -43,12 +50,12 @@ describe('Signing', function () {
                 inputvalues.encrypt.good.fingerprint,
                 'detached'
             ).then(function (answer) {
-                    expect(answer).to.not.be.empty;
-                    expect(answer.data).to.be.a('string');
-                    expect(answer.data).to.include(data);
-                    expect(answer.signature).to.be.a('string');
-                    expect(answer.signature).to.be.a('string');
-                    done();
+                expect(answer).to.not.be.empty;
+                expect(answer.data).to.be.a('string');
+                expect(answer.data).to.include(data);
+                expect(answer.signature).to.be.a('string');
+                expect(answer.signature).to.be.a('string');
+                done();
             });
         });
     });
diff --git a/lang/js/BrowserTestExtension/tests/startup.js b/lang/js/BrowserTestExtension/tests/startup.js
index 7d13ea47..d434b6d4 100644
--- a/lang/js/BrowserTestExtension/tests/startup.js
+++ b/lang/js/BrowserTestExtension/tests/startup.js
@@ -16,25 +16,31 @@
  * You should have received a copy of the GNU Lesser General Public
  * License along with this program; if not, see <http://www.gnu.org/licenses/>.
  * SPDX-License-Identifier: LGPL-2.1+
+ *
+ * Author(s):
+ *     Maximilian Krambach <[email protected]>
  */
 
- describe('GPGME context', function(){
+/* global describe, it, expect, Gpgmejs */
+/* global inputvalues */
+
+describe('GPGME context', function(){
     it('Starting a GpgME instance', function(done){
         let prm = Gpgmejs.init();
         prm.then(
-         function(context){
-            expect(context).to.be.an('object');
-            expect(context.encrypt).to.be.a('function');
-            expect(context.decrypt).to.be.a('function');
-            done();
-        });
+            function(context){
+                expect(context).to.be.an('object');
+                expect(context.encrypt).to.be.a('function');
+                expect(context.decrypt).to.be.a('function');
+                done();
+            });
     });
 });
 
 describe('GPGME does not start with invalid parameters', function(){
     for (let i=0; i < inputvalues.init.invalid_startups.length; i++){
         it('Parameter '+ i, function(done){
-        let prm = Gpgmejs.init(inputvalues.init.invalid_startups[i]);
+            let prm = Gpgmejs.init(inputvalues.init.invalid_startups[i]);
             prm.then(function(context){
                 expect(context).to.be.undefined;
                 done();
@@ -43,6 +49,6 @@ describe('GPGME does not start with invalid parameters', function(){
                 expect(error.code).to.equal('PARAM_WRONG');
                 done();
             });
-        })
+        });
     }
 });
\ No newline at end of file