tree 9be4d1b606aba138ae2c54a0ee3758a4265d1539
parent 36e88209ca1d2fd869bbfe1de398145c66b8847a
author srawlins <srawlins@google.com> 1591302323 -0400
committer Sam Rawlins <sam.rawlins@gmail.com> 1594309593 -0700

MockBuilder: Fix a few nullability bugs.

Methods should not be overridden unless they have a non-nullable return type, or at least one non-nullable parameter type. Previously, there was a complicated, pre-non-nullability logic to determine this for methods. This has been fixed to simply ask the type system.

Type variables are now correctly suffixed with a `?` in order to match the source type annotation; the _typeReference logic now handles TypeParameterTypes.

Stop using NullabilitySuffix; the TypeSystem is just a better way to handle question marks. dynamic can be handled separately.

PiperOrigin-RevId: 314792190