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

Commit 39415c3

Browse files
authored
[Impeller] added an assert that goldens are generated (#50222)
fixes flutter/flutter#142657 ## Pre-launch Checklist - [x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs. - [x] I read the [Tree Hygiene] wiki page, which explains my responsibilities. - [x] I read and followed the [Flutter Style Guide] and the [C++, Objective-C, Java style guides]. - [x] I listed at least one issue that this PR fixes in the description above. - [x] I added new tests to check the change I am making or feature I am adding, or the PR is [test-exempt]. See [testing the engine] for instructions on writing and running engine tests. - [x] I updated/added relevant documentation (doc comments with `///`). - [x] I signed the [CLA]. - [x] All existing and new tests are passing. If you need help, consider asking for advice on the #hackers-new channel on [Discord]. <!-- Links --> [Contributor Guide]: https://github.com/flutter/flutter/wiki/Tree-hygiene#overview [Tree Hygiene]: https://github.com/flutter/flutter/wiki/Tree-hygiene [test-exempt]: https://github.com/flutter/flutter/wiki/Tree-hygiene#tests [Flutter Style Guide]: https://github.com/flutter/flutter/wiki/Style-guide-for-Flutter-repo [C++, Objective-C, Java style guides]: https://github.com/flutter/engine/blob/main/CONTRIBUTING.md#style [testing the engine]: https://github.com/flutter/flutter/wiki/Testing-the-engine [CLA]: https://cla.developers.google.com/ [flutter/tests]: https://github.com/flutter/tests [breaking change policy]: https://github.com/flutter/flutter/wiki/Tree-hygiene#handling-breaking-changes [Discord]: https://github.com/flutter/flutter/wiki/Chat
1 parent 8c43332 commit 39415c3

File tree

1 file changed

+12
-0
lines changed

1 file changed

+12
-0
lines changed

testing/run_tests.py

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,9 @@
4343

4444
ENCODING = 'UTF-8'
4545

46+
# This number must be updated when adding new golden tests to impeller.
47+
_NUM_EXPECTED_GENERATED_IMPELLER_GOLDEN_FILES = 566
48+
4649
logger = logging.getLogger(__name__)
4750
logger_handler = logging.StreamHandler()
4851

@@ -1115,6 +1118,15 @@ def run_impeller_golden_tests(build_dir: str):
11151118
)
11161119
with tempfile.TemporaryDirectory(prefix='impeller_golden') as temp_dir:
11171120
run_cmd([tests_path, '--working_dir=%s' % temp_dir], cwd=build_dir)
1121+
num_generated_files = len(os.listdir(temp_dir))
1122+
if num_generated_files != _NUM_EXPECTED_GENERATED_IMPELLER_GOLDEN_FILES:
1123+
raise Exception(
1124+
'`impeller_golden_tests` was expected to generate '
1125+
f'{_NUM_EXPECTED_GENERATED_IMPELLER_GOLDEN_FILES} files, '
1126+
f'{num_generated_files} were generated. If this is expected, update '
1127+
'_NUM_EXPECTED_GENERATED_IMPELLER_GOLDEN_FILES.'
1128+
)
1129+
11181130
with DirectoryChange(harvester_path):
11191131
run_cmd(['dart', 'pub', 'get'])
11201132
bin_path = Path('.').joinpath('bin'

0 commit comments

Comments
 (0)