MessageGroup.replaceMessage does not work as expected

[victorchapado]

Hi! There is an error with the implementation of MessageGroup.replaceMessage. Right now, the first condition always returns false because the length of the array of messages is always greater than the index that we need to replace. Besides that, the concatenation preserves the original messages array so, this one increases each time that this method is called.

My code suggestion:

replaceMessage(message, index) { 
  if (this.messages.length <= index) {
    return false;
  }

  this.messages = this.messages.slice(0, index)
    .concat(message)
    .concat(this.messages.slice(index + 1));

  return true;
}

[supersnager]

:tada: This issue has been resolved in version 2.0.2 :tada:

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:

[supersnager]

@victorchapado Thank you for the report!