test web-frontend authentication

Makefile

@@ -13,7 +13,7 @@ install-backend-dependencies:
 	pip install -r backend/requirements/base.txt
 	pip install -r backend/requirements/dev.txt
 
-install-dependencies: backend-dependencies install-web-frontend-dependencies
+install-dependencies: install-backend-dependencies install-web-frontend-dependencies
 
 eslint-web-frontend:
 	(cd web-frontend && yarn run eslint) || exit;
@@ -32,4 +32,6 @@ lint-backend:
 test-backend:
 	(cd backend && pytest tests) || exit;
 
-make lint-and-test: lint-backend lint-web-frontend test-backend test-web-frontend
+lint: lint-backend lint-web-frontend
+test: test-backend test-web-frontend
+lint-and-test: lint test

backend/manage.py

@@ -1,21 +0,0 @@
-#!/usr/bin/env python
-"""Django's command-line utility for administrative tasks."""
-import os
-import sys
-
-
-def main():
-    os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'baserow.config.settings.dev')
-    try:
-        from django.core.management import execute_from_command_line
-    except ImportError as exc:
-        raise ImportError(
-            "Couldn't import Django. Are you sure it's installed and "
-            "available on your PYTHONPATH environment variable? Did you "
-            "forget to activate a virtual environment?"
-        ) from exc
-    execute_from_command_line(sys.argv)
-
-
-if __name__ == '__main__':
-    main()

backend/requirements/dev.txt

@@ -1,3 +1,3 @@
 flake8==3.7.7
-pytest-django==3.5.0
+pytest-django>=3.5.0
 Faker==1.0.7

sandbox/yarn.lock

@@ -935,7 +935,7 @@
     webpack-node-externals "^1.7.2"
     webpackbar "^3.2.0"
 
-"@nuxtjs/axios@^5.3.6":
+"@nuxtjs/axios@^5.5.4":
   version "5.5.4"
   resolved "https://registry.yarnpkg.com/@nuxtjs/axios/-/axios-5.5.4.tgz#c4aee2322901b19d4072670c03144662a73ea6f4"
   integrity sha512-/Ljsyh5VIc9paXGrQue7RQ+PpBNES1oit0g4l+ya1tfyKnZMpHSbghuLcv0xq+BpXlSEr690uemHbz54/N6U5w==
@@ -962,6 +962,11 @@
     mustache "^2.3.0"
     stack-trace "0.0.10"
 
+"@types/cookie@^0.3.1":
+  version "0.3.3"
+  resolved "https://registry.yarnpkg.com/@types/cookie/-/cookie-0.3.3.tgz#85bc74ba782fb7aa3a514d11767832b0e3bc6803"
+  integrity sha512-LKVP3cgXBT9RYj+t+9FDKwS5tdI+rPBXaNSkma7hvqy35lc7mAokC2zsqWJH0LaqIt3B962nuYI77hsJoT1gow==
+
 "@types/q@^1.5.1":
   version "1.5.2"
   resolved "https://registry.yarnpkg.com/@types/q/-/q-1.5.2.tgz#690a1475b84f2a884fd07cd797c00f5f31356ea8"
@@ -2186,6 +2191,22 @@ cookie-signature@1.0.6:
   resolved "https://registry.yarnpkg.com/cookie-signature/-/cookie-signature-1.0.6.tgz#e303a882b342cc3ee8ca513a79999734dab3ae2c"
   integrity sha1-4wOogrNCzD7oylE6eZmXNNqzriw=
 
+cookie-universal-nuxt@^2.0.16:
+  version "2.0.16"
+  resolved "https://registry.yarnpkg.com/cookie-universal-nuxt/-/cookie-universal-nuxt-2.0.16.tgz#8d528098c973162b352199240e40da0e5429b13f"
+  integrity sha512-wRK2zw8w+a5xPehb5kLbgOic/4mbjl2exUCxWZwGuttcwsFgOymiwDrCOzmQslqrDevPDL2SsBbH6wtOm7dB9g==
+  dependencies:
+    "@types/cookie" "^0.3.1"
+    cookie-universal "^2.0.16"
+
+cookie-universal@^2.0.16:
+  version "2.0.16"
+  resolved "https://registry.yarnpkg.com/cookie-universal/-/cookie-universal-2.0.16.tgz#ec8b55789b502a377ef02ad230923c1dfa5c1061"
+  integrity sha512-EHtQ5Tg3UoUHG7LmeV3rlV3iYthkhUuYZ0y86EseypxGcUuvzxuHExEb6mHKDhDPrIrdewAHdG/aCHuG/T4zEg==
+  dependencies:
+    "@types/cookie" "^0.3.1"
+    cookie "^0.3.1"
+
 cookie@0.4.0:
   version "0.4.0"
   resolved "https://registry.yarnpkg.com/cookie/-/cookie-0.4.0.tgz#beb437e7022b3b6d49019d088665303ebe9c14ba"
@@ -4200,6 +4221,11 @@ jsprim@^1.2.2:
     json-schema "0.2.3"
     verror "1.10.0"
 
+jwt-decode@^2.2.0:
+  version "2.2.0"
+  resolved "https://registry.yarnpkg.com/jwt-decode/-/jwt-decode-2.2.0.tgz#7d86bd56679f58ce6a84704a657dd392bba81a79"
+  integrity sha1-fYa9VmefWM5qhHBKZX3TkruoGnk=
+
 kind-of@^3.0.2, kind-of@^3.0.3, kind-of@^3.2.0:
   version "3.2.2"
   resolved "https://registry.yarnpkg.com/kind-of/-/kind-of-3.2.2.tgz#31ea21a734bab9bbb0f32466d893aea51e4a3c64"
@@ -4317,11 +4343,6 @@ lodash.memoize@^4.1.2:
   resolved "https://registry.yarnpkg.com/lodash.memoize/-/lodash.memoize-4.1.2.tgz#bcc6c49a42a2840ed997f323eada5ecd182e0bfe"
   integrity sha1-vMbEmkKihA7Zl/Mj6tpezRguC/4=
 
-lodash.merge@^4.6.1:
-  version "4.6.1"
-  resolved "https://registry.yarnpkg.com/lodash.merge/-/lodash.merge-4.6.1.tgz#adc25d9cb99b9391c59624f379fbba60d7111d54"
-  integrity sha512-AOYza4+Hf5z1/0Hztxpm2/xiPZgi/cjMqdnKTUWTBSKchJlxXXuUSxCCl8rJlf4g6yww/j6mA8nC8Hw/EZWxKQ==
-
 lodash.tail@^4.1.1:
   version "4.1.1"
   resolved "https://registry.yarnpkg.com/lodash.tail/-/lodash.tail-4.1.1.tgz#d2333a36d9e7717c8ad2f7cacafec7c32b444664"

web-frontend/config/nuxt.config.dev.js

@@ -1,4 +1,4 @@
-import merge from 'lodash.merge'
+import _ from 'lodash'
 import StyleLintPlugin from 'stylelint-webpack-plugin'
 
 import base from './nuxt.config.base.js'
@@ -24,4 +24,4 @@ const config = {
   }
 }
 
-export default merge(base, config)
+export default _.assign(base, config)

web-frontend/config/nuxt.config.production.js

@@ -3,18 +3,18 @@ import path from 'path'
 import base from './nuxt.config.base.js'
 
 export default function(rootDir) {
-  const merge = require(rootDir + '/node_modules/lodash.merge')
+  const _ = require(rootDir + '/node_modules/lodash')
 
   /**
-   * Because the nuxt source files are located in we web-frontend directory, but
-   * the project is started from another directory we have to explicitly set the
-   * source directory which contains the nuxt files and the root directory which
-   * contains the node modules.
+   * Because the nuxt source files are located in the web-frontend directory,
+   * but the project is started from another directory we have to explicitly set
+   * the source directory which contains the nuxt files and the root directory
+   * which contains the node modules.
    */
   const config = {
     rootDir: rootDir,
     srcDir: path.resolve(__dirname, '../')
   }
 
-  return merge(base, config)
+  return _.assign(base, config)
 }

web-frontend/config/nuxt.config.test.js

@@ -0,0 +1,17 @@
+import { resolve } from 'path'
+import _ from 'lodash'
+
+import base from './nuxt.config.base.js'
+
+const config = {
+  rootDir: resolve(__dirname, '../'),
+  css: [],
+  dev: false,
+  debug: false,
+  env: {
+    // The API base url, this will be prepended to the urls of the remote calls.
+    baseUrl: 'http://localhost/api/v0'
+  }
+}
+
+export default _.assign({}, base, config)

web-frontend/jest.config.js

@@ -1,4 +1,7 @@
 module.exports = {
+  testEnvironment: 'node',
+  expand: true,
+  forceExit: true,
   moduleNameMapper: {
     '^@/(.*)$': '<rootDir>/$1',
     '^~/(.*)$': '<rootDir>/$1',

web-frontend/package.json

@@ -11,7 +11,7 @@
     "start": "nuxt start",
     "eslint": "eslint --ext .js,.vue .",
     "stylelint": "stylelint **/*.scss --syntax scss",
-    "test": "jest"
+    "test": "jest -i --verbose false test/"
   },
   "dependencies": {
     "@fortawesome/fontawesome-free": "^5.8.2",
@@ -19,7 +19,7 @@
     "cookie-universal-nuxt": "^2.0.16",
     "cross-env": "^5.2.0",
     "jwt-decode": "^2.2.0",
-    "lodash.merge": "^4.6.1",
+    "lodash": "^4.17.11",
     "node-sass": "^4.12.0",
     "normalize-scss": "^7.0.1",
     "nuxt": "^2.4.0",
@@ -45,6 +45,9 @@
     "eslint-plugin-standard": ">=4.0.0",
     "eslint-plugin-vue": "^5.2.2",
     "jest": "^24.1.0",
+    "jsdom": "^15.1.1",
+    "moxios": "^0.4.0",
+    "node-mocks-http": "^1.7.6",
     "nodemon": "^1.18.9",
     "prettier": "^1.16.4",
     "stylelint": "^9.2.1",

web-frontend/pages/index.vue

@@ -1,12 +1,12 @@
 <template>
-  <div></div>
+  <div>{{ test }}</div>
 </template>
 
 <script>
 export default {
-  mounted() {
-    const name = this.$store.getters['auth/isAuthenticated'] ? 'app' : 'login'
-    this.$nuxt.$router.replace({ name: name })
+  fetch({ store, redirect }) {
+    const name = store.getters['auth/isAuthenticated'] ? 'app' : 'login'
+    redirect({ name: name })
   }
 }
 </script>

web-frontend/pages/login/signup.vue

@@ -123,10 +123,10 @@ export default {
       error: '',
       loading: false,
       account: {
-        email: 'lol@lol.nl',
-        name: 'lol',
-        password: 'lol',
-        passwordConfirm: 'lol'
+        email: '',
+        name: '',
+        password: '',
+        passwordConfirm: ''
       }
     }
   },

web-frontend/test/config/config.spec.js

@@ -0,0 +1,20 @@
+import createNuxt from '@/test/helpers/create-nuxt'
+
+let nuxt = null
+
+describe('Config', () => {
+  beforeAll(async done => {
+    nuxt = await createNuxt(true)
+    done()
+  }, 120000)
+
+  test('Plugins', () => {
+    const plugins = nuxt.options.plugins.map(option => option.src)
+    expect(plugins.includes('@/plugins/auth.js')).toBe(true)
+    expect(plugins.includes('@/plugins/vuelidate.js')).toBe(true)
+  })
+
+  afterAll(async () => {
+    await nuxt.close()
+  })
+})

web-frontend/test/helpers/create-nuxt.js

@@ -0,0 +1,13 @@
+import { Builder, Nuxt } from 'nuxt'
+
+import config from '@/config/nuxt.config.test'
+
+export default async function createNuxt(buildAndListen = false) {
+  const nuxt = new Nuxt(config)
+  await nuxt.ready()
+  if (buildAndListen) {
+    await new Builder(nuxt).build()
+    await nuxt.server.listen(3001, 'localhost')
+  }
+  return nuxt
+}

web-frontend/test/pages/index.spec.js

@@ -1,3 +1,56 @@
-describe('Home', () => {
-  test('is a Vue instance', () => {})
+import moxios from 'moxios'
+import httpMocks from 'node-mocks-http'
+
+import createNuxt from '@/test/helpers/create-nuxt'
+
+let nuxt = null
+
+describe('index redirect', () => {
+  beforeAll(async done => {
+    moxios.install()
+
+    // Because the token 'test1' exists it will be refreshed immediately, the
+    // refresh endpoint is stubbed so that it will always provide a valid
+    // unexpired token.
+    moxios.stubRequest('http://localhost/api/v0/user/token-refresh/', {
+      status: 200,
+      response: {
+        token:
+          'eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJ1c2VyX2lkIjoxLCJ1c2' +
+          'VybmFtZSI6InRlc3RAdGVzdCIsImV4cCI6MTk5OTk5OTk5OSwiZW1haWwiO' +
+          'iJ0ZXN0QHRlc3QubmwiLCJvcmlnX2lhdCI6MTU2Mjc3MzQxNH0.2i0gqrcH' +
+          '5uy7mk4kf3LoLpZYXoyMrOfi0fDQneVcaFE',
+        user: {
+          first_name: 'Test',
+          username: 'test@test.nl'
+        }
+      }
+    })
+
+    nuxt = await createNuxt(true)
+    done()
+  }, 120000)
+
+  test('if not authenticated', async () => {
+    const { redirected } = await nuxt.server.renderRoute('/')
+    expect(redirected.path).toBe('/login')
+    expect(redirected.status).toBe(302)
+  })
+
+  test('if authenticated', async () => {
+    const req = httpMocks.createRequest({
+      headers: {
+        cookie: 'jwt_token=test1'
+      }
+    })
+    const res = httpMocks.createResponse()
+    const { redirected } = await nuxt.server.renderRoute('/', { req, res })
+    expect(redirected.path).toBe('/app')
+    expect(redirected.status).toBe(302)
+  })
+
+  afterAll(async () => {
+    await nuxt.close()
+    moxios.uninstall()
+  })
 })

web-frontend/test/pages/login/index.spec.js

@@ -0,0 +1,19 @@
+import createNuxt from '@/test/helpers/create-nuxt'
+
+let nuxt = null
+
+describe('children', () => {
+  beforeAll(async done => {
+    nuxt = await createNuxt(true)
+    done()
+  }, 120000)
+
+  test('/login', async () => {
+    const { html } = await nuxt.server.renderRoute('/login')
+    expect(html).toContain('Login')
+  })
+
+  afterAll(async () => {
+    await nuxt.close()
+  })
+})

web-frontend/test/pages/login/signup.spec.js

@@ -0,0 +1,19 @@
+import createNuxt from '@/test/helpers/create-nuxt'
+
+let nuxt = null
+
+describe('children', () => {
+  beforeAll(async done => {
+    nuxt = await createNuxt(true)
+    done()
+  }, 120000)
+
+  test('/login', async () => {
+    const { html } = await nuxt.server.renderRoute('/login/signup')
+    expect(html).toContain('Sign up')
+  })
+
+  afterAll(async () => {
+    await nuxt.close()
+  })
+})