Skip to content
This repository has been archived by the owner on Dec 11, 2018. It is now read-only.

Technical test for Schibsted #36

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

gdapice
Copy link

@gdapice gdapice commented Sep 2, 2018

I did not include the Repository tests because I think Unit Tests are not necessary there but we should use integration tests for this scenario.

New properties for tweet added, publicationDate, discarded, discardedDate
TweetRepository created for the SQL methods
TweetService refactored for extracting the SQL from it
Ability to add links in the tweet without affecting the lenght of it
/discarded endpoint added for discarding an existing tweet and for listing all discarded tweets
New tests added
No tests for TweetRepository
.getignore updated for excluding the out folder
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant