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

I wish test running was modular/cacheable #985

Closed
matanlurey opened this issue Feb 8, 2018 · 3 comments
Closed

I wish test running was modular/cacheable #985

matanlurey opened this issue Feb 8, 2018 · 3 comments
Labels
area-testing closed-not-planned Closed as we don't intend to take action on the reported issue P3 A lower priority bug or feature request package:build_runner type-enhancement A request for a change that isn't a bug type-performance

Comments

@matanlurey
Copy link
Contributor

For example, I have a PR where I removed some unused code in _test.dart files (8 to be precise), but the test runner will execute all of the tests again (locally, and on travis). I wish there was a way we could store the test graph and avoid re-running tests that couldn't have changed.

(I realize this is hard, related to #967, but even worse - we'd have to for example include the chrome binary as an external dependency and track if it changes)

@jakemac53
Copy link
Contributor

There is also another issue of people depending on external things we can't track in their tests. The simple example is a goldens test.

@matanlurey
Copy link
Contributor Author

I realize we become Bazel at some point, but I guess we could have input globs?

@matanlurey matanlurey added type-enhancement A request for a change that isn't a bug P3 A lower priority bug or feature request package:build_runner labels Feb 8, 2018
@jakemac53
Copy link
Contributor

I think if we could just override what dart:io does that would be ideal, I know there was some talk of that but I don't know if it actually became a thing or not.

That would still require more tight integration with package:test as well though.

@jakemac53 jakemac53 added the closed-not-planned Closed as we don't intend to take action on the reported issue label Sep 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-testing closed-not-planned Closed as we don't intend to take action on the reported issue P3 A lower priority bug or feature request package:build_runner type-enhancement A request for a change that isn't a bug type-performance
Projects
None yet
Development

No branches or pull requests

2 participants