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
Jkmarx/connect add file service #2838
Conversation
…ry-platform into jkmarx/data-file-replace-modal
…ry-platform into jkmarx/data-file-replace-modal
…ry-platform into jkmarx/data-file-replace-modal
…ry-platform into jkmarx/data-file-replace-modal
Codecov Report
@@ Coverage Diff @@
## develop #2838 +/- ##
===========================================
- Coverage 58.29% 58.21% -0.08%
===========================================
Files 430 430
Lines 26837 26700 -137
Branches 1292 1262 -30
===========================================
- Hits 15644 15544 -100
+ Misses 11193 11156 -37
Continue to review full report at Codecov.
|
…ry-platform into jkmarx-data-file-modal-unit-tests
…m/refinery-platform/refinery-platform into jkmarx/connect-add-file-service
refinery/data_set_manager/views.py
Outdated
def post(self, request): | ||
try: | ||
data_set = DataSet.objects.get(uuid=request.POST['data_set_uuid']) | ||
data_set = DataSet.objects.get(uuid=request | ||
.data.get('data_set_uuid')) | ||
except KeyError: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think this KeyError has the potential to be raised anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@scottx611x Yes, that got updated when I wrote some unit tests.
Ref #2700