Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

[meta] Automate screenshots #980

Closed
6 of 8 tasks
pocmo opened this issue Jul 19, 2017 · 8 comments
Closed
6 of 8 tasks

[meta] Automate screenshots #980

pocmo opened this issue Jul 19, 2017 · 8 comments
Labels
addressed build testing meta label for all features/bugs requiring test

Comments

@pocmo
Copy link
Contributor

pocmo commented Jul 19, 2017

As soon as we auto-import translations (#929) it would make sense automating the screenshot process too.

The iOS team wants this too - so let's join forces!

Tasks

  • Proof of concept
  • Modify docker image to support taking screenshots (emulator, fastlane, image tools)
  • Test execution on taskcluster
  • Run complete screenshot test suite
  • Resize and minify screenshots
  • Stabelize UI test
  • Split task into multiple tasks (decision task)
  • Upload screenshots
@pocmo pocmo added build needs triage testing meta label for all features/bugs requiring test labels Jul 19, 2017
@pocmo
Copy link
Contributor Author

pocmo commented Jul 19, 2017

@npark-mozilla FYI! You probably know best what we would need for that. :)

@nojunpark
Copy link

nojunpark commented Jul 19, 2017

Hmm, on Android, it is matter of running series of following commands:
export LC_ALL="en_US.UTF-8"
./gradlew assembleDebug assembleAndroidTest
fastlane screengrab run (need to install fastlane and screengrab beforehand)

BUT, this entails that we run it on an actual device, and BB real devices seem pretty finicky... we'll find out
maybe we can put this on buddybuild_postbuild.sh or something like that? (depends on how we automate l10n import)

@pocmo
Copy link
Contributor Author

pocmo commented Jul 20, 2017

The iOS team was thinking about planting a machine in some office. Maybe we could just attach an Android phone to it? But from past experience: This often has issues for random reasons. :)

@pocmo pocmo added v3.0 and removed needs triage labels Jul 25, 2017
@bbinto bbinto modified the milestone: Cobalt Jul 26, 2017
@bbinto bbinto removed the Cobalt label Aug 1, 2017
@jenn-chaulk jenn-chaulk modified the milestone: Cobalt* Aug 22, 2017
@pocmo pocmo modified the milestones: Cobalt*, Denim Sep 6, 2017
@pocmo
Copy link
Contributor Author

pocmo commented Sep 15, 2017

pocmo added a commit to pocmo/focus-android that referenced this issue Sep 20, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 20, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 20, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 20, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 20, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 20, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 22, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
pocmo added a commit that referenced this issue Sep 25, 2017
pocmo added a commit that referenced this issue Sep 25, 2017
pocmo added a commit to pocmo/focus-android that referenced this issue Sep 25, 2017
@pocmo pocmo mentioned this issue Sep 28, 2017
6 tasks
@pocmo pocmo added the ENG | L label Oct 6, 2017
@jenn-chaulk jenn-chaulk changed the title Automate screenshots [meta] Automate screenshots Oct 18, 2017
@ekager
Copy link
Contributor

ekager commented Oct 2, 2018

Can we close this meta?

@nojunpark
Copy link

This was complicated by having older sim running on taskcluster and having intermittent failures - it's not done, but it seems unclear that we'll be working on this. @pocmo , thoughts?

@pocmo
Copy link
Contributor Author

pocmo commented Oct 2, 2018

I talked to the L10N team some time ago and it is likely that they are going to own the screenshot automation for our apps.

I definitely won't be able to work on that.

@nojunpark
Copy link

Okay let's close this issue then.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
addressed build testing meta label for all features/bugs requiring test
Projects
None yet
Development

No branches or pull requests

5 participants