Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add @mockable support #120

Merged
merged 3 commits into from Jul 18, 2020
Merged

Add @mockable support #120

merged 3 commits into from Jul 18, 2020

Conversation

timaliberdov
Copy link
Collaborator

Closes #28

@timaliberdov
Copy link
Collaborator Author

ZStream mocks generation should be fixed in zio/zio#3895, ZSink generation still doesn't work in zio's @mockable macro so I didn't implement it in the plugin

clause <- method.paramClauses.clauses
param <- clause.parameters
paramType <- param.paramType
presentation = presentationString(paramType.typeElement)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

presentationString is removed in 2020.2. Any other way to do this?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm.. I'll look for an alternative. Note to self: same for accessible macro

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I noticed it there too!

Thanks!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Borrowed rendering related code from 2020.1 and 2020.2 versions of Scala plugin 😅
We'll need some refactoring once 2020.2 is out of beta.

@hmemcpy hmemcpy merged commit cd1ab74 into zio:idea201.x Jul 18, 2020
@timaliberdov timaliberdov deleted the mockable-support branch September 11, 2020 21:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement @mockable support
2 participants