Merge pull request #1047 from josenavas/drop-remove-category

Removing the remove_category function

Author: antgonza

qiita_db/metadata_template/sample_template.py

@@ -481,32 +481,6 @@ def update(self, md_template):
                     if '_qiime_' not in basename(fp):
                         pt.create_qiime_mapping_file(fp)
 
-    def remove_category(self, category):
-        """Remove a category from the sample template
-
-        Parameters
-        ----------
-        category : str
-            The category to remove
-
-        Raises
-        ------
-        QiitaDBColumnError
-            If the column does not exist in the table
-        """
-        table_name = self._table_name(self.study_id)
-        conn_handler = SQLConnectionHandler()
-
-        if category not in self.categories():
-            raise QiitaDBColumnError("Column %s does not exist in %s" %
-                                     (category, table_name))
-
-        # This operation may invalidate another user's perspective on the
-        # table
-        conn_handler.execute("""
-            ALTER TABLE qiita.{0} DROP COLUMN {1}""".format(table_name,
-                                                            category))
-
     def update_category(self, category, samples_and_values):
         """Update an existing column
 

qiita_db/metadata_template/test/test_sample_template.py

@@ -1237,18 +1237,6 @@ def test_add_category(self):
         obs = {k: v['new_column'] for k, v in self.tester.items()}
         self.assertEqual(obs, exp)
 
-    def test_remove_category(self):
-        with self.assertRaises(QiitaDBColumnError):
-            self.tester.remove_category('does not exist')
-
-        for v in self.tester.values():
-            self.assertIn('elevation', v)
-
-        self.tester.remove_category('elevation')
-
-        for v in self.tester.values():
-            self.assertNotIn('elevation', v)
-
     def test_to_file(self):
         """to file writes a tab delimited file with all the metadata"""
         fd, fp = mkstemp()