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.

Sign up for GitHub

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

Jump to bottom

add docs about reflective test authoring #58809

Open
pq opened this issue Aug 2, 2022 · 3 comments
Open

add docs about reflective test authoring #58809

pq opened this issue Aug 2, 2022 · 3 comments
Labels
area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) devexp-linter Issues with the analyzer's support for the linter package P2 A bug or feature request we're likely to work on type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable type-documentation A request to add or improve documentation

Comments

@pq
Copy link
Member

pq commented Aug 2, 2022

Follow-up from: dart-lang/linter#3567 (comment).

As a general rule we want to encourage reflective tests over integration tests. It would be great to have some docs here.
@pq pq added type-documentation A request to add or improve documentation type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable labels Aug 2, 2022
@bwilkerson
Copy link
Member

Consider adding the docs to test_reflective_loader and referencing them from here (and from analyzer and analysis_server and any other package that uses that support).

@srawlins srawlins added the P2 A bug or feature request we're likely to work on label Sep 23, 2022
@devoncarew devoncarew added devexp-linter Issues with the analyzer's support for the linter package area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. labels Nov 19, 2024
@devoncarew devoncarew transferred this issue from dart-lang/linter Nov 19, 2024
@srawlins
Copy link
Member

Now that all tests are reflective, do we still need docs encouraging their use? Should we just repurpose this ticket into a ticket to document how to write reflective tests?

@bwilkerson
Copy link
Member

Or point people to an enhanced version of https://github.com/dart-lang/sdk/blob/main/pkg/analyzer/doc/implementation/tests.md?

@bwilkerson bwilkerson added area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) and removed area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. labels Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area-devexp Developer Experience related issues (DevTools, IDEs, Analysis Server, completions, refactoring, etc) devexp-linter Issues with the analyzer's support for the linter package P2 A bug or feature request we're likely to work on type-code-health Internal changes to our tools and workflows to make them cleaner, simpler, or more maintainable type-documentation A request to add or improve documentation
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@pq @srawlins @devoncarew @bwilkerson