feat: add tests

VSCodeVim · Feb 6, 2019 · c863403 · c863403
1 parent adff634
commit c863403
Show file tree

Hide file tree

Showing 8 changed files with 150 additions and 10 deletions.
diff --git a/extension.ts b/extension.ts
@@ -71,9 +71,9 @@ async function loadConfiguration() {
   const logger = Logger.get('Configuration');
 
   const validatorResults = await configuration.load();
-  logger.debug(`${validatorResults.numErrors()} errors found with vim configuration`);
+  logger.debug(`${validatorResults.numErrors} errors found with vim configuration`);
 
-  if (validatorResults.numErrors() > 0) {
+  if (validatorResults.numErrors > 0) {
     for (let validatorResult of validatorResults.get()) {
       switch (validatorResult.level) {
         case 'error':

diff --git a/src/configuration/configurationValidator.ts b/src/configuration/configurationValidator.ts
@@ -20,7 +20,7 @@ class ConfigurationValidator {
 
     for (const validator of this._validators) {
       let validatorResults = await validator.validate(config);
-      if (validatorResults.hasError()) {
+      if (validatorResults.hasError) {
         // errors found in configuration, disable feature
         validator.disable(config);
       }

diff --git a/src/configuration/iconfiguration.ts b/src/configuration/iconfiguration.ts
@@ -255,6 +255,13 @@ export interface IConfiguration {
   visualModeKeyBindings: IKeyRemapping[];
   visualModeKeyBindingsNonRecursive: IKeyRemapping[];
 
+  insertModeKeyBindingsMap: Map<string, IKeyRemapping>;
+  insertModeKeyBindingsNonRecursiveMap: Map<string, IKeyRemapping>;
+  normalModeKeyBindingsMap: Map<string, IKeyRemapping>;
+  normalModeKeyBindingsNonRecursiveMap: Map<string, IKeyRemapping>;
+  visualModeKeyBindingsMap: Map<string, IKeyRemapping>;
+  visualModeKeyBindingsNonRecursiveMap: Map<string, IKeyRemapping>;
+
   /**
    * Comma-separated list of motion keys that should wrap to next/previous line.
    */

diff --git a/src/configuration/iconfigurationValidator.ts b/src/configuration/iconfigurationValidator.ts
@@ -20,12 +20,12 @@ export class ValidatorResults {
     return this.errors;
   }
 
-  public numErrors(): number {
+  public get numErrors(): number {
     return this.errors.filter(e => e.level === 'error').length;
   }
 
-  public hasError(): boolean {
-    return this.numErrors() > 0;
+  public get hasError(): boolean {
+    return this.numErrors > 0;
   }
 }
 

diff --git a/src/configuration/validators/remappingValidator.ts b/src/configuration/validators/remappingValidator.ts
@@ -1,7 +1,6 @@
 import * as vscode from 'vscode';
 import { IConfiguration, IKeyRemapping } from '../iconfiguration';
 import { Notation } from '../notation';
-import { configuration } from '../configuration';
 import { IConfigurationValidator, ValidatorResults } from '../iconfigurationValidator';
 
 export class RemappingValidator implements IConfigurationValidator {
@@ -27,7 +26,7 @@ export class RemappingValidator implements IConfigurationValidator {
         // validate
         let remappingError = await this.isRemappingValid(remapping);
         result.concat(remappingError);
-        if (remappingError.hasError()) {
+        if (remappingError.hasError) {
           // errors with remapping, skip
           keybindings.splice(i, 1);
           continue;
@@ -50,7 +49,7 @@ export class RemappingValidator implements IConfigurationValidator {
         const beforeKeys = remapping.before.join('');
         if (modeKeyBindingsMap.has(beforeKeys)) {
           result.append({
-            level: 'error',
+            level: 'warning',
             message: `${remapping.before}. Duplicate remapped key for ${beforeKeys}.`,
           });
           continue;
@@ -60,7 +59,7 @@ export class RemappingValidator implements IConfigurationValidator {
         modeKeyBindingsMap.set(beforeKeys, remapping);
       }
 
-      configuration[modeKeyBindingsKey + 'Map'] = modeKeyBindingsMap;
+      config[modeKeyBindingsKey + 'Map'] = modeKeyBindingsMap;
     }
 
     return result;

diff --git a/test/configuration/validators/neovimValidator.test.ts b/test/configuration/validators/neovimValidator.test.ts
@@ -0,0 +1,36 @@
+import * as assert from 'assert';
+import { Configuration } from '../../testConfiguration';
+import { NeovimValidator } from '../../../src/configuration/validators/neovimValidator';
+
+suite('Neovim Validator', () => {
+  test('neovim enabled without path', async () => {
+    // setup
+    let configuration = new Configuration();
+    configuration.enableNeovim = true;
+    configuration.neovimPath = '';
+
+    // test
+    const validator = new NeovimValidator();
+    let actual = await validator.validate(configuration);
+    validator.disable(configuration);
+
+    // assert
+    assert.equal(actual.numErrors, 1);
+    assert.equal(actual.hasError, true);
+    assert.equal(configuration.enableNeovim, false);
+  });
+
+  test('neovim disabled', async () => {
+    // setup
+    let configuration = new Configuration();
+    configuration.enableNeovim = false;
+    configuration.neovimPath = '';
+
+    // test
+    const validator = new NeovimValidator();
+    let actual = await validator.validate(configuration);
+
+    // assert
+    assert.equal(actual.numErrors, 0);
+  });
+});
diff --git a/test/configuration/validators/remappingValidator.test.ts b/test/configuration/validators/remappingValidator.test.ts
@@ -0,0 +1,92 @@
+import * as assert from 'assert';
+import { Configuration } from '../../testConfiguration';
+import { RemappingValidator } from '../../../src/configuration/validators/remappingValidator';
+
+suite('Remapping Validator', () => {
+  test('no remappings', async () => {
+    // setup
+    const configuration = new Configuration();
+    configuration.insertModeKeyBindings = [];
+    configuration.insertModeKeyBindingsNonRecursive = [];
+    configuration.normalModeKeyBindings = [];
+    configuration.normalModeKeyBindingsNonRecursive = [];
+    configuration.visualModeKeyBindings = [];
+    configuration.visualModeKeyBindingsNonRecursive = [];
+
+    // test
+    const validator = new RemappingValidator();
+    let actual = await validator.validate(configuration);
+
+    // assert
+    assert.equal(actual.numErrors, 0);
+    assert.equal(actual.hasError, false);
+
+    assert.equal(configuration.insertModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.insertModeKeyBindingsNonRecursiveMap.size, 0);
+    assert.equal(configuration.normalModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.normalModeKeyBindingsNonRecursiveMap.size, 0);
+    assert.equal(configuration.visualModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.visualModeKeyBindingsNonRecursiveMap.size, 0);
+  });
+
+  test('jj->esc', async () => {
+    // setup
+    let configuration = new Configuration();
+    configuration.insertModeKeyBindings = [
+      {
+        before: ['j', 'j'],
+        after: ['<Esc>'],
+      }];
+    configuration.insertModeKeyBindingsNonRecursive = [];
+    configuration.normalModeKeyBindings = [];
+    configuration.normalModeKeyBindingsNonRecursive = [];
+    configuration.visualModeKeyBindings = [];
+    configuration.visualModeKeyBindingsNonRecursive = [];
+
+    // test
+    const validator = new RemappingValidator();
+    let actual = await validator.validate(configuration);
+
+    // assert
+    assert.equal(actual.numErrors, 0);
+    assert.equal(actual.hasError, false);
+
+    assert.equal(configuration.insertModeKeyBindingsMap.size, 1);
+    assert.equal(configuration.insertModeKeyBindingsNonRecursiveMap.size, 0);
+    assert.equal(configuration.normalModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.normalModeKeyBindingsNonRecursiveMap.size, 0);
+    assert.equal(configuration.visualModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.visualModeKeyBindingsNonRecursiveMap.size, 0);
+
+    assert.equal(configuration.insertModeKeyBindingsMap.get("jj"), configuration.insertModeKeyBindings[0]);
+  });
+
+  test('remapping missing after and command', async () => {
+    // setup
+    let configuration = new Configuration();
+    configuration.insertModeKeyBindings = [
+      {
+        before: ['j', 'j']
+      }];
+    configuration.insertModeKeyBindingsNonRecursive = [];
+    configuration.normalModeKeyBindings = [];
+    configuration.normalModeKeyBindingsNonRecursive = [];
+    configuration.visualModeKeyBindings = [];
+    configuration.visualModeKeyBindingsNonRecursive = [];
+
+    // test
+    const validator = new RemappingValidator();
+    let actual = await validator.validate(configuration);
+
+    // assert
+    assert.equal(actual.numErrors, 1);
+    assert.equal(actual.hasError, true);
+
+    assert.equal(configuration.insertModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.insertModeKeyBindingsNonRecursiveMap.size, 0);
+    assert.equal(configuration.normalModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.normalModeKeyBindingsNonRecursiveMap.size, 0);
+    assert.equal(configuration.visualModeKeyBindingsMap.size, 0);
+    assert.equal(configuration.visualModeKeyBindingsNonRecursiveMap.size, 0);
+  });
+});
diff --git a/test/testConfiguration.ts b/test/testConfiguration.ts
@@ -85,6 +85,12 @@ export class Configuration implements IConfiguration {
   normalModeKeyBindingsNonRecursive: IKeyRemapping[] = [];
   visualModeKeyBindings: IKeyRemapping[] = [];
   visualModeKeyBindingsNonRecursive: IKeyRemapping[] = [];
+  insertModeKeyBindingsMap: Map<string, IKeyRemapping>;
+  insertModeKeyBindingsNonRecursiveMap: Map<string, IKeyRemapping>;
+  normalModeKeyBindingsMap: Map<string, IKeyRemapping>;
+  normalModeKeyBindingsNonRecursiveMap: Map<string, IKeyRemapping>;
+  visualModeKeyBindingsMap: Map<string, IKeyRemapping>;
+  visualModeKeyBindingsNonRecursiveMap: Map<string, IKeyRemapping>;
   whichwrap = '';
   wrapKeys = {};
   report = 2;