arXiv / arxiv-submission-ui

User interface of NG submit system.
MIT License
2 stars 6 forks source link

ARXIVNG-992 Implementation of integration with file management service #21

Closed erickpeirson closed 6 years ago

erickpeirson commented 6 years ago

This is based on https://github.com/cul-it/arxiv-filemanager/blob/b809d22a83510e9698cb4191978b6b4c3c134531/schema/openapi.yaml

I wrote a few tests with a mock file management app, but didn't want to get too far down that road until response structure is known. At this point, it probably makes sense to review this for correctness (consistency with the OpenAPI spec), and then build out error handling/tests further when the content spec is known.

erickpeirson commented 6 years ago

Changed base to task/ARXIVNG-892, since this branch was forked from there.

coveralls commented 6 years ago

Pull Request Test Coverage Report for Build 199


Changes Missing Coverage Covered Lines Changed/Added Lines %
submit/services/filemanager.py 67 89 75.28%
<!-- Total: 67 89 75.28% -->
Totals Coverage Status
Change from base Build 196: 0.06%
Covered Lines: 649
Relevant Lines: 868

💛 - Coveralls
erickpeirson commented 6 years ago

Will delete workspace be available to submitter and act like 'release'? or is this actual delete intended for admins? Maybe this logic is handled elsewhere. I'm wondering if admin commands should be organized together [delete,lock,unlock].

Ah, good catch. I was working through the OpenAPI spec, and not thinking about how these aren't desired in the UI. Will remove.

erickpeirson commented 6 years ago

I believe delete file request should take the relative path from root of submission workspace. I guess file_path argument is slightly misleading though description as key is accurate.

Agreed. Will try to make this clearer in the docstring.