Fix missing interface's fields and methods in builder

[markgravity]

Steps to Reproduce

abstract class DemoDelegate {
  String get title;
  void show();
}

abstract class DemoClass implements DemoDelegate {
  String get description;

  void close();
}

@GenerateMocks([DemoClass])
void main() {}

Expected

/// A class which mocks [DemoClass].
///
/// See the documentation for Mockito's code generation for more information.
class MockDemoClass extends _i1.Mock implements _i3.DemoClass {
  MockDemoClass() {
    _i1.throwOnMissingStub(this);
  }

  @override
  String get description =>
      (super.noSuchMethod(Invocation.getter(#description), returnValue: '')
          as String);
  @override
  String get title =>
      (super.noSuchMethod(Invocation.getter(#title), returnValue: '')
          as String);
  @override
  void close() => super.noSuchMethod(Invocation.method(#close, []),
      returnValueForMissingStub: null);
  @override
  void show() => super.noSuchMethod(Invocation.method(#show, []),
      returnValueForMissingStub: null);
}

Actual

/// A class which mocks [DemoClass].
///
/// See the documentation for Mockito's code generation for more information.
class MockDemoClass extends _i1.Mock implements _i3.DemoClass {
  MockDemoClass() {
    _i1.throwOnMissingStub(this);
  }

  @override
  String get description =>
      (super.noSuchMethod(Invocation.getter(#description), returnValue: '')
          as String);
  @override
  void close() => super.noSuchMethod(Invocation.method(#close, []),
      returnValueForMissingStub: null);
}

[srawlins]

This is an interesting case. Do you have a real use case like what you described? I honestly didn't know that a class like DemoClass was legal.

[markgravity]

This is an interesting case. Do you have a real use case like what you described? I honestly didn't know that a class like DemoClass was legal.

Yea. I faced it in my repo

My LandingViewDelegate is an abstract class that implements AlertViewMixinDelegate
So my expectation that MockLandingViewDelegate should contain fields and methods from AlertViewMixinDelegate

[srawlins]

Cool, it makes sense to me. Could you add some tests in auto_mocks_test? Maybe just one for methods and one for fields.

[markgravity]

Cool, it makes sense to me. Could you add some tests in auto_mocks_test? Maybe just one for methods and one for fields.

Sure. Let's me update it now 👍

[markgravity]

@srawlins I added 2 tests into auto_mocks_test 👌

[srawlins]

Apologies for letting this languish; can you reformat auto_mocks_test, as CI complains about it?

[markgravity]

Apologies for letting this languish; can you reformat auto_mocks_test, as CI complains about it?

Done it! 👌

[srawlins]

Thanks much!