Fix removeLeft module and add more tests

dleitee · Feb 6, 2018 · e499b2f · e499b2f
1 parent a1411d1
commit e499b2f
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/src/removeleft.js b/src/removeleft.js
@@ -31,7 +31,7 @@ import substr from './substr'
  */
 export default (value: string, prefix: string, caseSensitive: boolean = true): string => {
   if (startsWith(value, prefix, 0, caseSensitive)) {
-    return substr(value, prefix.length)
+    return substr(value, prefix.length, value.length - prefix.length)
   }
   return value
 }
diff --git a/test/removeleft.strman.test.js b/test/removeleft.strman.test.js
@@ -8,4 +8,12 @@ describe('strman.removeLeft', () => {
       expect(removeLeft(el, 'foo')).toBe('bar')
     })
   })
+
+  test('should be Foobar', () => {
+    expect(removeLeft('Foobar', 'foo')).toBe('Foobar')
+  })
+
+  test('should be bar', () => {
+    expect(removeLeft('Foobar', 'foo', false)).toBe('bar')
+  })
 })
diff --git a/test/removeleft.test.js b/test/removeleft.test.js
@@ -8,4 +8,12 @@ describe('strman.removeLeft', () => {
       expect(removeLeft(el, 'foo')).toBe('bar')
     })
   })
+
+  test('should be Foobar', () => {
+    expect(removeLeft('Foobar', 'foo')).toBe('Foobar')
+  })
+
+  test('should be bar', () => {
+    expect(removeLeft('Foobar', 'foo', false)).toBe('bar')
+  })
 })