getMacroMatches panics if given a macro arg string missing its close bracket

[njvrzm]

What happened:
getMacroMatches panics if given a query string that's missing the close bracket of an argument list, eg: select * from foo where $__timeTo(uhoh.

What you expected to happen:
An error should be returned instead and bubbled up from Interpolate.

How to reproduce it (as minimally and precisely as possible):
Remove the close parenthesis from any test in macros_test.go/TestInterpolate, like this one.

[academo]

@njvrzm can you confirm if this issue is closed by the commits you pushed mentioning this issue?