Add a failing test for sample sheet parser

This case tests the behaviour described in #888

When the issue is solved, we should unmark this case as "Failure".

micall/tests/test_sample_sheet_parser.py

@@ -777,3 +777,46 @@ def test_read_sample_sheet_overrides(tmpdir):
     assert len(split_rows) == 2
     assert split_rows[0]['project'] == 'Proj1'
     assert split_rows[1]['project'] == 'AltB'
+
+
+class UnderscoreCases(unittest.TestCase):
+
+    @unittest.expectedFailure
+    def test_underscores_in_sample_name1(tmpdir):
+        sample_sheet_path = Path(str(tmpdir)) / 'SampleSheet.csv'
+
+        sample_sheet_path.write_text("""\
+[Header]
+IEMFileVersion,3
+Investigator Name,RL
+Project Name,10-Jul-2014_v1test
+Experiment Name,10-Jul-2014_v1test
+Date,07/10/2014
+Workflow,GenerateFASTQ
+Assay,Nextera
+Description,Nextera
+Chemistry,Amplicon
+[Reads]
+251
+251
+[Settings]
+[Data]
+Sample_ID,Sample_Name,Sample_Plate,Sample_Well,index,index2,Sample_Project,Description,GenomeFolder
+CFE_SomeId_10-Jul-2014_N501-N701_Sample1_Proj1,Sample1_Proj1,10-Jul-2014_testing,N/A,ACGTACGT,TGCATGCA,\
+10-Jul-2014_testing,Research:Sample1_Proj1:TRUE Comments:Sample1_Proj1:thisiscommentone \
+Disablecontamcheck:Sample1_Proj1:FALSE,
+CFE_SomeId_10-Jul-2014_N501-N702_Sample2_Proj2,Sample2_Proj2,10-Jul-2014_testing,N/A,AAAAGGGG,CCCCTTTT,\
+10-Jul-2014_testing,Research:Sample2_Foo_Proj2:FALSE Comments:Sample2_Foo_Proj2:thisiscommenttwo \
+Chemistry:Sample2_Foo_Proj2:BreakingBad Disablecontamcheck:Sample2_Foo_Proj2:TRUE,
+""")
+
+        with sample_sheet_path.open() as f:
+            run_info = sample_sheet_parser(f)
+
+        sample_map = run_info['Data']
+        assert len(sample_map) == 2
+
+        split_rows = run_info['DataSplit']
+        assert len(split_rows) == 2
+        assert split_rows[0]['project'] == 'Proj1'
+        assert split_rows[1]['project'] == 'Proj2'