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

[WIP] Added AutoValueAttribute to NUnit3 glue library #1012

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Kralizek
Copy link
Contributor

@Kralizek Kralizek commented Apr 3, 2018

  • Added relevant tests
  • Added dependency to Moq in AutoFixture.NUnit3.UnitTest

Related to #343

@Kralizek
Copy link
Contributor Author

Kralizek commented Apr 3, 2018

@zvirja I dared to make a PR even if we saw some things to be fixed in #343 just to keep the development happening in one place instead of gists and random scripts :)

[InlineAutoData(typeof(DoubleParameterType<int, string>))]
public void AutoValueAttribute_generates_a_model(Type typeToGenerate, AutoValueAttribute sut)
{
var mockParameter = new Mock<IParameterInfo>();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zvirja I added a dependency on Moq to quickly write this test. If you have better ways to solve this scenario, I'll make the proper changes.

I was too lazy to create a fake implementation of IParameterInfo.

@zvirja zvirja changed the title Added AutoValueAttribute to NUnit3 glue library [WIP] Added AutoValueAttribute to NUnit3 glue library Apr 6, 2018
@zvirja zvirja changed the title [WIP] Added AutoValueAttribute to NUnit3 glue library [WIP]Added AutoValueAttribute to NUnit3 glue library Apr 7, 2018
@zvirja zvirja changed the title [WIP]Added AutoValueAttribute to NUnit3 glue library [WIP] Added AutoValueAttribute to NUnit3 glue library Apr 7, 2018
@zvirja
Copy link
Member

zvirja commented Apr 7, 2018

Added the WIP header to indicate that PR should not be merged as it's in progress.

@Kralizek
Copy link
Contributor Author

@zvirja I haven't forgotten this. Just super busy, as of late.

- Added relevant tests
- Added dependency to Moq in AutoFixture.NUnit3.UnitTest
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.

None yet

2 participants