Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tests] verify_checks_test is not being run (or is ignored) on the bots #59867

Open
pq opened this issue Jan 8, 2025 · 2 comments
Open

[tests] verify_checks_test is not being run (or is ignored) on the bots #59867

pq opened this issue Jan 8, 2025 · 2 comments
Labels
analyzer-linter Issues with the analyzer's support for the linter package analyzer-test area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on

Comments

@pq
Copy link
Member

pq commented Jan 8, 2025

The test verify_checks_test.dart is failing locally but not causing a breakage on the bots. How come?

Local run:

01:11 +5373 -1: custom checks visit_registered_nodes, no_solo_tests, no_trailing_spaces [E]

  Expected: empty
    Actual: [
              AnalysisError:/Users/pquitslund/src/repos/dart/sdk/pkg/linter/lib/src/rules/strict_top_level_inference.dart(1403..1426): Declare 'visit' methods for all registered node types.,
              AnalysisError:/Users/pquitslund/src/repos/dart/sdk/pkg/linter/test/rules/unnecessary_underscores_test.dart(1392..1441): Don't create string literals with trailing spaces in tests.,
              AnalysisError:/Users/pquitslund/src/repos/dart/sdk/pkg/linter/test/rules/unnecessary_underscores_test.dart(1521..1568): Don't create string literals with trailing spaces in tests.
            ]
  

  package:matcher/src/expect/expect.dart 152:31     fail
  package:matcher/src/expect/expect.dart 147:3      _expect
  package:matcher/src/expect/expect.dart 56:3       expect
  test/verify_checks_test.dart 14:7                 main.<fn>.<fn>
verify_checks_test.dart:14
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 229:9  Declarer.test.<fn>.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 227:7  Declarer.test.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/invoker.dart 258:9   Invoker._waitForOutstandingCallbacks.<fn>

/fyi @srawlins: the strict_top_level_inference failure is legit and worth looking at; I'm fixing the unnecessary_underscores_test ones

@pq pq added area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. analyzer-linter Issues with the analyzer's support for the linter package analyzer-test P2 A bug or feature request we're likely to work on labels Jan 8, 2025
@srawlins
Copy link
Member

srawlins commented Jan 8, 2025

Ah I'll look at strict_top_level_inference. Thanks for noticing!

@pq
Copy link
Member Author

pq commented Jan 8, 2025

@stereotype441 pointed me at this:

https://dart-current-results.web.app/#/filter=pkg/linter/test/verify_checks_test&showAll

which suggests that (at least the last time the tests ran) they passed. The plot thickens!

@athomas?

copybara-service bot pushed a commit that referenced this issue Jan 8, 2025
…ource

This *should* be breaking the bots (see: #59867)

Change-Id: I3855954e28e442f189ae9b716a432fc1e1d409fb
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/403741
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Phil Quitslund <pquitslund@google.com>
copybara-service bot pushed a commit that referenced this issue Jan 8, 2025
Work towards #59867

Change-Id: I61e82e43252b24b63ef04f511cac3cbe70771308
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/403704
Commit-Queue: Samuel Rawlins <srawlins@google.com>
Commit-Queue: Phil Quitslund <pquitslund@google.com>
Auto-Submit: Samuel Rawlins <srawlins@google.com>
Reviewed-by: Phil Quitslund <pquitslund@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer-linter Issues with the analyzer's support for the linter package analyzer-test area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on
Projects
None yet
Development

No branches or pull requests

2 participants