If were making these changes, can we also get the pubspec updated to >= 3.16.0 just to make sure we don't run into other issues later on?

Can we merge this? I can't update project Flutter version because of this.

Thanks a lot!

Can we merge this? I can't update project Flutter version because of this.

Thanks a lot!

You can temporarily override your dependency:

  mockingjay:
    git: https://github.com/Tregan/mockingjay

Is there a particular reason other than convenience that canPop is stubbed by default?

I'd rather have an error during my test telling me I haven't mocked that method, rather than looking for a default that's hard to find.

Other than that, I'm okay with the changes. Thanks for the contribution!

Is there a particular reason other than convenience that canPop is stubbed by default?

I'd rather have an error during my test telling me I haven't mocked that method, rather than looking for a default that's hard to find.

Other than that, I'm okay with the changes. Thanks for the contribution!

Nope, just for convenience since it will have to be stubbed for every MockNavigator. I'll take it out and update the tests.
Maybe the docs hould be updated as well?

Sounds good. If that's fixed then I think we're good to go on this one.