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

Moving example/test_driver/cloud_firestore_behaviors.dart to test_driver/cloud_firestore_behaviors.dart #52

Merged
merged 4 commits into from
Mar 28, 2020

Conversation

suztomo
Copy link
Collaborator

@suztomo suztomo commented Mar 27, 2020

  • Moved example/test_driver/cloud_firestore_behaviors.dart to test_driver/cloud_firestore_behaviors.dart
    This test is for cloud_firestore_mocks, not for the example.
  • Copied example/ios to ios, and example/android to android. I believe this project has not used these destination directories. (I don't test with Android)
  • Updated pubspec.yaml to support driver tests.

@atn832 Does this relocation look good to you?

@suztomo suztomo requested a review from atn832 March 27, 2020 22:04
@atn832
Copy link
Owner

atn832 commented Mar 28, 2020

Looks like when I copied the example project, I accidentally submitted the .gradle folder even though I shouldn't have (https://github.com/atn832/cloud_firestore_mocks/tree/master/example/android vs https://github.com/FirebaseExtended/flutterfire/tree/master/packages/cloud_firestore/cloud_firestore/example/android).

Could you add .gradle/ to .gitignore and remove the .gradle folder from your PR?

@atn832
Copy link
Owner

atn832 commented Mar 28, 2020

I've filed #53 to remove it from example/android later.

Copy link
Owner

@atn832 atn832 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

@suztomo suztomo merged commit 7a40868 into atn832:master Mar 28, 2020
@suztomo suztomo deleted the driver_test branch March 28, 2020 01:57
@suztomo
Copy link
Collaborator Author

suztomo commented Mar 28, 2020

Not touching android file removal.

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.

2 participants