Member

See #4021.

• Address // TODO(dantup): This is right for Flutter, but must check package:test version for Dart too!
• Remove const runTestsByLine = true
• Address // TODO(dantup): Ensure we don't get here for Dart SDK or Bazel where we shouldn't use "dart run"?
• Address // TODO(dantup): WIRE THIS UP!
• Review if findMatchingDynamicNode should be taking lines into account?
• Ensure there are tests that cover multiple tests in one run ("program": "foo_test.dart?line=1", "args": ["foo_test.dart?line=20"])
  • manually tested
• Test all issues noted/referenced in Run tests using line/col to support groups/test with dynamic names #4021
• Address // TODO(dantup): Ensure this never runs in the SDK, because it may produce unwanted package_configs?
• Revert setting the default to "line" that was used for testing on the bots ( c00add9 )
• Ensure Flutter has automated tests in the same way as Dart
  • manually tested

Fixes several issues:

• Fixes Run tests using line/col to support groups/test with dynamic names #4021
• Fixes Multiple tests execution #4150
• Fixes Parens around test name prevents CodeLens links from working #4168
• Fixes "No tests match regular expression...", customize regex #4099
• Fixes Debugging flutter tests fail if the descrription string has an interpolated value and a is composed of two concatenated strings #4250

See #4021.

Fix some minor issues when running tests by line

0fdca26

Don't hard-code decision to run by line

cfe6404

Handle both flutter+package:test capabilities for running by line

37f7749

…ific

Prevent running "dart run test:test --version" in the SDK

0e5f82e

See #4021
See #4150
See #4168
See #4099
See #4250

…cy in the hello_world sample project

Temporarily set test invocation mode to line for CI testing

f4f0fd7

This reverts commit  f4f0fd7 .

Ensure all flutter test tests run by line + name

4b9158b

…e runner

Fix tests

d9e6d19

DanTup commented Apr 19, 2023

View reviewed changes