Add more tests

Author: SpaceK33z

.gitignore

@@ -1,3 +1,4 @@
 node_modules
 dist
 *.log
+coverage

packages/graphql-authentication/src/__tests__/mutations.ts

@@ -0,0 +1,103 @@
+import { client, startServer } from './setup';
+
+test('signup - a new user', async () => {
+  const req = client(await startServer());
+
+  const result = await req.request(`mutation {
+    signup(data: {name: "Roger", email: "[email protected]", password: "testtest2"}) {
+      token
+      user {
+        id
+        name
+      }
+    }
+  }`);
+
+  expect((result as any).signup).toEqual({
+    // Poorly check for a JWT token
+    token: expect.stringContaining('.'),
+    user: {
+      id: '3',
+      name: 'Roger'
+    }
+  });
+});
+
+test('signup - with existent user', async () => {
+  const req = client(await startServer());
+
+  try {
+    await req.request(`mutation {
+    signup(data: {name: "Kees", email: "[email protected]", password: "testtest2"}) {
+      token
+    }
+  }`);
+  } catch (e) {
+    expect(String(e)).toMatch(/User already exists with this email/);
+  }
+});
+
+test('signup - with weak password', async () => {
+  const req = client(await startServer());
+
+  try {
+    await req.request(`mutation {
+    signup(data: {name: "Roger", email: "[email protected]", password: "test"}) {
+      token
+    }
+  }`);
+  } catch (e) {
+    expect(String(e)).toMatch(/Password is too short/);
+  }
+});
+
+test('login - correct', async () => {
+  const req = client(await startServer());
+
+  const result = await req.request(`mutation {
+    login(email: "[email protected]", password: "testtest2") {
+      token
+      user {
+        id
+        name
+      }
+    }
+  }`);
+
+  expect((result as any).login).toEqual({
+    // Poorly check for a JWT token
+    token: expect.stringContaining('.'),
+    user: {
+      id: '2',
+      name: 'Kees'
+    }
+  });
+});
+
+test('login - non-existent user', async () => {
+  const req = client(await startServer());
+
+  try {
+    await req.request(`mutation {
+      login(email: "[email protected]", password: "testtest2") {
+        token
+      }
+  }`);
+  } catch (e) {
+    expect(String(e)).toMatch(/No user found/);
+  }
+});
+
+test('login - wrong password', async () => {
+  const req = client(await startServer());
+
+  try {
+    await req.request(`mutation {
+      login(email: "[email protected]", password: "testtest1") {
+        token
+      }
+  }`);
+  } catch (e) {
+    expect(String(e)).toMatch(/No user found/);
+  }
+});

packages/graphql-authentication/src/__tests__/setup.ts

@@ -1,18 +1,49 @@
 import { GraphQLServer } from 'graphql-yoga';
 import { GraphQLClient } from 'graphql-request';
 import { graphqlAuthenticationConfig, authQueries, authMutations } from '..';
+import { GraphqlAuthenticationAdapter, User, ID } from '..';
 
-const KEES_USER = {
-  id: '2',
-  name: 'Kees',
-  email: '[email protected]'
-};
-
-export class FakeAdapter {
-  findUserById(ctx: any, id: any, info?: any) {
-    if (id === '2') {
-      return KEES_USER;
+export class FakeAdapter implements GraphqlAuthenticationAdapter {
+  users: User[] = [
+    {
+      id: '2',
+      name: 'Kees',
+      password: '$2a$10$3dcRen7qMwJmzUzgj7cjUukHYlPTTCAjFhfF00.5WAFhhClTp6H4y', // testtest2
+      email: '[email protected]',
+      inviteAccepted: true,
+      emailConfirmed: true,
+      joinedAt: '2018-06-29T14:26:57+00:00',
+      isSuper: false,
+      lastLogin: ''
     }
+  ];
+
+  // If you'd use a database you wouldn't need this
+  _generateId() {
+    const lastUser = this.users[this.users.length - 1];
+    return String(parseInt(lastUser.id) + 1);
+  }
+  findUserById(ctx: object, id: ID, info?: any) {
+    return Promise.resolve(this.users.find(user => user.id === id) || null);
+  }
+  findUserByEmail(ctx: any, email: string) {
+    return Promise.resolve(
+      this.users.find(user => user.email === email) || null
+    );
+  }
+  userExistsByEmail(ctx: any, email: string) {
+    return Promise.resolve(this.users.some(user => user.email === email));
+  }
+  createUserBySignup(ctx: any, data: any) {
+    const lastUser = this.users[this.users.length - 1];
+    const user = { id: this._generateId(), ...data };
+    this.users.push(user);
+    return user;
+  }
+  async updateUserLastLogin(ctx: any, userId: string, data: any) {
+    const user = await this.findUserById(ctx, userId);
+    Object.assign(user, data); // iel
+    return Promise.resolve(user);
   }
 }
 

packages/graphql-authentication/src/mutations.ts

@@ -1,7 +1,7 @@
 import * as bcrypt from 'bcryptjs';
 import * as jwt from 'jsonwebtoken';
 import * as validator from 'validator';
-import * as uuidv4 from 'uuid/v4';
+import uuidv4 from 'uuid/v4';
 import { getUser, Context } from './utils';
 import { User } from './Adapter';
 import {