-
-
Notifications
You must be signed in to change notification settings - Fork 201
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
Issue #330 + CRUD operations for Comments #344
Conversation
rwjdk
commented
Jun 22, 2018
- Added support for issue Can you add ability to get the post comments ? #330
- Added support for CRUD operations of Comments in Articles and Posts
- Added support for CRUD operations of Comments on Articles and Posts
Still learning Git (day to day I'm using TFS) and this whole Pull Request thing so sorry if this was done incorrectly (Looks like previous merge is also in this pull request :-( ) |
CodeFactor found an issue: The code must not contain multiple blank lines in a row. It's currently on: |
CodeFactor found an issue: A C# document may only contain a single class at the root level unless all of the classes are partial and are of the same type. It's currently on: |
[OneTimeSetUp] | ||
public void OneTimeSetUp() | ||
{ | ||
DeleteAllArticlesFromTestSection(); //Get rid of all articles in the test-section (there should be none but there could be if previous tests have failed) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the test cleanup.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code looks good
public class InitializeTests | ||
{ | ||
[OneTimeSetUp] | ||
public void OneTimeSetUp() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for adding this.
Love the work thanks for so it. |
* Test "CanCreateTicketWithDueDate" can't run on PCs that use dd/MM/yyyy as time format * Added Code to make tests work * - Added support for issue Speedygeek#330 - Added support for CRUD operations of Comments on Articles and Posts