tree a32fa781e9fc45c0409bb4a4d2b315b7aab431c6
parent 4479e26172b997a4118599ca45742a7ef224e5b6
author stuartmorgan <stuartmorgan@google.com> 1652397219 -0400
committer GitHub <noreply@github.com> 1652397219 -0700
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJifZSjCRBK7hj4Ov3rIwAAI20IAK+iEgud/v9m4uQZaeC4xy0i
 o0sSSK511sJm+B9BsD9Hg/IrsukCCbN54Mul5Y7J9732LxzQnwzOBxJ2yCvU6IvH
 1ZDyAN82IhhlbVaXM3c0RUuIcs1ye94fiLCPr+PM3vBIGAZum5ajChP8z9RKOTkS
 84hkshVzfuMLMrqO8SbzNXOyOH6K5vhE6+gurW3dziq3xhfO/qufcY99LJVGzB9+
 d5sTqOvGpNP68LhceabTBMb/8z9WUi8rMCz0AWn4h7BDykFkrD/07geAKkJK4YiB
 wyv6mTQB+hRs6OST/+YVzGX3NVA6159f5RnraiK2/eVioqWpB6LF4itUSJyf9rU=
 =RhBo
 -----END PGP SIGNATURE-----
 

[image_picker] Switch unit tests to mock plaform implementation (#5706)

`image_picker`'s app-facing tests were never updated during federation to
use a mock platform implementation, and instead were still mocking method
channels. That makes them fragile to implementation details of the
default method channel implementation that is part of another package,
and thus subject to breakage when the method channel changes.

This converts them to using a mock platform implementation, so it's only
testing the layer within this package.

Removes some tests that were testing things that only made sense at the
method channel layer.

Adds argument assertions that there were tests for, but were previously
only enforced in the implementations. As these are API constraints, they
should be enforced at the API layer, not at each implementation's layer
as they currently are.