Add test preventing use of built-in String repeat method

MaoShizhong · MaoShizhong · commit 5ea6746a8d64 · 2025-02-04T04:08:26.000Z
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');
   });
