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

Gallery App is built and tested for iOS and Android #1600

Closed
3 tasks done
sethladd opened this issue Feb 4, 2016 · 13 comments
Closed
3 tasks done

Gallery App is built and tested for iOS and Android #1600

sethladd opened this issue Feb 4, 2016 · 13 comments
Assignees
Labels
c: new feature Nothing broken; request for a new capability customer: gallery Relating to flutter/gallery repository. Please transfer non-framework issues there. f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.

Comments

@sethladd
Copy link
Contributor

sethladd commented Feb 4, 2016

Testing and verification can include:

  • Testing app on Nexus 5
  • Testing app on iPhone 6
  • Performance
    • Startup
    • Animations (TODO, don't have the infra for this quite yet)
    • Scrolling (grid, table, lists, calendar)
  • Correctness
    • smoke test for automated bots
    • Manual testing plan
@sethladd sethladd added c: new feature Nothing broken; request for a new capability f: material design flutter/packages/flutter/material repository. customer: gallery Relating to flutter/gallery repository. Please transfer non-framework issues there. labels Feb 4, 2016
@sethladd sethladd added this to the Gallery App milestone Feb 4, 2016
@eseidelGoogle
Copy link
Contributor

At time of writing, none of Flutter's framework supports anything other than phones in portrait orientation. Various widgets/layouts will need updating for tablet. @Hixie @krisgiesing

@eseidelGoogle
Copy link
Contributor

We build for both iOS and Android on the bots, we briefly ran flutter drive in the iOS simulator, but we definitely have more we could do here.

FYI @chinmaygarde @yjbanov

@yjbanov
Copy link
Contributor

yjbanov commented May 13, 2016

We now run the benchmarks for the gallery app regularly. The startup benchmark is already on the dashboard. @HansMuller wrote a benchmark for screen transitions. I'm adding it to the dashboard now.

Can we reduce the scope of the issue to Android phones only for now? Otherwise, to fix it one would find themselves shaving a herd of yaks in no time. Instead, I propose that as we expand our test device matrix to iOS and then tablets we just take all existing tests and benchmarks with us.

With a reduced scope I'd be willing to take this issue.

@eseidelGoogle
Copy link
Contributor

Sold. Lets dump the tablet bits for now.

@yjbanov yjbanov self-assigned this May 13, 2016
@yjbanov
Copy link
Contributor

yjbanov commented May 19, 2016

We now have the screen transition tests running and on the dashboard. The remaining parts are to make this all run on iOS. This is currently blocked by a number of other issues, so I'll put this on hold until we're unblocked.

Blockers:

@sethladd
Copy link
Contributor Author

Thanks for the links to the blockers! I've updated their milestones to indicate they are part of the Gallery milestone.

@chinmaygarde
Copy link
Member

I believe this is resolved once #4575 lands.

@yjbanov
Copy link
Contributor

yjbanov commented Jun 16, 2016

@chinmaygarde not quite. I still need to get the tests running on the dashboard box. But #4575 gets us very close.

@sethladd sethladd modified the milestones: August, Gallery App Jul 21, 2016
@sethladd
Copy link
Contributor Author

Any updates on testing on real iOS hardware?

@yjbanov
Copy link
Contributor

yjbanov commented Sep 9, 2016

We now build and test on both Android and iOS. I even had iOS briefly passing until we regressed again. Not sure if we should mark it as fixed and open a separate issue to fix the iOS build, or wait until our iOS tests are green and mark this issue as fixed.

@sethladd
Copy link
Contributor Author

sethladd commented Sep 9, 2016

Don't think we should close this yet. We aren't able to verify gallery builds via our tests on iOS hardware yet, which is the intent of this issue.

@sethladd sethladd modified the milestones: August, Top Customer Requests Sep 12, 2016
@Hixie Hixie added the framework flutter/packages/flutter repository. See also f: labels. label Sep 12, 2016
@yjbanov
Copy link
Contributor

yjbanov commented Nov 29, 2016

Closing this. Android is all done, and we are running some of iOS. I created multiple bite-sized issues in the Critical Metrics project to have iOS catch up:

#7074 - gallery perf/smoke test
#7075 - gallery start-up test
#7076 - scroll test

@yjbanov yjbanov closed this as completed Nov 29, 2016
TahaTesser pushed a commit to NevercodeHQ/flutter that referenced this issue Aug 13, 2020
@github-actions
Copy link

github-actions bot commented Sep 5, 2021

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
c: new feature Nothing broken; request for a new capability customer: gallery Relating to flutter/gallery repository. Please transfer non-framework issues there. f: material design flutter/packages/flutter/material repository. framework flutter/packages/flutter repository. See also f: labels.
Projects
None yet
Development

No branches or pull requests

5 participants