update error message on /classifications/users when querying for all …

…project contributions [ordered by desc count]. (no longer top projects) (#22)
zooniverse · Aug 1, 2023 · 45ec15b · 45ec15b
1 parent 16807bb
commit 45ec15b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/controllers/user_classification_count_controller.rb b/app/controllers/user_classification_count_controller.rb
@@ -16,7 +16,7 @@ def query
 
   def validate_params
     super
-    raise ValidationError, 'Cannot query top projects and query by project/workflow' if params[:project_contributions] && (params[:workflow_id] || params[:project_id])
+    raise ValidationError, 'Cannot query for project contributions and query by project/workflow' if params[:project_contributions] && (params[:workflow_id] || params[:project_id])
   end
 
   def sanitize_params

diff --git a/spec/controllers/user_classification_count_controller_spec.rb b/spec/controllers/user_classification_count_controller_spec.rb
@@ -99,13 +99,13 @@
       it 'ensures you cannot query by workflow and project_contributions' do
         get :query, params:  { id: 1, project_contributions: true, workflow_id: 1 }
         expect(response.status).to eq(400)
-        expect(response.body).to include('Cannot query top projects and query by project/workflow')
+        expect(response.body).to include('Cannot query for project contributions and query by project/workflow')
       end
 
       it 'ensures you cannot query by project and top project contributions' do
         get :query, params: { id: 1, project_contributions: true, project_id: 1 }
         expect(response.status).to eq(400)
-        expect(response.body).to include('Cannot query top projects and query by project/workflow')
+        expect(response.body).to include('Cannot query for project contributions and query by project/workflow')
       end
 
       it 'ensures project_contributions is an boolean' do