Update to match style guide for params for analysis view set

refinery/core/test_views.py

@@ -1683,22 +1683,22 @@ def test_get_analysis_returns_offset_analyses_for_user(self):
 
     def test_get_analysis_with_data_set_uuid_returns_401(self):
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         get_response = self.view(get_request_with_ds)
         self.assertEqual(get_response.status_code, 401)
 
     def test_get_analysis_with_invalid_data_set_uuid_returns_404(self):
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': 'xxx5'}
+            self.url_root, {'data_set_uuid': 'xxx5'}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
         self.assertEqual(get_response.status_code, 404)
 
     def test_get_analysis_with_data_set_uuid_returns_analyses(self):
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1710,7 +1710,7 @@ def test_get_analysis_with_data_set_uuid_returns_analyses(self):
     def test_get_analysis_with_data_set_uuid_returns_names_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1719,7 +1719,7 @@ def test_get_analysis_with_data_set_uuid_returns_names_field(self):
     def test_get_analysis_with_data_set_uuid_returns_status_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1729,7 +1729,7 @@ def test_get_analysis_with_data_set_uuid_returns_status_field(self):
     def test_get_analysis_with_data_set_uuid_returns_summary_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1739,7 +1739,7 @@ def test_get_analysis_with_data_set_uuid_returns_summary_field(self):
     def test_get_analysis_with_data_set_uuid_returns_time_start_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1749,7 +1749,7 @@ def test_get_analysis_with_data_set_uuid_returns_time_start_field(self):
     def test_get_analysis_with_data_set_uuid_returns_time_end_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1759,7 +1759,7 @@ def test_get_analysis_with_data_set_uuid_returns_time_end_field(self):
     def test_get_analysis_with_data_set_uuid_returns_uuid_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1768,7 +1768,7 @@ def test_get_analysis_with_data_set_uuid_returns_uuid_field(self):
     def test_get_analysis_with_data_set_uuid_returns_workflow_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1778,7 +1778,7 @@ def test_get_analysis_with_data_set_uuid_returns_workflow_field(self):
     def test_get_analysis_with_data_set_uuid_returns_owner_field(self):
         self.analysis2.delete()
         get_request_with_ds = self.factory.get(
-            self.url_root, {'dataSetUuid': self.data_set.uuid}
+            self.url_root, {'data_set_uuid': self.data_set.uuid}
         )
         force_authenticate(get_request_with_ds, user=self.user)
         get_response = self.view(get_request_with_ds)
@@ -1789,7 +1789,7 @@ def test_get_analysis_with_data_set_uuid_returns_owner_field(self):
 
     def test_get_analysis_with_data_set_uuid_returns_sorted_analyses(self):
         get_request = self.factory.get(self.url_root,
-                                       {'dataSetUuid': self.data_set.uuid})
+                                       {'data_set_uuid': self.data_set.uuid})
         force_authenticate(get_request, user=self.user)
         get_response = self.view(get_request)
         # sorted in reverse time_start
@@ -1801,7 +1801,7 @@ def test_get_analysis_with_data_set_uuid_returns_paged_analyses(self):
         get_request = self.factory.get(self.url_root,
                                        {'limit': limit,
                                         'offset': 0,
-                                        'dataSetUuid': self.data_set.uuid})
+                                        'data_set_uuid': self.data_set.uuid})
         force_authenticate(get_request, user=self.user)
         get_response = self.view(get_request)
         self.assertEqual(len(get_response.data), limit)
@@ -1810,7 +1810,7 @@ def test_get_analysis_with_data_set_uuid_returns_offset_analyses(self):
         offset = 1
         get_request = self.factory.get(self.url_root,
                                        {'offset': offset,
-                                        'dataSetUuid': self.data_set.uuid})
+                                        'data_set_uuid': self.data_set.uuid})
         force_authenticate(get_request, user=self.user)
         get_response = self.view(get_request)
         # know the total length off analysis is 2

refinery/core/views.py

@@ -838,7 +838,7 @@ class AnalysisViewSet(APIView):
         get:
             description: Returns analyses filtered by either data set or user
             parameters:
-                - name: dataSetUuid
+                - name: data_set_uuid
                   description: param to have analyses filtered by a data set
                   paramType: param
                   type: string
@@ -848,7 +848,7 @@ class AnalysisViewSet(APIView):
     http_method_names = ['get', 'delete']
 
     def get(self, request):
-        data_set_uuid = request.query_params.get('dataSetUuid')
+        data_set_uuid = request.query_params.get('data_set_uuid')
         paginator = LimitOffsetPagination()
         paginator.default_limit = 100
 

refinery/ui/source/js/analysis-monitor/ctrls/ctrl.js

@@ -34,7 +34,7 @@ function AnalysisMonitorCtrl (
   // analysis list and refreshes details for running analyses.
   vm.updateAnalysesList = function () {
     var param = {
-      dataSetUuid: $window.dataSetUuid
+      data_set_uuid: $window.dataSetUuid
     };
 
     vm.timerList = $timeout(vm.updateAnalysesList, 15000);

refinery/ui/source/js/analysis-monitor/services/factory.js

@@ -108,7 +108,7 @@ function analysisMonitorFactory (
 
   // Copies and sorts analyses list
   var processAnalysesList = function (data, params) {
-    if ('dataSetUuid' in params) {
+    if ('data_set_uuid' in params) {
       addElapseAndHumanTime(data);
     } else {
       angular.copy(data, analysesGlobalList);

refinery/ui/source/js/commons/services/analyses.spec.js

@@ -4,7 +4,7 @@ describe('Common.service.analysis: unit tests', function () {
   var $httpBackend;
   var $rootScope;
   var fakeUuid = 'x508x83x-x9xx-4740-x9x7-x7x0x631280x';
-  var params = '/?dataSetUuid=' + fakeUuid;
+  var params = '/?data_set_uuid=' + fakeUuid;
   var service;
   var fakeResponse = {
     meta: {
@@ -55,7 +55,7 @@ describe('Common.service.analysis: unit tests', function () {
     it('should return a resolving promise', function () {
       var results;
       var promise = service.get({
-        dataSetUuid: fakeUuid
+        data_set_uuid: fakeUuid
       }).$promise.then(function (response) {
         results = response;
       });

refinery/ui/source/js/provvis/ctrls/ctrl.js

@@ -75,7 +75,7 @@
     // Ajax calls, grabs the analysis & files promises for a particular data set
     function getData () {
       var analysisParams = {
-        dataSetUuid: _dataSetUuid
+        data_set_uuid: _dataSetUuid
       };
 
       var filesParams = {