addressing @josenavas comments

Author: antgonza

qiita_db/study.py

@@ -1137,15 +1137,15 @@ def unshare(self, user):
             qdb.sql_connection.TRN.add(sql, [self._id, user.id])
             qdb.sql_connection.TRN.execute()
 
-    def add_tags(self, user, tags):
+    def updata_tags(self, user, tags):
         """Sets the tags of the study
 
         Parameters
         ----------
         user: User object
-            The user to unshare the study with
+            The user reqesting the study tags update
         tags : list of str
-            The tags to add to the study
+            The tags to update within the study
 
         Returns
         -------
@@ -1180,7 +1180,7 @@ def add_tags(self, user, tags):
                     if user_level != 'admin':
                         admin_tags = to_add & system_tags_admin
                         if admin_tags:
-                            message += ('Only admins can assing: '
+                            message += ('Only admins can assign: '
                                         '%s' % ', '.join(admin_tags))
                         to_add = to_add - admin_tags
 

qiita_db/test/test_study.py

@@ -855,19 +855,19 @@ def test_study_tags(self):
         # assigning the tags to study as user
         study = qdb.study.Study(1)
         tags = ['this is my tag', 'actual GOLD!']
-        message = study.add_tags(user, tags)
+        message = study.updata_tags(user, tags)
         self.assertItemsEqual(study.tags, tags[:1])
-        self.assertEqual(message, 'Only admins can assing: actual GOLD!')
+        self.assertEqual(message, 'Only admins can assign: actual GOLD!')
         # now like admin
-        message = study.add_tags(admin, tags)
+        message = study.updata_tags(admin, tags)
         self.assertItemsEqual(study.tags, tags)
         self.assertEqual(message, '')
 
         # cleaning tags
-        message = study.add_tags(user, [])
+        message = study.updata_tags(user, [])
         self.assertEqual(study.tags, ['actual GOLD!'])
         self.assertEqual(message, 'You cannot remove: actual GOLD!')
-        message = study.add_tags(admin, [])
+        message = study.updata_tags(admin, [])
         self.assertEqual(study.tags, [])
         self.assertEqual(message, '')
 

qiita_pet/handlers/api_proxy/studies.py

@@ -384,7 +384,7 @@ def study_tags_patch_request(user_id, study_id,
         study = Study(study_id)
 
         if attribute == 'tags':
-            message = study.add_tags(User(user_id), req_value)
+            message = study.updata_tags(User(user_id), req_value)
             return {'status': 'success',
                     'message': message}
         else: