Skip to content

Commit 52f0f26

Browse files
authored
Merge pull request #370 from AdamaJava/qbasepileup_mt
fix(qbasepileup): chrM was automatically changed to chrMT
2 parents e4c20ee + 5d23393 commit 52f0f26

File tree

3 files changed

+3
-11
lines changed

3 files changed

+3
-11
lines changed

qbasepileup/src/org/qcmg/qbasepileup/QBasePileupUtil.java

-8
Original file line numberDiff line numberDiff line change
@@ -39,17 +39,9 @@ public static String getFullChromosome(String ref) {
3939

4040
// if ref starts with chr or GL, just return it
4141
if (ref.startsWith("chr") || ref.startsWith("GL")) {
42-
if (ref.equals("chrM")) {
43-
return "chrMT";
44-
}
4542
return ref;
4643
}
4744

48-
49-
if (ref.equals("M")) {
50-
return "chrMT";
51-
}
52-
5345
if (addChromosomeReference(ref)) {
5446
return "chr" + ref;
5547
} else {

qbasepileup/test/org/qcmg/qbasepileup/QBasePileupUtilTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ public void testGetFullChromosome() {
1616

1717
assertEquals("chr10", QBasePileupUtil.getFullChromosome("10"));
1818
assertEquals("chr10", QBasePileupUtil.getFullChromosome("chr10"));
19-
assertEquals("chrMT", QBasePileupUtil.getFullChromosome("M"));
19+
assertEquals("chrM", QBasePileupUtil.getFullChromosome("M"));
2020
assertEquals("chrMT", QBasePileupUtil.getFullChromosome("MT"));
2121
assertEquals("GL12345", QBasePileupUtil.getFullChromosome("GL12345"));
2222
}

qbasepileup/test/org/qcmg/qbasepileup/snp/SnpPositionTest.java

+2-2
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,9 @@ public void testEquals() throws QBasePileupException {
5050
}
5151

5252
@Test
53-
public void doesMurnIntochrMT() {
53+
public void doesMurnIntochrM() {
5454
SnpPosition p = new SnpPosition("test", "M", 1234, 1235);
55-
assertEquals("chrMT", p.getFullChromosome());
55+
assertEquals("chrM", p.getFullChromosome());
5656
}
5757
@Test
5858
public void doesMTurnIntochrMT() {

0 commit comments

Comments
 (0)