Edit introduction and update proof.ci.js, error-handling.js, and error-handling.spec.js

Author: A-O-Emmanuel

exercises/practice/error-handling/.docs/introduction.md

@@ -4,16 +4,12 @@ In this exercise, you will implement a function called `processString` that proc
 
 You will learn how to:
 
-- Check input types and handle invalid inputs by throwing errors.
-- Throw custom errors for specific cases (e.g., empty strings).
-- Use a `try...finally` block to ensure certain code runs regardless of success or failure (such as cleanup or logging).
+- Check input types and throw errors for invalid inputs.
+- Throw errors for specific cases (e.g., empty strings).
 - Return the uppercase version of the string if it is valid.
 
 Your function should:
 
-- Throw a `TypeError` if the input is not a string.
-- Throw an `Error` with the message `EmptyStringError` if the input string is empty.
-- Return the uppercase form of the input string if valid.
-- Use a `finally` block to log a cleanup message every time the function runs.
-
-This exercise is a great way to practice writing robust functions that can gracefully handle unexpected inputs while always executing necessary cleanup logic.
+- Throw a `TypeError` if input is not a string.
+- Throw a general `Error` if input is an empty string.
+- Return the uppercase form of the string 'hello'.

exercises/practice/error-handling/.meta/proof.ci.js

@@ -1,14 +1,9 @@
 export const processString = (input) => {
-  try {
-    if (typeof input !== 'string') {
-      throw new TypeError('input must be a string');
-    }
-    if (input === '') {
-      throw new Error('EmptyStringError');
-    }
-    return input.toUpperCase();
-  } finally {
-    console.log('Resource cleaned up');
+  if (typeof input !== 'string') {
+    throw new TypeError();
   }
-}
-
+  if (input === '') {
+    throw new Error();
+  }
+  return input.toUpperCase();
+};

exercises/practice/error-handling/error-handling.js

@@ -1,7 +1,6 @@
 export const processString = (input) => {
-    //TODO: implement this
-    //should throw Error if input is not a string
-    //should throw Error if input is an empty string
-    //should return the uppercase version of the string if valid
-    //should use finally to log cleanup message
-}
+  //TODO: implement this
+  //should throw TypeError if input is not a string
+  //should throw a general Error if input is an empty string
+  //should return the uppercase version of the string 'hello'
+};

exercises/practice/error-handling/error-handling.spec.js

@@ -1,35 +1,16 @@
-import { describe, expect, it, jest } from '@jest/globals';
+import { describe, expect, test, xtest } from '@jest/globals';
 import { processString } from './error-handling';
 
 describe('Error Handling', () => {
-  it('returns uppercase if valid string', () => {
-    expect(processString('hello')).toBe('HELLO');
-  });
-
-  it('throws TypeError if input is not a string', () => {
+  test('throws TypeError if input is not a string', () => {
     expect(() => processString(42)).toThrow(TypeError);
-    expect(() => processString(42)).toThrow('input must be a string');
   });
 
-  it('throws Error with EmptyStringError if string is empty', () => {
+  xtest('throws Error message if string is empty', () => {
     expect(() => processString('')).toThrow(Error);
-    expect(() => processString('')).toThrow('EmptyStringError');
   });
 
-  it('always logs cleanup message', () => {
-    console.log = jest.fn();
-
-    try {
-      processString('');
-    } catch {
-      /*
-                intentionally left empty,
-                I expext this call to throw,
-                but only care about verifying that the finally block is executed
-                and clean up message logged.
-            */
-    }
-
-    expect(console.log).toHaveBeenCalledWith('Resource cleaned up');
+  xtest('returns uppercase string if input is valid', () => {
+    expect(processString('hello')).toBe('HELLO');
   });
 });