Skip to content

Fix unit tests by calling TestWidgetsFlutterBinding.ensureInitialized() #11

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
Merged
merged 3 commits into from
Aug 23, 2019
Merged

Fix unit tests by calling TestWidgetsFlutterBinding.ensureInitialized() #11

merged 3 commits into from
Aug 23, 2019

Conversation

collinjackson
Copy link
Contributor

@collinjackson collinjackson commented Aug 22, 2019 •
edited
Loading

Fixes tests that were broken by flutter/flutter#38464

See also flutter/flutter#39077

@collinjackson collinjackson requested a review from kroikie August 23, 2019 12:55
@collinjackson collinjackson mentioned this pull request Aug 23, 2019
Merged
mehmetf
mehmetf reviewed Aug 23, 2019
View reviewed changes
@collinjackson
Copy link
Contributor Author

I'll land this one first and then land #3 afterwards and then publish all the plugins.

@mehmetf
Copy link
Contributor

mehmetf commented Aug 23, 2019

LGTM

@collinjackson collinjackson merged commit 3630aa9 into firebase:master Aug 23, 2019
@collinjackson collinjackson deleted the fix_tests branch August 23, 2019 21:33
@collinjackson collinjackson mentioned this pull request Aug 23, 2019
Closed
@urbanjagodic urbanjagodic mentioned this pull request Sep 9, 2019
Closed
@YonatanAmitzur YonatanAmitzur mentioned this pull request Sep 16, 2019
Closed
@thai2805 thai2805 mentioned this pull request Sep 25, 2019
Closed
@caseyryan caseyryan mentioned this pull request Oct 2, 2019
Closed
@pinkeshAubergine pinkeshAubergine mentioned this pull request Oct 9, 2019
Closed
This was referenced Oct 13, 2019
Closed
Closed
@exilonX exilonX mentioned this pull request Oct 16, 2019
Closed
@kroikie kroikie mentioned this pull request Oct 21, 2019
Closed
@agnoam agnoam mentioned this pull request Oct 23, 2019
Closed
@Parmendrakumar Parmendrakumar mentioned this pull request Nov 5, 2019
Closed
@PrismaSoftGC PrismaSoftGC mentioned this pull request Nov 5, 2019
Closed
@IvanOnishchenko IvanOnishchenko mentioned this pull request Nov 6, 2019
Closed
@habtish1 habtish1 mentioned this pull request Nov 7, 2019
Closed
@Sonderman Sonderman mentioned this pull request Nov 18, 2019
Closed
@amitverma-cf amitverma-cf mentioned this pull request Nov 18, 2019
Closed
@boylenssen boylenssen mentioned this pull request Nov 20, 2019
Closed
@fabianogadenz fabianogadenz mentioned this pull request Nov 20, 2019
Closed
@shojaeddin shojaeddin mentioned this pull request Apr 3, 2020
Closed
@YohanWadia YohanWadia mentioned this pull request Apr 3, 2020
Closed
@aliheader aliheader mentioned this pull request Apr 3, 2020
Closed
@adworzynski adworzynski mentioned this pull request Apr 15, 2020
Closed
@vineetpandit90 vineetpandit90 mentioned this pull request Apr 17, 2020
Closed
@ishtms ishtms mentioned this pull request Apr 17, 2020
Closed
@HEMANT96 HEMANT96 mentioned this pull request Apr 22, 2020
Closed
@zameerhossain zameerhossain mentioned this pull request Apr 28, 2020
Closed
@YashMobileFirst YashMobileFirst mentioned this pull request Apr 28, 2020
Closed
@navigui navigui mentioned this pull request Apr 30, 2020
Closed
@besidesordinary besidesordinary mentioned this pull request May 5, 2020
Closed
@sunilcoder36 sunilcoder36 mentioned this pull request May 6, 2020
Closed
@Niccolohui Niccolohui mentioned this pull request May 8, 2020
Closed
@clus90 clus90 mentioned this pull request May 8, 2020
Closed
@ADTheJava ADTheJava mentioned this pull request May 11, 2020
Closed
@hashika2 hashika2 mentioned this pull request May 22, 2020
Closed
@DyaryRaoof DyaryRaoof mentioned this pull request May 27, 2020
Closed
@momo1606 momo1606 mentioned this pull request Jun 2, 2020
Closed
@aislingorg aislingorg mentioned this pull request Jun 3, 2020
Closed
@MassiveSuccess MassiveSuccess mentioned this pull request Jun 7, 2020
Closed
@kartikpateldev kartikpateldev mentioned this pull request Jun 8, 2020
Closed
@bytejutsu bytejutsu mentioned this pull request Jun 12, 2020
Closed
@CarltonK CarltonK mentioned this pull request Jun 14, 2020
Closed
@braysonjohn148 braysonjohn148 mentioned this pull request Jun 26, 2020
Closed
@sumanthkumarps sumanthkumarps mentioned this pull request Jul 5, 2020
Closed
@lukaszciastko lukaszciastko mentioned this pull request Jul 11, 2020
Closed
@hitanshu1 hitanshu1 mentioned this pull request Aug 12, 2020
Closed
@giorodri0584 giorodri0584 mentioned this pull request Aug 12, 2020
Closed
@itssidhere itssidhere mentioned this pull request Aug 18, 2020
Closed
@firebase firebase locked and limited conversation to collaborators Aug 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kroikie kroikie Awaiting requested review from kroikie

1 more reviewer

@mehmetf mehmetf mehmetf approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@collinjackson @mehmetf