search

NYU-DevOps-Spring2018-Orders / orders

Orders microservice
Apache License 2.0
1 stars 2 forks source link

Fix delete item bug #44

Closed andyd0 closed 6 years ago

andyd0 commented 6 years ago

Previously did not check order id even though item id is unique. Now there is a proper route that first validates that the order id matches what the item details has for order id

codecov[bot] commented 6 years ago

Codecov Report

Merging #44 into master will increase coverage by 0.69%. The diff coverage is 75%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #44      +/-   ##
==========================================
+ Coverage   95.72%   96.41%   +0.69%     
==========================================
  Files           2        2              
  Lines         304      307       +3     
==========================================
+ Hits          291      296       +5     
+ Misses         13       11       -2
Impacted Files Coverage Δ
server.py 93.85% <75%> (+1.24%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update e760533...41d7eae. Read the comment docs.