Skip to content

Avoid duplicates#158

Open
AndreaCrotti wants to merge 5 commits intomasterfrom
avoid-duplicates
Open

Avoid duplicates#158
AndreaCrotti wants to merge 5 commits intomasterfrom
avoid-duplicates

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