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

TestFixtures refactoring #37

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Conversation

SimonMarquis
Copy link
Owner

@SimonMarquis SimonMarquis commented Jun 22, 2024

Support for Kotlin TestFixtures on Android modules was added in AGP 8.5.0!

A lot more cleanup could be done, especially on test classes that redefine over and over again topics, newsResources, etc.

This is a new take on this PR:

To close this issue from 2022:


The main changes are split by commits:

  • AGP 8.5.0 update with enableTestFixturesKotlinSupport flag
  • :core:data-test, :sync:test and :core:datastore-test have been merged as testFixtures sourceSet of their respective module.
  • :core:testing content has been migrated when possible to already existing modules.
  • :sync:work has been move to it's parent :sync module that was previously empty.

  • Regenerate graphs

@SimonMarquis SimonMarquis force-pushed the test-fixtures-agp-8.5.0 branch 6 times, most recently from e9f6791 to bffa71d Compare June 23, 2024 09:42
@SimonMarquis SimonMarquis force-pushed the test-fixtures-agp-8.5.0 branch 2 times, most recently from 7066812 to 4f10c36 Compare June 23, 2024 11:52
@SimonMarquis SimonMarquis force-pushed the test-fixtures-agp-8.5.0 branch from 4f10c36 to 8f377a3 Compare June 23, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant