Multiple children on same origin

[aaronius]

Multiple children on same origin

[Aaronius]

This would be a fantastic contribution. Thank you for offering. Contributing guidelines:

Try to follow the conventions already in the code. Don't be offended if I tweak it later.
File size is a feature. Consider it as such.
Add tests where you can (looks like you already have the idea).

Thanks!

[loganvolkers]

Well shoot. My fix of adding a childId didn't fix my test. Turns out the problem was in my test code.

Here's the revised test code:

  it('should call an asynchronous function on the child, but only the right child!', (done) => {
    const c1 = Penpal.connectToChild({
      url: `http://${HOST}:9000/child.html`
    });
    const c2 = Penpal.connectToChild({
      url: `http://${HOST}:9000/child.html`
    });

    c1.promise.then((child) => {
      child.multiplyAsync(2, 5).then((value) => {
        expect(value).toEqual(10);
        c1.destroy();
        done();
      });
    });
    c2.promise.then((child) => {
      child.multiplyAsync(3, 5).then((value) => {
        expect(value).toEqual(15);
        c2.destroy();
        done();
      });
    });

  });

Also today I learned that using event.source and data.id already provides adequate filtering for this use case.

          if (event.source === remote &&
              event.origin === remoteOrigin &&
              event.data.penpal === REPLY &&
              event.data.id === id) {

Thanks for the prompt response! Looking forward to using Penpal for more.

[Aaronius]

This test has been added to the test suite: 3e61c23

I tweaked it a bit to make sure done will only get called after both method calls resolve.