`dcm analyze` false positives

Open Beginner friendly
#9,942 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
68/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Active
Tech stack
dart

Research direction

Start by running dcm analyze against packages/devtools_app/test/screens/debugger/syntax_highlighter_test.dart and inspect the empty test group reported at line 514. Determine why avoid-empty-test-groups flags this case, then verify that analysis no longer reports this false positive while still detecting genuinely empty groups.

Written by the indexing model from the issue text.

Description

a: quality cost: low P2

@incendial finding some more false positives here. I can file these somewhere else if that is better for you, just let me know.

packages/devtools_app/test/screens/debugger/syntax_highlighter_test.dart (1 issue):
WARNING   This test group does not contain any test cases.
          at packages/devtools_app/test/screens/debugger/syntax_highlighter_test.dart:514:7
          avoid-empty-test-groups : https://dcm.dev/docs/rules/common/avoid-empty-test-groups
Dominant language
Dart
Stars
1.7k
Forks
405
Avg merge
4d 23h
Merged PRs (30d)
12

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from flutter/devtools

All issues in flutter/devtools

Similar issues

More Dart issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.