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

fix docs wrong code that causes to all test failed #2713

Merged
merged 1 commit into from
Feb 5, 2020

Conversation

yunusd
Copy link
Contributor

@yunusd yunusd commented Jan 25, 2020

No description provided.

@hikalkan
Copy link
Member

Thank you @yunusd for your contribution.

But, which tests fail, can you point?

@yunusd
Copy link
Contributor Author

yunusd commented Jan 27, 2020

Thank you @yunusd for your contribution.

But, which tests fail, can you point?

Current documentation is pointing that Book entity has Id prop while don't. Abviously this causing compile time error. When we try to seed Book data without Id in BookStoreTestDataSeedContributor.cs all tests failing. So test section of documentation can't be done with current docs.

Screenshot from 2020-01-27 10-35-33

@acjh
Copy link
Contributor

acjh commented Jan 27, 2020

Caused by 8f2de58 (0.20.0)

Follows 1b35606 (0.22.0) for Book, BookStoreTestDataSeedContributor
Follows c5cf2ae (1.0.0) for Book

@hikalkan
Copy link
Member

hikalkan commented Feb 5, 2020

Thank you @yunusd :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants