fix(): Fixing context passing while calling child model function (dollarshaveclub#182)

* fix(): Fixing context passing while calling child model function

dollarshaveclub#181

* Fixing circle ci build to build the dist files

Author: hemkaran

.circleci/config.yml

@@ -25,6 +25,7 @@ jobs:
       - restore_cache: *restore_cache
       - run: npm install
       - run: npm run eslint:ci
+      - run: npm run build
       - run: npm test
       - run: npx codecov
       - save_cache: *save_cache

.gitignore

@@ -9,3 +9,4 @@ coverage/
 coverage.lcov
 coverage_*/
 yarn.lock
+.idea/

src/postmate.js

@@ -202,7 +202,7 @@ export class ChildAPI {
 
       if (e.data.postmate === 'call') {
         if (property in this.model && typeof this.model[property] === 'function') {
-          this.model[property].call(this, data)
+          this.model[property](data)
         }
         return
       }

test/acceptance/fixtures/child.html

@@ -45,7 +45,10 @@
         a: a,
         b: b,
         setRandomId: setRandomId,
-        getRandomId: getRandomId
+        getRandomId: getRandomId,
+        setFoo: function (foo) {
+          this.foo = foo;
+        }
       })
       .then(function (parent) { parentAPI = parent; });
     </script>

test/acceptance/test.js

@@ -69,6 +69,21 @@ describe('postmate', function () {
     })
   })
 
+  it('should call a function in the child model with the child model context', function (done) {
+    new Postmate({
+      container: document.getElementById('frame'),
+      url: 'http://localhost:9000/child.html',
+    }).then(function (child) {
+      child.call('setFoo', 'bar')
+      child.get('foo').then(function (foo) {
+        expect(foo).to.equal('bar')
+        child.destroy()
+        done()
+      })
+        .catch(function (err) { done(err) })
+    })
+  })
+
   it('should fetch values from the child model from defaults set by the parent', function (done) {
     var uid = new Date().getTime()