Allow concatenation when ensuring assertion message is a string

[Yash-Singh1]

Currently, doing:

t.notThrowsAsync(asyncNotThrowingFn, 'abc' + someString)

Would fail, saying that the assertion message is not a string. This is happening over here:

eslint-plugin-ava/rules/assertion-arguments.js

Line 206 in dee1802

function isString(node) {

The function should also support concatenation. Looking at the AST Explorer, I think that the function should run recursively and recheck its left and its right to ensure that they are either a Binary Expression or a String Literal, or an Identifier.

[sindresorhus]

I think we can make it better by using getStaticValue there.