[go: nahoru, domu]

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

Clarify test package use on 'An introduction to unit testing' page #9931

Open
1 task
eliasyishak opened this issue Dec 15, 2023 · 0 comments
Open
1 task
Labels
a.cookbook Relates to a cookbook recipe or guide e1-hours Effort: < 8 hrs from.page-issue Reported in a reader-filed concern p2-medium Necessary but not urgent concern. Resolve when possible. st.triage.ltw Indicates Lead Tech Writer has triaged test.unit Relates to unit testing

Comments

@eliasyishak
Copy link
Contributor

Page URL

https://docs.flutter.dev/cookbook/testing/unit/introduction/

Page source

https://github.com/flutter/website/tree/main/src/cookbook/testing/unit/introduction.md

Describe the problem

Per this issue, we are now recommending users that running flutter test to only use the flutter_test dependency in their pubspec.yaml file. However, in this link, for unit tests, it gives the user the option to either use the test or flutter_test depedency.

Expected fix

We should only recommend to use the flutter_test dependency to ensure that we don't have any version solving issues for the flutter test command specifically.

Additional context

Because flutter_test depends on pinned versions of test_api, we can ensure that we won't have any version mismatches between the app pubspec and the flutter tool pubspec. More details can be found in the below issue:

I also have a PR to ensure we check for this dependency in the app pubspec:

I would like to fix this problem.

  • I will try and fix this problem on docs.flutter.dev.
@dam-ease dam-ease added st.triage.triage-team Triage team reviewing and categorizing the issue a.cookbook Relates to a cookbook recipe or guide and removed st.triage.triage-team Triage team reviewing and categorizing the issue labels Dec 18, 2023
@dam-ease dam-ease modified the milestone: commit sug Dec 18, 2023
@dam-ease dam-ease added p2-medium Necessary but not urgent concern. Resolve when possible. e1-hours Effort: < 8 hrs from.page-issue Reported in a reader-filed concern test.unit Relates to unit testing labels Dec 18, 2023
@atsansone atsansone added the st.triage.ltw Indicates Lead Tech Writer has triaged label Dec 18, 2023
@atsansone atsansone changed the title [PAGE ISSUE]: 'An introduction to unit testing' Clarify test package use on 'An introduction to unit testing' page Dec 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a.cookbook Relates to a cookbook recipe or guide e1-hours Effort: < 8 hrs from.page-issue Reported in a reader-filed concern p2-medium Necessary but not urgent concern. Resolve when possible. st.triage.ltw Indicates Lead Tech Writer has triaged test.unit Relates to unit testing
Projects
None yet
Development

No branches or pull requests

3 participants