Override method with generic array argument generated with incorrect type about mockingbird HOT 4 CLOSED

Just found out that the minimal example doesn't even involve generics. The following example also produces a duplicate method in the generated code:

class Base {
    func method(withArg: UInt) {
    }
}


class Derived: Base {
    override func method(withArg: UInt) {
    }
}

from mockingbird.

Thanks for reporting, Paul! This regression in 0.12 should be fixed by #119. The original plan in the #118 issue was to wait until the upcoming release, but since this extends to declarations in class mocks we’ll push out a patch on Monday.

from mockingbird.

Thanks for the quick fix. I can confirm that the duplicate override method has been fixed when using the latest code in master.

However another issue has still remained with regards to overriding methods with generics.
Take for example this example code:

class Concrete {
}

class Base<T> {
    func method(vertices: [T]) {
    }
}

class Derived: Base<Concrete> {
}

It results in the following generated code:

public final class DerivedMock: Mocky.Derived, Mockingbird.Mock {

  ...

  // MARK: Mocked `method`(`vertices`: [T])

  public override func `method`(`vertices`: [T]) -> Void {
    let invocation: Mockingbird.Invocation = Mockingbird.Invocation(selectorName: "`method`(`vertices`: [T]) -> Void", arguments: [Mockingbird.ArgumentMatcher(`vertices`)], returnType: Swift.ObjectIdentifier((Void).self))
    mockingContext.didInvoke(invocation) { () -> Void in
      let implementation = stubbingContext.implementation(for: invocation)
      if let concreteImplementation = implementation as? ([T]) -> Void {
        concreteImplementation(`vertices`)
      } else if let concreteImplementation = implementation as? () -> Void {
        concreteImplementation()
      }
    }
  }
  ...
}

This results in the error Use of undeclared type 'T'. The expected generated code would use the Concrete type here.

Interestingly it does generate the correct code when the type parameter isn't an array

I've tested this on a246a83 (master)

from mockingbird.

Thanks for the update!

The latest issue is caused by the type specializer not working recursively on type declarations. There isn’t a great workaround for this right now, but I’ll look into it for 0.13.

from mockingbird.