Merge pull request #459 from 18F/winners-page-tests

Winners page tests
18F · Apr 29, 2016 · d6659e6 · d6659e6
2 parents 3b7033d + eda5c0c
commit d6659e6
Show file tree

Hide file tree

Showing 11 changed files with 770 additions and 252 deletions.
diff --git a/app/assets/javascripts/utility.js b/app/assets/javascripts/utility.js
@@ -55,14 +55,18 @@
   };
 
   microp.format.removeGitPrefix = function(str) {
-    return str.split('https://github.com/').join('');
+    return str.includes('https')
+      ? str.split("https://github.com/").join("")
+      : str;
   };
 
   microp.format.transformDollars = function(str) {
+    str = String(str);
     if (str.charAt(0) === '-') {
-      str = '-' + str.substr(1);
+      return '-$' + str.substr(1, str.length);
+    } else {
+      return '$' + str;
     }
-    return '$' + str;
   };
 
   microp.format.commaSeparatedDollars = microp.format.transform(