Improved Coverage

Author: presidenten

src/api/global.spec.js

@@ -1,4 +1,5 @@
 import global from './global.js';
+import * as api from './api.js';
 import vuexInstance from '../vuexInstance.js';
 
 beforeEach(() => {
@@ -12,6 +13,14 @@ beforeEach(() => {
   };
 });
 
+describe('global.api', () => {
+  it('returns the api', () => {
+    const newApi = { id: 42 };
+    api.setApi(newApi);
+    expect(global.api).toEqual(newApi);
+  });
+});
+
 describe('global.get', () => {
   it('returns getter value for fullpath', () => {
     global.get('foo/bar');

src/common/helpers.js

@@ -7,7 +7,7 @@ export const getStoreInstanceName = (storeName, instance) => {
 
 
 export const toCamelCase = (str) => {
-  if (!str) {
+  if (!str || typeof str !== 'string') {
     return '';
   }
   return str.replace(/(-|_)([\w])/g, s => s[1].toUpperCase());

src/common/helpers.spec.js

@@ -23,6 +23,10 @@ describe('helpers.toCamelCase', () => {
   it('keeps camel case intact', () => {
     expect(helpers.toCamelCase('aFooBarA$b')).toEqual('aFooBarA$b');
   });
+  it('returns empty string if invalid value', () => {
+    expect(helpers.toCamelCase(undefined)).toEqual('');
+    expect(helpers.toCamelCase(5)).toEqual('');
+  });
 });
 
 describe('helpers.getLocalPath', () => {
@@ -121,4 +125,8 @@ describe('helpers.getSubInstances', () => {
   it('returns all instances from substring', () => {
     expect(helpers.getSubInstances('a$a1/b/c$c1/d')).toEqual(['$a1', '$c1']);
   });
+
+  it('returns empty array on empty path', () => {
+    expect(helpers.getSubInstances('')).toEqual([]);
+  });
 });