Fix AVA tests and made them mandatory. #2705

Removed obsolete test cases and included unit tests a mandatory step while build and push.

Fixed #2703
This commit is contained in:
Ameya Advankar 2018-02-25 03:29:51 +05:30 committed by CHaBou
parent 82c7afbeec
commit 76beaa32f5
3 changed files with 1 additions and 51 deletions

View file

@ -7,7 +7,7 @@
"dev": "webpack -w", "dev": "webpack -w",
"build": "cross-env NODE_ENV=production webpack", "build": "cross-env NODE_ENV=production webpack",
"lint": "eslint .", "lint": "eslint .",
"test": "yarn run lint", "test": "yarn run lint && yarn run test:unit",
"test:unit": "ava test/unit", "test:unit": "ava test/unit",
"test:unit:watch": "yarn run test:unit -- --watch", "test:unit:watch": "yarn run test:unit -- --watch",
"prepush": "yarn test", "prepush": "yarn test",

View file

@ -1,34 +0,0 @@
import test from 'ava';
import findCommandByKeys from '../../app/utils/keymaps/find-command-by-keys';
const expectedCommand = 'test-command';
const expectedLocalizedCommand = 'test-localized-command';
const commands = {
'alt+p+shift': expectedCommand,
'ç+cmd+ctrl': expectedLocalizedCommand
};
test(`returns a command`, t => {
t.is(findCommandByKeys('alt+shift+p', commands), expectedCommand);
t.is(findCommandByKeys('shift+p+alt', commands), expectedCommand);
t.is(findCommandByKeys('p+alt+shift', commands), expectedCommand);
t.is(findCommandByKeys('alt+shift+P', commands), expectedCommand);
t.is(findCommandByKeys('Shift+P+Alt', commands), expectedCommand);
});
test(`returns a localized command`, t => {
t.is(findCommandByKeys('cmd+ctrl+ç', commands), expectedLocalizedCommand);
t.is(findCommandByKeys('ç+cmd+ctrl', commands), expectedLocalizedCommand);
t.is(findCommandByKeys('ctrl+ç+cmd', commands), expectedLocalizedCommand);
t.is(findCommandByKeys('ctrl+Ç+cmd', commands), expectedLocalizedCommand);
t.is(findCommandByKeys('Cmd+Ctrl+Ç', commands), expectedLocalizedCommand);
});

View file

@ -1,16 +0,0 @@
import test from 'ava';
import normalize from '../../app/utils/keymaps/normalize';
test(`is normalizing keymaps correctly`, t => {
const notNormalized = 'cmd+alt+o';
const normalized = 'alt+cmd+o';
t.is(normalize(notNormalized), normalized);
});
test(`is normalizing localized keymaps correctly`, t => {
const notNormalized = 'cmd+alt+ç';
const normalized = 'alt+ç+cmd';
t.is(normalize(notNormalized), normalized);
});