nteract / bookstore

📚 Notebook storage and publishing workflows for the masses
https://bookstore.readthedocs.io
BSD 3-Clause "New" or "Revised" License
202 stars 23 forks source link

Add s3clone integration test #162

Closed mpacer closed 5 years ago

mpacer commented 5 years ago

As I was working on the fs cloning API handler, I realized that we didn't have an integration test for the s3 cloning API handler.

We will need to cleanup the files created by the the cloning APIs, because otherwise we'll create a new file with a new name with each POST request.

codecov[bot] commented 5 years ago

Codecov Report

Merging #162 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #162   +/-   ##
=======================================
  Coverage   81.15%   81.15%           
=======================================
  Files          10       10           
  Lines         467      467           
=======================================
  Hits          379      379           
  Misses         88       88
codecov[bot] commented 5 years ago

Codecov Report

Merging #162 into master will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #162   +/-   ##
=======================================
  Coverage   81.15%   81.15%           
=======================================
  Files          10       10           
  Lines         467      467           
=======================================
  Hits          379      379           
  Misses         88       88