Merge pull request TheOdinProject#523 from MaoShizhong/repeat-string-builtin

JoshDevHub · web-flow · commit ace1d6eb4db6 · 2025-02-10T20:03:00.000-07:00
02_repeatString: Add test preventing use of `String.prototype.repeat`
diff --git a/02_repeatString/repeatString.spec.js b/02_repeatString/repeatString.spec.js
@@ -13,6 +13,13 @@ describe('repeatString', () => {
   test.skip('repeats the string 0 times', () => {
     expect(repeatString('bye', 0)).toEqual('');
   });
+  test.skip('does not use the built-in String repeat method', () => {
+    /* Even though there is a built-in String repeat method,
+      in this exercise specifically, we want you to practise using loops */
+    jest.spyOn(String.prototype, 'repeat').mockName('Built-in String repeat method');
+    repeatString("don't use the built-in repeat method!", 1);
+    expect(String.prototype.repeat).not.toHaveBeenCalled();
+  });
   test.skip('returns ERROR with negative numbers', () => {
     expect(repeatString('goodbye', -1)).toEqual('ERROR');
   });
diff --git a/02_repeatString/solution/repeatString-solution.spec.js b/02_repeatString/solution/repeatString-solution.spec.js
@@ -13,6 +13,13 @@ describe('repeatString', () => {
   test('repeats the string 0 times', () => {
     expect(repeatString('bye', 0)).toEqual('');
   });
+  test('does not use the built-in String repeat method', () => {
+    /* Even though there is a built-in String repeat method,
+      in this exercise specifically, we want you to practise using loops */
+    jest.spyOn(String.prototype, 'repeat').mockName('Built-in String repeat method');
+    repeatString("don't use the built-in repeat method!", 1);
+    expect(String.prototype.repeat).not.toHaveBeenCalled();
+  });
   test('returns ERROR with negative numbers', () => {
     expect(repeatString('goodbye', -1)).toEqual('ERROR');
   });
