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

Avoid duplicates #158

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

Avoid duplicates #158

wants to merge 5 commits into from

Conversation

AndreaCrotti
Copy link
Owner

@AndreaCrotti AndreaCrotti commented Feb 19, 2019

Check the last game before inserting a game, and if it's a duplicate return 400

@AndreaCrotti AndreaCrotti added the WIP work in progress label Feb 19, 2019
@AndreaCrotti AndreaCrotti force-pushed the avoid-duplicates branch 2 times, most recently from 1ac1586 to f0af976 Compare February 19, 2019 22:20
@codecov
Copy link

codecov bot commented Feb 19, 2019

Codecov Report

Merging #158 into master will increase coverage by 0.63%.
The diff coverage is 96.55%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   77.46%   78.09%   +0.63%     
==========================================
  Files          25       25              
  Lines         710      735      +25     
  Branches       53       54       +1     
==========================================
+ Hits          550      574      +24     
  Misses        107      107              
- Partials       53       54       +1

@AndreaCrotti AndreaCrotti changed the title sample tests to prove how the backend should behave Avoid duplicates Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WIP work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant