Promise.bind does not bind context for the entire promise chain when thisArg is a Promise

[seebees]

Perhaps I do not understand what bind is suppose to do? I would expect both of these tests to pass.

node: 0.12.7
bluebird: 2.9.33

var expect = require('chai').expect;
var Promise = require('bluebird');

describe('Derived promises should have the bound context', function () {

  it('when the context is an object', function() {
    return Promise
      .bind({hasContext: true})
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
  })

  // This fails :(
  it('when the context is a Promise', function() {
    return Promise
      .bind(new Promise(function(resolve){
        setTimeout(function(){
          resolve({hasContext: true})
        }, 100)
      }))
      .then(function(){
        // this will pass
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
         // these all fail
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
      .then(function(){
        expect(this.hasContext).to.be.true()
      })
  })
})

[petkaantonov]

In the latter case it's the test that's broken.

You have resolve({getId: function(){return 1}}) while it should be resolve({hasContext: true})

[seebees]

Thanks! Especially for looking at the test again. I'm sorry I did not catch that. I edited it for posterity.

I'm sure you have heard it before, but great library, and thanks for all the time you put into maintaining it.