chore: update sub-deps and fix lint

This commit is contained in:
Matteo Pagliazzi
2020-07-01 18:20:18 +02:00
parent b9d42bf0bc
commit 3d5630d5a2
72 changed files with 2519 additions and 3931 deletions
-1
View File
@@ -56,7 +56,6 @@ describe('Base model plugin', () => {
expect(sanitized).not.to.have.property('usuallySettable');
});
it('can make fields private', () => {
schema.plugin(baseModel, {
private: ['amPrivate'],
@@ -7,7 +7,6 @@ import amzLib from '../../../../../../website/server/libs/payments/amazon';
import payments from '../../../../../../website/server/libs/payments/payments';
import common from '../../../../../../website/common';
describe('#upgradeGroupPlan', () => {
let spy; let data; let user; let group; let
uuidString;
-1
View File
@@ -1,4 +1,3 @@
import path from 'path';
import nconf from 'nconf';
import setupNconf from '../../../../website/server/libs/setupNconf';
-1
View File
@@ -16,7 +16,6 @@ describe('response middleware', () => {
next = generateNext();
});
it('attaches respond method to res', () => {
responseMiddleware(req, res, next);
-1
View File
@@ -769,7 +769,6 @@ describe('Group Model', () => {
expect(res.t).to.not.be.called;
});
it('does not throw an error if only user ids are passed in', async () => {
await Group.validateInvitations({ uuids: ['user-id', 'user-id2'] }, res);
expect(res.t).to.not.be.called;
-2
View File
@@ -432,7 +432,6 @@ describe('User Model', () => {
user = new User();
});
it('returns false if user does not have customer id', () => {
expect(user.isSubscribed()).to.be.undefined;
});
@@ -558,7 +557,6 @@ describe('User Model', () => {
});
});
context('hasCancelled', () => {
let user;
beforeEach(() => {
-1
View File
@@ -299,7 +299,6 @@ describe('Webhook Model', () => {
});
});
context('type is globalActivity', () => {
let config;
@@ -179,7 +179,6 @@ describe('GET /tasks/user', () => {
expect(dailys2[0].isDue).to.be.false;
});
xit('returns dailies with isDue for the date specified and will add CDS offset if time is not supplied and assumes timezones', async () => {
const timezone = 240;
await user.update({
@@ -687,7 +687,6 @@ describe('POST /tasks/user', () => {
});
});
it('can create checklists', async () => {
const task = await user.post('/tasks/user', {
text: 'test daily',
@@ -147,7 +147,6 @@ describe('PUT /tasks/:id', () => {
notes: 'some new notes',
});
const memberTasks = await member.get('/tasks/user');
const syncedTask = find(memberTasks, findAssignedTask);
@@ -191,7 +190,6 @@ describe('PUT /tasks/:id', () => {
notes: 'some new notes',
});
const memberTasks = await member.get('/tasks/user');
const syncedTask = find(memberTasks, findAssignedTask);
@@ -1,4 +1,3 @@
import { v4 as generateUUID } from 'uuid';
import { find } from 'lodash';
import {
@@ -325,7 +324,6 @@ describe('POST /user/class/cast/:spellId', () => {
expect(result.user.stats.mp).to.equal(10);
});
// TODO find a way to have sinon working in integration tests
// it doesn't work when tests are running separately from server
it('passes correct target to spell when targetType === \'tasks\'');
@@ -4,7 +4,6 @@ import {
translate as t,
} from '../../../../helpers/api-integration/v3';
describe('PUT /user', () => {
let user;
@@ -53,7 +52,6 @@ describe('PUT /user', () => {
expect(user.tags.length).to.be.eql(userTags.length + 1);
});
it('validates profile.name', async () => {
await expect(user.put('/user', {
'profile.name': ' ', // string should be trimmed
@@ -10,7 +10,6 @@ import {
sha1Encrypt as sha1EncryptPassword,
} from '../../../../../../website/server/libs/password';
describe('POST /user/auth/local/login', () => {
let api;
let user;
@@ -9,7 +9,6 @@ import {
sha1Encrypt as sha1EncryptPassword,
} from '../../../../../../website/server/libs/password';
const ENDPOINT = '/user/auth/update-email';
describe('PUT /user/auth/update-email', () => {
@@ -1,4 +1,3 @@
import { v4 as generateUUID } from 'uuid';
import { find } from 'lodash';
import {
@@ -310,7 +309,6 @@ describe('POST /user/class/cast/:spellId', () => {
expect(result.user.stats.mp).to.equal(10);
});
// TODO find a way to have sinon working in integration tests
// it doesn't work when tests are running separately from server
it('passes correct target to spell when targetType === \'tasks\'');
-1
View File
@@ -130,7 +130,6 @@ describe('POST /user/reset', () => {
},
});
await hero.post('/user/reset');
const heroRes = await admin.get(`/hall/heroes/${hero.auth.local.username}`);
-2
View File
@@ -4,7 +4,6 @@ import {
translate as t,
} from '../../../helpers/api-integration/v4';
describe('PUT /user', () => {
let user;
@@ -53,7 +52,6 @@ describe('PUT /user', () => {
expect(user.tags.length).to.be.eql(userTags.length + 1);
});
it('profile.name cannot be an empty string or null', async () => {
await expect(user.put('/user', {
'profile.name': ' ', // string should be trimmed