cr updates

pybossa/api/__init__.py

@@ -1049,14 +1049,11 @@ def project_clone(project_id=None, short_name=None):
 
     if not (payload and payload.get('input_data_class') and payload.get('output_data_class') \
         and payload.get('name') and payload.get('short_name')):
-        return abort(400)
+        return abort(400, 'Request is missing one or more of the required fields: name, short_name, input_data_class, output_data_class')
 
     if not payload.get('password'):
         payload['password'] = ""
 
-    project.input_data_class = project.info.get('data_classification', {}).get('input_data')
-    project.output_data_class = project.info.get('data_classification', {}).get('output_data')
-
     try:
         new_project = clone_project(project, payload)
         current_app.logger.info(

test/test_api/test_project_api.py

@@ -2654,7 +2654,7 @@ def test_clone_project_edge_cases(self):
 
 
     @with_context
-    def test_clone_project(self):
+    def test_clone_project_success(self):
         """Test API clone project success state"""
         from pybossa.view.projects import data_access_levels
 
@@ -2676,7 +2676,7 @@ def test_clone_project(self):
     @with_context
     @patch('pybossa.api.clone_project')
     def test_clone_project_error(self, clone_project):
-        """Test API clone project success state"""
+        """Test API clone project error when cloning project"""
         from pybossa.view.projects import data_access_levels
 
         clone_project.side_effect = Exception("Project clone error!")
@@ -2696,7 +2696,7 @@ def test_clone_project_error(self, clone_project):
 
 
     @with_context
-    def test_clone_project_by_id(self):
+    def test_clone_project_by_id_success(self):
         """Test API clone project by id success state"""
         from pybossa.view.projects import data_access_levels