Prevent repetition of emitted code when mocking extended/implemented interfaces

[typescript-tdd]

Prevent repetition of emitted code when mocking extended/implemented interfaces

[uittorio]

To implement this feature we would have to merge two different mocks.
Any merge functionality will have to check if a field exist into an object. This will cause trigger getters of our internal mock. If the getter is a reference to a mock that has recursion it will cause Maximum call stack.

Feel free to close the issue if you are happy with the conclusion

Example

interface A {
 mySelf: A
}

interface B extends A {
   bProperty: string;
}

repository.register('A', (function(){
  var __mySelf;
  return {
      get mySelf() { return __mySelf || (__mySelf = repository.get('A')) },
      set mySelf(_mySelf) { mySelf= _mySelf}
    },
  );
}))

repository.register('B', (function(){
  var __bProperty;
  var mock;

  mock = {
      get bProperty() { return __bProperty || (__bProperty = '') },
      set bProperty(_bProperty) { bProperty = _bProperty }
  };
  // this will cause the error because property mySelf of A is recursive
  return  mergeFunctionality(mock, repository.get('A')) 
  );
}))

[Pmyl]

I have another idea to solve this that won't trigger the getters, let's keep this open at low priority right now

[Pmyl]

@uittorio I still have an idea on how to do it but I think it's not worth it from every angle.
Not worth it because other features are definitely more important.
Not worth it because it doesn't really improve performance.
Not worth it because it will probably affect the performance negatively (in my solution I would trade lines of code for run time).

I'm closing this, if we actually see people having performance issues with this then we can reopen it.