Add --cache-sksl flag to drive and run#42353
Conversation
|
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie. Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
jonahwilliams
left a comment
There was a problem hiding this comment.
This needs tests:
There should be some existing test coverage that asserts that command line arguments are turned into the correct adb/process command. I think that would be all that is needed in this case
So we can test SkSL precompile using the command line tools. See flutter/engine#12412.
|
Thank you @jonahwilliams for the quick comment! Can you please give me a link to the existing test that I can reference? |
|
Hmm, I was sure what existing test cases that cover these. At any rate, what you'll likely want are two tests. One for iOS in https://github.com/flutter/flutter/blob/master/packages/flutter_tools/test/general.shard/ios/devices_test.dart which calls startApp and ensures that the command line args are correctly set. Specifically, something like https://github.com/flutter/flutter/blob/master/packages/flutter_tools/test/general.shard/ios/devices_test.dart#L172 but with assertions on the command line arguments. Another one for Android in https://github.com/flutter/flutter/blob/master/packages/flutter_tools/test/general.shard/android/android_device_test.dart . Unfortunately we don't have an existing one, but it would look much like the iOS tests. To run a single test in the tools repo, you can run |
Codecov Report
@@ Coverage Diff @@
## master #42353 +/- ##
==========================================
- Coverage 59.94% 59.81% -0.13%
==========================================
Files 194 194
Lines 18871 18997 +126
==========================================
+ Hits 11313 11364 +51
- Misses 7558 7633 +75
Continue to review full report at Codecov.
|
|
@jonahwilliams : tests added. |
8d4f837 to
a4fc249
Compare
|
@jonahwilliams : friendly ping... |
So we can test SkSL precompile using the command line tools. See flutter/engine#12412.
So we can test SkSL precompile using the command line tools.
See flutter/engine#12412.