Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

Add flow test fixtures and tests #13986

Merged
merged 15 commits into from
Dec 3, 2019
Merged

Add flow test fixtures and tests #13986

merged 15 commits into from
Dec 3, 2019

Conversation

arbreng
Copy link
Contributor

@arbreng arbreng commented Nov 22, 2019

We use a MockCanvas to enable unit-testing the flow code without interacting with the GPU.

These tests compile and pass on all platforms, including Fuchsia.

Test: Ran workstation on Fuchsia; ran flow_unittests
Bug: https://bugs.fuchsia.dev/p/fuchsia/issues/list/23711
Bug: https://bugs.fuchsia.dev/p/fuchsia/issues/list/24163

Copy link
Member

@chinmaygarde chinmaygarde left a comment

Choose a reason for hiding this comment

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

Primary review in #13949.

@arbreng arbreng force-pushed the flow-unittests branch 8 times, most recently from 385e637 to e61a611 Compare November 26, 2019 03:18
Copy link
Contributor

@liyuqian liyuqian left a comment

Choose a reason for hiding this comment

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

Finally finished reviewing 1 commit :) Looks good! Left some nit comments.

@arbreng arbreng force-pushed the flow-unittests branch 5 times, most recently from adec981 to bfec613 Compare November 28, 2019 00:19
Copy link
Contributor

@liyuqian liyuqian left a comment

Choose a reason for hiding this comment

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

LGTM modulo two missing TODO()s, and a curious question about the ThreadHost change.

@arbreng arbreng force-pushed the flow-unittests branch 3 times, most recently from 5a2488d to 054632a Compare November 29, 2019 01:41
@arbreng arbreng added the waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land. label Dec 1, 2019
@fluttergithubbot fluttergithubbot merged commit 620f528 into master Dec 3, 2019
liyuqian added a commit to liyuqian/engine that referenced this pull request Dec 3, 2019
liyuqian added a commit that referenced this pull request Dec 3, 2019
* Revert "Add flow test fixtures and tests (#13986)"

This reverts commit 620f528.

* Revert "Dynamically determine whether to use offscreen surface based on need (#13976)"

This reverts commit a86ef94.
filmil pushed a commit to filmil/engine that referenced this pull request Mar 13, 2020
* Revert "Add flow test fixtures and tests (flutter#13986)"

This reverts commit 620f528.

* Revert "Dynamically determine whether to use offscreen surface based on need (flutter#13976)"

This reverts commit a86ef94.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
affects: engine cla: yes code health waiting for tree to go green This PR is approved and tested, but waiting for the tree to be green to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants