• Andrew Kolos's avatar
    Fix `AssetsEntry::equals` (#143355) · 14bcc694
    Andrew Kolos authored
    In service of https://github.com/flutter/flutter/issues/143348.
    
    **Issue.** The `equals` implementation of `AssetsEntry` is incorrect. It compares `flavors` lists using reference equality. This PR addresses this.
    
    This also adds a test to make sure valid asset `flavors` declarations are parsed correctly.
    
    While we are here, this PR also includes a couple of refactorings:
      * `flutter_manifest_test.dart` is a bit large. To better match our style guide, I've factored out some related tests into their own file.
      *  A couple of changes to the `_validateListType` function in `flutter_manifest.dart`:
          * The function now returns a list of errors instead of accepting a list to append onto. This is more readable and also allows callers to know which errors were found by the call.
          * The function is renamed to `_validateList` and now accepts an `Object?` instead of an `YamlList`. If the argument is null, an appropriate error message is contained in the output. This saves callers that are only interested in validation from having to write their own null-check, which they all did before.
          * Some error strings were tweaked for increased readability and/or grammatical correctness.
    14bcc694
Name
Last commit
Last update
..
analyze_size.dart Loading commit data...
async_guard.dart Loading commit data...
bot_detector.dart Loading commit data...
build.dart Loading commit data...
command_help.dart Loading commit data...
common.dart Loading commit data...
config.dart Loading commit data...
context.dart Loading commit data...
dds.dart Loading commit data...
deferred_component.dart Loading commit data...
error_handling_io.dart Loading commit data...
file_system.dart Loading commit data...
fingerprint.dart Loading commit data...
io.dart Loading commit data...
logger.dart Loading commit data...
multi_root_file_system.dart Loading commit data...
net.dart Loading commit data...
os.dart Loading commit data...
platform.dart Loading commit data...
process.dart Loading commit data...
project_migrator.dart Loading commit data...
signals.dart Loading commit data...
task_queue.dart Loading commit data...
template.dart Loading commit data...
terminal.dart Loading commit data...
time.dart Loading commit data...
user_messages.dart Loading commit data...
utils.dart Loading commit data...
version.dart Loading commit data...
version_range.dart Loading commit data...