Consistency with returning new Assertion instead of this

A change was made a few months ago to return a new Assertion object with flags copied over instead of returning this. References: #562 #642

However, it looks like there are still some spots where this is returned. The only one I’ve actually tested so far is overwriteProperty.

For example:

const {expect, use} = require("chai");                                                                                  

use(function (chai, utils) {                                                                                            
  utils.overwriteProperty(chai.Assertion.prototype, "and", function (_super) {                                          
    return function blah () {                                                                                           
      _super.call(this);                                                                                                
    };                                                                                                                  
  });                                                                                                                   
});                                                                                                                     

it("but", () => {                                                                                                      
  expect([1, 2, 3]).to.be.an.but.length.below(1000)    // Passes
});

it("and", () => {                                                                                                      
  expect([1, 2, 3]).to.be.an.and.length.below(1000)    // Fails
});

As expected, the second assertion fails with error: “TypeError: expect(…).to.be.an.and.length.below is not a function” because the overwritten and is returning the function that was originally returned by the chainable method an instead of returning a new Assertion.

Other places where this might be a problem (but I haven’t tested):

• overwriteMethod
• addChainableMethod
• overwriteChainableMethod

[lucasfcosta]

@vieiralucas This PR #642 has more details about it. I think this post explains the reason behind this whole thing, I highly recommend anyone involved with this issue to read that.

[vieiralucas]

YES SIR! 😸

I’m still reading through all the issues and PR’s involved in this issue. I think I’ll be able to submit a PR addressing thi later today.