Generate Screenshots Automation

TomerPacific commented 1 year ago

Fixes #80

Following the requested changes in the issue, there were a things that needed to happen in order for the feature to work:

A new folder called integration_test was created in the main project hierarchy
The integration_test dependency was added under dev_dependencies
The older screenshots_test.dart has been deleted
Integration tests use a WidgetTester, so the older logic had to be refactored
Since the older test relied on a timer with an id of 42, which does not exist, the logic has changed to rely on the current default timers which are running.
This means that the test selects the first running timer (UI Layout), stops it and then clicks on it to go to the editing screen

The test has been run on an emulated device several times and appears to be working.

A video of the test running:

giphy

12people commented 1 year ago

@TomerPacific There's one more thing you didn't address — see my latest comment in issue #80 .

I think there's a driver file missing, as described in step 1 at https://dev.to/mjablecnik/take-screenshot-during-flutter-integration-tests-435k . Without it, I don't know where to find the generated screenshots. But it's possible I wasn't looking in the right places.

TomerPacific commented 1 year ago

@12people - You're probably right. Let me check things and get back to you.

12people commented 1 year ago

Looks good to me!

I would probably print the error once it's caught in the driver, but that's a simple line that I can add later. :)

Thanks a lot for contributing this, hope it was a good experience for you. :)

TomerPacific commented 1 year ago

@12people - I didn't know if you would prefer to have the error or not so that is why I omitted it. Thanks for your patience and assistance 🙏

hamaluik / timecop