Merge pull request #365 from mikeu/unit-tests

WIP adding unit tests

Author: Nicolò Maria Mezzopera

jest.config.js

@@ -5,13 +5,17 @@ module.exports = {
   coverageDirectory: 'coverage',
   restoreMocks: true,
   moduleFileExtensions: ['js', 'jsx', 'json', 'vue'],
+  setupFiles: [
+    '<rootDir>/tests/setup.js'
+  ],
   transform: {
     '^.+\\.vue$': 'vue-jest',
     '.+\\.(css|styl|less|sass|scss|svg|png|jpg|ttf|woff|woff2)$':
       'jest-transform-stub',
     '^.+\\.jsx?$': 'babel-jest'
   },
   moduleNameMapper: {
+    '^@/tests/(.*)$': '<rootDir>/tests/$1',
     '^@/(.*)$': '<rootDir>/src/$1'
   },
   snapshotSerializers: ['jest-serializer-vue'],

package.json

@@ -19,7 +19,7 @@
     "@babel/plugin-transform-runtime": "^7.3.4",
     "@babel/preset-env": "^7.3.1",
     "@babel/runtime": "^7.3.4",
-    "@vue/test-utils": "^1.0.0-beta.20",
+    "@vue/test-utils": "^1.0.0-beta.29",
     "babel-eslint": "^8.2.6",
     "babel-jest": "^24.1.0",
     "coveralls": "^3.0.3",

src/components/LCircle.vue

@@ -15,7 +15,7 @@ export default {
   props: {
     latLng: {
       type: [Object, Array],
-      default: () => []
+      default: () => [0, 0]
     }
   },
   data () {

src/components/LCircleMarker.vue

@@ -15,7 +15,7 @@ export default {
   props: {
     latLng: {
       type: [Object, Array],
-      default: () => []
+      default: () => [0, 0]
     },
     pane: {
       type: String,

src/utils/utils.js

@@ -16,6 +16,7 @@ export const debounce = (fn, time) => {
 };
 
 export const capitalizeFirstLetter = (string) => {
+  if (!string || typeof string.charAt !== 'function') { return string; }
   return string.charAt(0).toUpperCase() + string.slice(1);
 };
 
@@ -78,7 +79,7 @@ export const optionsMerger = (props, instance) => {
 
 export const findRealParent = (firstVueParent) => {
   let found = false;
-  while (!found) {
+  while (firstVueParent && !found) {
     if (firstVueParent.mapObject === undefined) {
       firstVueParent = firstVueParent.$parent;
     } else {

tests/mixin-tests/circle-tests.js

@@ -0,0 +1,32 @@
+import { getWrapperWithMap } from '@/tests/test-helpers';
+import { testPathFunctionality } from './path-tests';
+
+export const testCircleFunctionality = (component, componentName = 'it') => {
+  // The Circle mixin includes the Path mixin.
+  testPathFunctionality(component, componentName);
+
+  // Radius
+
+  test(`${componentName} has a getRadius method`, () => {
+    const { wrapper } = getWrapperWithMap(component);
+
+    expect(typeof wrapper.vm.mapObject.getRadius).toEqual('function');
+  });
+
+  test(`${componentName} accepts and uses radius prop`, () => {
+    const radius = 100;
+    const { wrapper } = getWrapperWithMap(component, { radius });
+
+    expect(wrapper.vm.mapObject.getRadius()).toEqual(radius);
+  });
+
+  test(`${componentName} updates radius when prop changes`, async () => {
+    const { wrapper } = getWrapperWithMap(component, { radius: 100 });
+
+    const newRadius = 2000;
+    wrapper.setProps({ radius: newRadius });
+    await wrapper.vm.$nextTick();
+
+    expect(wrapper.vm.mapObject.getRadius()).toEqual(newRadius);
+  });
+};