fixing tidyUpAnnotation logic and adding a test

TeselaGen · Oct 19, 2021 · 1d37492 · 1d37492
1 parent 1c76930
commit 1d37492
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/tidyUpAnnotation.js b/src/tidyUpAnnotation.js
@@ -84,8 +84,8 @@ module.exports = function tidyUpAnnotation(
   if (
     !annotation.type ||
     typeof annotation.type !== "string" ||
-    FeatureTypes.some(function(featureType) {
-      if (featureType.toLowerCase === annotation.type.toLowerCase()) {
+    !FeatureTypes.some(function(featureType) {
+      if (featureType.toLowerCase() === annotation.type.toLowerCase()) {
         annotation.type = featureType; //this makes sure the annotation.type is being set to the exact value of the accepted featureType
         return true;
       }

diff --git a/src/tidyUpSequenceData.test.js b/src/tidyUpSequenceData.test.js
@@ -217,6 +217,12 @@ describe("tidyUpSequenceData", function() {
     });
     res.features[0].type.should.equal("misc_feature");
   });
+  it("should add feature type = misc_feature if an invalid type is provided", function() {
+    const res = tidyUpSequenceData({
+      features: [{ start: 4, end: 5, type: "idontexist" }]
+    });
+    res.features[0].type.should.equal("misc_feature");
+  });
   // it("should normalize strange upper/lower casing in feature types", function() {
   //   const res = tidyUpSequenceData(
   //     {