You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Tests with more than one newlines in names are not runnable from CodeLens links #4125
Thanks. I think there's something slightly different going on here, as if I use the original case from that issue things still work, but the case above does not:
// This one failstest(''' Given formType is signIn When signInWithEmailAndPassword succeeds Then return true And state is AsyncData ''', () async {});
// This one workstestWidgets(''' test ''', (tester) async {});
So let's keep this issue open and use it for this rather than re-open that one.
changed the title [-]I just wanted to make sure this issue gets reopened ... #4007[/-][+]Some tests with multiline names are not runnable from CodeLens links[/+]on Aug 30, 2022
changed the title [-]Some tests with multiline names are not runnable from CodeLens links[/-][+]Tests with more than one newlines in names are not runnable from CodeLens links[/+]on Aug 30, 2022
Activity
DanTup commentedon Aug 30, 2022
Thanks. I think there's something slightly different going on here, as if I use the original case from that issue things still work, but the case above does not:
So let's keep this issue open and use it for this rather than re-open that one.
[-]I just wanted to make sure this issue gets reopened ... #4007[/-][+]Some tests with multiline names are not runnable from CodeLens links[/+][-]Some tests with multiline names are not runnable from CodeLens links[/-][+]Tests with more than one newlines in names are not runnable from CodeLens links[/+]DanTup commentedon Aug 30, 2022
This turned out to be the same issue but when there was more than one newline. It was a bad regex escape that only replaced the first:
Not the first time I've had a bug like that, perhaps I need a lint :-)
jeff9315 commentedon Aug 30, 2022
Ironic that this bug was in the testing module :-)
Thanks for the quick fix.
Handle test names with multiple lines