tree d7f11a24602e758cf3cd19b30404b6536ac29776
parent 3a9c6217dfe91076f51a58df5be34ccb72e1097d
author srawlins <srawlins@google.com> 1612379841 -0500
committer Sam Rawlins <sam.rawlins@gmail.com> 1612392426 -0800

Allow real method calls for unstubbed methods if they return void or Future<void>.

This requires a breaking change, changing Mock.noSuchMethod's optional second parameter to two optional named parameters. Any code which calls noSuchMethod with a second positional parameter, or which overrides Mock.noSuchMethod, will need to change.

Users have found the throw-if-unstubbed setting to be too constrained for setters, other methods that return void, and methods which return a (non-nullable) Future<void>. The method calls should be allowed, as stubbing is probably meaningless.

Fixes https://github.com/dart-lang/mockito/issues/305

PiperOrigin-RevId: 355438518