Skip to content

Commit 52505bc

Browse files
josenavasjosenavas
committed
Fixing prep template tests
1 parent da56fc5 commit 52505bc

File tree

1 file changed

+56
-50
lines changed

1 file changed

+56
-50
lines changed

‎qiita_db/metadata_template/test/test_prep_template.py

Lines changed: 56 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -582,39 +582,40 @@ def test_add_common_creation_steps_to_queue(self):
582582
'VALUES (%s, %s, %s)')
583583

584584
sql_create_table = (
585-
'CREATE TABLE qiita.prep_2 '
586-
'(sample_id varchar NOT NULL, str_column varchar, '
587-
'run_prefix varchar, barcodesequence varchar, platform varchar, '
588-
'linkerprimersequence varchar, '
589-
'experiment_design_description varchar, '
590-
'library_construction_protocol varchar)')
585+
'CREATE TABLE qiita.prep_2 (sample_id varchar NOT NULL, '
586+
'barcodesequence varchar, experiment_design_description varchar, '
587+
'library_construction_protocol varchar, '
588+
'linkerprimersequence varchar, platform varchar, '
589+
'run_prefix varchar, str_column varchar)')
591590

592591
sql_insert_dynamic = (
593592
'INSERT INTO qiita.prep_2 '
594-
'(sample_id, str_column, run_prefix, barcodesequence, platform, '
595-
'linkerprimersequence, experiment_design_description, '
596-
'library_construction_protocol) '
593+
'(sample_id, barcodesequence, experiment_design_description, '
594+
'library_construction_protocol, linkerprimersequence, platform, '
595+
'run_prefix, str_column) '
597596
'VALUES (%s, %s, %s, %s, %s, %s, %s, %s)')
598597

599598
sql_insert_dynamic_params_1 = (
600-
'2.SKB8.640193', 'Value for sample 1', 's_G1_L001_sequences',
601-
'GTCCGCAAGTTA', 'ILLUMINA', 'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA')
599+
'2.SKB8.640193', 'GTCCGCAAGTTA', 'BBBB', 'AAAA',
600+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
601+
'Value for sample 1')
602602
sql_insert_dynamic_params_2 = (
603-
'2.SKD8.640184', 'Value for sample 2', 's_G1_L001_sequences',
604-
'CGTAGAGCTCTC', 'ILLUMINA', 'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA')
603+
'2.SKD8.640184', 'CGTAGAGCTCTC', 'BBBB', 'AAAA',
604+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
605+
'Value for sample 2')
605606

606607
exp = [
607608
(sql_insert_common, sql_insert_common_params_1),
608609
(sql_insert_common, sql_insert_common_params_2),
609-
(sql_insert_prep_columns, (2, 'str_column', 'varchar')),
610-
(sql_insert_prep_columns, (2, 'run_prefix', 'varchar')),
611610
(sql_insert_prep_columns, (2, 'barcodesequence', 'varchar')),
612-
(sql_insert_prep_columns, (2, 'platform', 'varchar')),
613-
(sql_insert_prep_columns, (2, 'linkerprimersequence', 'varchar')),
614611
(sql_insert_prep_columns,
615612
(2, 'experiment_design_description', 'varchar')),
616613
(sql_insert_prep_columns,
617614
(2, 'library_construction_protocol', 'varchar')),
615+
(sql_insert_prep_columns, (2, 'linkerprimersequence', 'varchar')),
616+
(sql_insert_prep_columns, (2, 'platform', 'varchar')),
617+
(sql_insert_prep_columns, (2, 'run_prefix', 'varchar')),
618+
(sql_insert_prep_columns, (2, 'str_column', 'varchar')),
618619
(sql_create_table, None),
619620
(sql_insert_dynamic, sql_insert_dynamic_params_1),
620621
(sql_insert_dynamic, sql_insert_dynamic_params_2)]
@@ -794,17 +795,20 @@ def test_create(self):
794795
# The new table hosts the correct values
795796
obs = self.conn_handler.execute_fetchall(
796797
"SELECT * FROM qiita.prep_2")
797-
# sample_id, study_id, str_column, ebi_submission_accession,
798-
# run_prefix, barcodesequence, linkerprimersequence
799-
exp = [['1.SKB7.640196', 'Value for sample 3', 'ILLUMINA',
800-
's_G1_L002_sequences', 'CCTCTGAGAGCT', None,
801-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA'],
802-
['1.SKB8.640193', 'Value for sample 1', 'ILLUMINA',
803-
's_G1_L001_sequences', 'GTCCGCAAGTTA', None,
804-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA'],
805-
['1.SKD8.640184', 'Value for sample 2', 'ILLUMINA',
806-
's_G1_L001_sequences', 'CGTAGAGCTCTC', None,
807-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA']]
798+
799+
# barcodesequence, ebi_submission_accession,
800+
# experiment_design_description, library_construction_protocol,
801+
# linkerprimersequence, platform, run_prefix, str_column
802+
exp = [['1.SKB7.640196', 'CCTCTGAGAGCT', None, 'BBBB', 'AAAA',
803+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L002_sequences',
804+
'Value for sample 3'],
805+
['1.SKB8.640193', 'GTCCGCAAGTTA', None, 'BBBB', 'AAAA',
806+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
807+
'Value for sample 1'],
808+
['1.SKD8.640184', 'CGTAGAGCTCTC', None, 'BBBB', 'AAAA',
809+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
810+
'Value for sample 2']]
811+
808812
self.assertEqual(sorted(obs), sorted(exp))
809813

810814
# prep and qiime files have been created
@@ -858,17 +862,18 @@ def test_create_already_prefixed_samples(self):
858862
# The new table hosts the correct values
859863
obs = self.conn_handler.execute_fetchall(
860864
"SELECT * FROM qiita.prep_2")
861-
# sample_id, study_id, str_column, ebi_submission_accession,
862-
# run_prefix, barcodesequence, linkerprimersequence
863-
exp = [['1.SKB7.640196', 'Value for sample 3', 'ILLUMINA',
864-
's_G1_L002_sequences', 'CCTCTGAGAGCT', None,
865-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA'],
866-
['1.SKB8.640193', 'Value for sample 1', 'ILLUMINA',
867-
's_G1_L001_sequences', 'GTCCGCAAGTTA', None,
868-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA'],
869-
['1.SKD8.640184', 'Value for sample 2', 'ILLUMINA',
870-
's_G1_L001_sequences', 'CGTAGAGCTCTC', None,
871-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA']]
865+
# barcodesequence, ebi_submission_accession,
866+
# experiment_design_description, library_construction_protocol,
867+
# linkerprimersequence, platform, run_prefix, str_column
868+
exp = [['1.SKB7.640196', 'CCTCTGAGAGCT', None, 'BBBB', 'AAAA',
869+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L002_sequences',
870+
'Value for sample 3'],
871+
['1.SKB8.640193', 'GTCCGCAAGTTA', None, 'BBBB', 'AAAA',
872+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
873+
'Value for sample 1'],
874+
['1.SKD8.640184', 'CGTAGAGCTCTC', None, 'BBBB', 'AAAA',
875+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
876+
'Value for sample 2']]
872877
self.assertEqual(sorted(obs), sorted(exp))
873878

874879
# prep and qiime files have been created
@@ -959,17 +964,18 @@ def test_create_data_type_id(self):
959964
# The new table hosts the correct values
960965
obs = self.conn_handler.execute_fetchall(
961966
"SELECT * FROM qiita.prep_2")
962-
# sample_id, str_column, ebi_submission_accession,
963-
# run_prefix, barcodesequence, linkerprimersequence
964-
exp = [['1.SKB7.640196', 'Value for sample 3', 'ILLUMINA',
965-
's_G1_L002_sequences', 'CCTCTGAGAGCT', None,
966-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA'],
967-
['1.SKB8.640193', 'Value for sample 1', 'ILLUMINA',
968-
's_G1_L001_sequences', 'GTCCGCAAGTTA', None,
969-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA'],
970-
['1.SKD8.640184', 'Value for sample 2', 'ILLUMINA',
971-
's_G1_L001_sequences', 'CGTAGAGCTCTC', None,
972-
'GTGCCAGCMGCCGCGGTAA', 'BBBB', 'AAAA']]
967+
# barcodesequence, ebi_submission_accession,
968+
# experiment_design_description, library_construction_protocol,
969+
# linkerprimersequence, platform, run_prefix, str_column
970+
exp = [['1.SKB7.640196', 'CCTCTGAGAGCT', None, 'BBBB', 'AAAA',
971+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L002_sequences',
972+
'Value for sample 3'],
973+
['1.SKB8.640193', 'GTCCGCAAGTTA', None, 'BBBB', 'AAAA',
974+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
975+
'Value for sample 1'],
976+
['1.SKD8.640184', 'CGTAGAGCTCTC', None, 'BBBB', 'AAAA',
977+
'GTGCCAGCMGCCGCGGTAA', 'ILLUMINA', 's_G1_L001_sequences',
978+
'Value for sample 2']]
973979
self.assertEqual(sorted(obs), sorted(exp))
974980

975981
def test_create_error(self):

0 commit comments

Comments
 (0)