Fix CommonJS module.exports, edit tests

.jshintrc

@@ -13,5 +13,8 @@
   "noarg": true,
   "quotmark": "single",
   "undef": true,
-  "unused": true
+  "unused": true,
+  "globals": {
+    "define": true
+  }
 }

README.md

@@ -1,16 +1,13 @@
 # Jaunt.js [![Build Status](https://img.shields.io/travis/yuanqing/jaunt.svg?style=flat)](https://travis-ci.org/yuanqing/jaunt) [![Coverage Status](https://img.shields.io/coveralls/yuanqing/jaunt.svg?style=flat)](https://coveralls.io/r/yuanqing/jaunt)
 
-> Access a value in an object or array given a dot-delimited string.
+> Access a value in an object or array using a dot-delimited string.
 
 ## Usage
 
 ```js
 var obj = {
   foo: {
-    bar: [
-      { baz: 'Hello' },
-      { qux: 'World' }
-    ]
+    bar: ['Hello', 'World']
   }
 };
 

dist/jaunt.js

@@ -1,5 +1,13 @@
-/* globals define */
-var f = function () {
+(function (root, factory) {
+  /* istanbul ignore next */
+  if (typeof define === 'function' && define.amd) {
+    define(factory);
+  } else if (typeof exports === 'object') {
+    module.exports = factory();
+  } else {
+    root.jaunt = factory();
+  }
+})(this, function() {
 
   'use strict';
 
@@ -22,15 +30,4 @@ var f = function () {
     return obj;
   };
 
-};
-
-/* istanbul ignore next */
-(function (root, factory) {
-  if (typeof define === 'function' && define.amd) {
-    define(factory);
-  } else if (typeof exports === 'object') {
-    module.exports = factory;
-  } else {
-    root.jaunt = factory(root);
-  }
-})(this, f);
+});

gulpfile.js

@@ -45,4 +45,4 @@ gulp.task('default', [
   'lint',
   'dist',
   'test'
-]);
+]);

package.json

@@ -1,7 +1,7 @@
 {
   "name": "jaunt",
-  "version": "1.0.0",
-  "description": "Access a value in an object or array given a dot-delimited string.",
+  "version": "1.0.1",
+  "description": "Access a value in an object or array using a dot-delimited string.",
   "author": "Lim Yuan Qing",
   "license": "MIT",
   "repository": {

src/jaunt.js

@@ -1,5 +1,13 @@
-/* globals define */
-var f = function () {
+(function (root, factory) {
+  /* istanbul ignore next */
+  if (typeof define === 'function' && define.amd) {
+    define(factory);
+  } else if (typeof exports === 'object') {
+    module.exports = factory();
+  } else {
+    root.jaunt = factory();
+  }
+})(this, function() {
 
   'use strict';
 
@@ -22,15 +30,4 @@ var f = function () {
     return obj;
   };
 
-};
-
-/* istanbul ignore next */
-(function (root, factory) {
-  if (typeof define === 'function' && define.amd) {
-    define(factory);
-  } else if (typeof exports === 'object') {
-    module.exports = factory;
-  } else {
-    root.jaunt = factory(root);
-  }
-})(this, f);
+});

test/karma.conf.js

@@ -22,4 +22,4 @@ module.exports = function (config) {
       dir: 'coverage/'
     }
   });
-};
+};

test/spec/jaunt.spec.js

@@ -1,5 +1,4 @@
 /* globals describe, it, expect, jaunt */
-
 describe('jaunt', function() {
 
   var obj = {
@@ -15,31 +14,31 @@ describe('jaunt', function() {
     expect(jaunt(obj, 'foo')).toBe(obj.foo);
     expect(jaunt(obj, 'foo.bar')).toBe(obj.foo.bar);
     expect(jaunt(obj, 'foo.bar.0')).toBe(obj.foo.bar[0]);
-    expect(jaunt(obj, 'foo.bar.1')).toBe(obj.foo.bar[1]);
     expect(jaunt(obj, ' foo . bar . 0 ')).toBe(obj.foo.bar[0]); // whitespace is trimmed
+    expect(jaunt(obj, 'foo.bar.1.qux')).toBe(obj.foo.bar[1].qux);
   });
 
   it('returns undefined if the `path` string is invalid', function() {
     expect(jaunt(obj, '')).toBe(undefined);
     expect(jaunt(obj, 'invalid')).toBe(undefined);
     expect(jaunt(obj, 'foo.invalid')).toBe(undefined);
-    expect(jaunt(obj, 'foo.bar.0.invalid')).toBe(undefined);
+    expect(jaunt(obj, 'foo.bar.0.baz.invalid')).toBe(undefined);
   });
 
   it('returns a value in `obj` if `path` is an array', function() {
     expect(jaunt(obj, ['foo'])).toBe(obj.foo);
     expect(jaunt(obj, ['foo', 'bar'])).toBe(obj.foo.bar);
     expect(jaunt(obj, ['foo', 'bar', 0])).toBe(obj.foo.bar[0]);
-    expect(jaunt(obj, ['foo', 'bar', 1])).toBe(obj.foo.bar[1]);
     expect(jaunt(obj, [' foo ', ' bar ', 0])).toBe(obj.foo.bar[0]); // whitespace is trimmed
+    expect(jaunt(obj, ['foo', 'bar', 1, 'qux'])).toBe(obj.foo.bar[1].qux);
   });
 
   it('returns undefined if the `path` array is invalid', function() {
     expect(jaunt(obj, [])).toBe(undefined);
     expect(jaunt(obj, [''])).toBe(undefined);
     expect(jaunt(obj, ['invalid'])).toBe(undefined);
     expect(jaunt(obj, ['foo', 'invalid'])).toBe(undefined);
-    expect(jaunt(obj, ['foo', 'bar', 0, 'invalid'])).toBe(undefined);
+    expect(jaunt(obj, ['foo', 'bar', 0, 'baz', 'invalid'])).toBe(undefined);
   });
 
-});
+});