Majd_jadalhaq-w3-javascript

.test-summary/TEST_SUMMARY.md

@@ -0,0 +1,13 @@
+## Test Summary
+
+**Mentors**: For more information on how to review homework assignments, please refer to the [Review Guide](https://github.com/HackYourFuture/mentors/blob/main/assignment-support/review-guide.md).
+
+### 1-JavaScript - Week3
+
+|          Exercise          | Passed | Failed | ESLint |
+|----------------------------|--------|--------|--------|
+| ex1-doubleEvenNumbers.test |   1    |   -    |   ✓    |
+| ex2-mondaysWorth.test      |   2    |   -    |   ✓    |
+| ex3-lemonAllergy.test      |   3    |   -    |   ✓    |
+| ex4-observable             |   3    |   -    |   ✓    |
+| ex5-wallet                 |   5    |   -    |   ✓    |

1-JavaScript/Week3/assignment/ex1-doubleEvenNumbers.test.js

@@ -12,13 +12,9 @@ Let's rewrite it (or _refactor_ it, as experienced developers would call it):
 // ! Function to be tested
 function doubleEvenNumbers(numbers) {
   // TODO rewrite the function body using `map` and `filter`.
-  const newNumbers = [];
-  for (let i = 0; i < numbers.length; i++) {
-    if (numbers[i] % 2 === 0) {
-      newNumbers.push(numbers[i] * 2);
-    }
-  }
-  return newNumbers;
+  return numbers
+    .filter((number) => number % 2 === 0)
+    .map((number) => number * 2);
 }
 
 // ! Unit test (using Jest)

1-JavaScript/Week3/assignment/ex2-mondaysWorth.test.js

@@ -30,8 +30,16 @@ const mondayTasks = [
 
 const hourlyRate = 25;
 
-function computeEarnings(/* TODO parameter(s) go here */) {
+function computeEarnings(tasks, rate /* TODO parameter(s) go here */) {
   // TODO complete this function
+  const totalHours = tasks
+    .map((task) => task.duration / 60) // convert minutes to hours
+    .reduce((sum, hours) => sum + hours, 0); // sum all hours
+
+  const totalEarnings = totalHours * rate;
+
+  // Format as Euro
+  return `€${totalEarnings.toFixed(2)}`;
 }
 
 // ! Unit tests (using Jest)

1-JavaScript/Week3/assignment/ex3-lemonAllergy.test.js

@@ -25,26 +25,32 @@ const fruitBasket = [
 ];
 
 // ! Function under test
-function sanitizeFruitBasket(/* TODO parameter(s) go here */) {
-  // TODO complete this function
+function sanitizeFruitBasket(
+  basket,
+  fruitToRemove /* TODO parameter(s) go here */
+) {
+  return basket.filter((fruit) => fruit !== fruitToRemove);
 }
 
 // ! Unit tests (using Jest)
 describe('js-wk3-ex3-lemonAllergy', () => {
   test('sanitizeFruitBasket should take two parameters', () => {
     // TODO replace next line with your code
-    expect(false).toBe(true);
+    expect(sanitizeFruitBasket).toHaveLength(2);
   });
 
   test('sanitizeFruitBasket should not modify the original `fruitBasket` array', () => {
     // Save the original contents of the fruit basket
     const originalFruitBasketContents = [...fruitBasket];
     // TODO replace next line with your code
-    expect(false).toBe(true);
+    sanitizeFruitBasket(fruitBasket, 'lemon');
+    expect(fruitBasket).toEqual(originalFruitBasketContents);
   });
 
   test('sanitizeFruitBasket should return a new array that does not include the unwanted `lemon`', () => {
     // TODO replace next line with your code
-    expect(false).toBe(true);
+    const sanitized = sanitizeFruitBasket(fruitBasket, 'lemon');
+    expect(sanitized).not.toContain('lemon');
+    expect(sanitized).toEqual(['apple', 'grapefruit', 'banana', 'watermelon']);
   });
 });

1-JavaScript/Week3/assignment/ex4-observable/ex4-observable.js

@@ -16,10 +16,10 @@ export function createObservable() {
   const subscribers = [];
   return {
     subscribe(subscriber) {
-      // TODO complete this function
+subscribers.push(subscriber);
     },
     notify(message) {
-      // TODO complete this function
+      subscribers.forEach((fn) => fn(message));
     },
   };
 }

1-JavaScript/Week3/assignment/ex5-wallet/index.js

@@ -75,7 +75,7 @@ const quiz = {
       b: 'cash, name', 
       c: 'amount, this, wallet'
     },
-    answer: '?',
+    answer: 'b',
   },
   q2: {
     question: 'What is in the Call Stack, from top to bottom?',
@@ -84,7 +84,7 @@ const quiz = {
       b: 'anonymous, transferInto', 
       c: 'transferInto, anonymous' 
     },
-    answer: '?',
+    answer: 'c',
   },
   q3: {
     question: 'What tooltip appears when hovering over the third debug button?',
@@ -93,7 +93,7 @@ const quiz = {
       b: 'Step out of current function', 
       c: 'Step' 
     },
-    answer: '?',
+    answer: 'a',
   },
   q4: {
     question: 'What is displayed in the console?',
@@ -102,7 +102,7 @@ const quiz = {
       b: 'Transferring € 50,00 from Jack to undefined', 
       c: 'Transferring € 50,00 from Jack to Jane' 
     },
-    answer: '?',
+    answer: 'a',
   },
   q5: {
     question: 'The owner of the wallet with insufficient funds is:',
@@ -111,6 +111,6 @@ const quiz = {
       b: 'Joe', 
       c: 'Jane' 
     },
-    answer: '?',
+    answer: 'c',
   },
 };

1-JavaScript/Week3/test-reports/ex1-doubleEvenNumbers.test.report.txt

@@ -0,0 +1,13 @@
+*** Unit Test Error Report ***
+
+ PASS  1-JavaScript/Week3/assignment/ex1-doubleEvenNumbers.test.js
+  js-wk3-ex1-doubleEvenNumbers
+    ✅ doubleEvenNumbers should take the even numbers and double them (6 ms)
+
+Test Suites: 1 passed, 1 total
+Tests:       1 passed, 1 total
+Snapshots:   0 total
+Time:        0.447 s
+Ran all test suites matching /\/Users\/majdjadalhaq\/Desktop\/Assignments-Cohort54\/1-JavaScript\/Week3\/assignment\/ex1-doubleEvenNumbers.test.js/i.
+No linting errors detected.
+No spelling errors detected.

1-JavaScript/Week3/test-reports/ex2-mondaysWorth.test.report.txt

@@ -0,0 +1,14 @@
+*** Unit Test Error Report ***
+
+ PASS  1-JavaScript/Week3/assignment/ex2-mondaysWorth.test.js
+  js-wk3-mondaysWorth
+    ✅ computeEarnings should take two parameters (2 ms)
+    ✅ computeEarnings should compute the earnings as a formatted Euro amount (1 ms)
+
+Test Suites: 1 passed, 1 total
+Tests:       2 passed, 2 total
+Snapshots:   0 total
+Time:        0.46 s, estimated 1 s
+Ran all test suites matching /\/Users\/majdjadalhaq\/Desktop\/Assignments-Cohort54\/1-JavaScript\/Week3\/assignment\/ex2-mondaysWorth.test.js/i.
+No linting errors detected.
+No spelling errors detected.

1-JavaScript/Week3/test-reports/ex3-lemonAllergy.test.report.txt

@@ -0,0 +1,15 @@
+*** Unit Test Error Report ***
+
+ PASS  1-JavaScript/Week3/assignment/ex3-lemonAllergy.test.js
+  js-wk3-ex3-lemonAllergy
+    ✅ sanitizeFruitBasket should take two parameters (2 ms)
+    ✅ sanitizeFruitBasket should not modify the original `fruitBasket` array (1 ms)
+    ✅ sanitizeFruitBasket should return a new array that does not include the unwanted `lemon`
+
+Test Suites: 1 passed, 1 total
+Tests:       3 passed, 3 total
+Snapshots:   0 total
+Time:        0.445 s
+Ran all test suites matching /\/Users\/majdjadalhaq\/Desktop\/Assignments-Cohort54\/1-JavaScript\/Week3\/assignment\/ex3-lemonAllergy.test.js/i.
+No linting errors detected.
+No spelling errors detected.

1-JavaScript/Week3/test-reports/ex4-observable.report.txt

@@ -0,0 +1,15 @@
+*** Unit Test Error Report ***
+
+ PASS  1-JavaScript/Week3/assignment/ex4-observable/ex4-observable.test.js
+  js-wk3-ex4-observable
+    ✅ createObservable should exist and be a function (2 ms)
+    ✅ createObservable should return an object with `subscribe` and a `notify` function properties
+    ✅ observable should notify all subscribers of any notification (2 ms)
+
+Test Suites: 1 passed, 1 total
+Tests:       3 passed, 3 total
+Snapshots:   0 total
+Time:        0.389 s, estimated 1 s
+Ran all test suites matching /\/Users\/majdjadalhaq\/Desktop\/Assignments-Cohort54\/1-JavaScript\/Week3\/assignment\/ex4-observable\/ex4-observable.test.js/i.
+No linting errors detected.
+No spelling errors detected.

1-JavaScript/Week3/test-reports/ex5-wallet.report.txt

@@ -0,0 +1,17 @@
+*** Unit Test Error Report ***
+
+ PASS  .dist/1-JavaScript/Week3/unit-tests/ex5-wallet.test.js
+  js-wk3-ex5-wallet
+    ✅ q1: At line 24, which variables are in the scope marked Closure? (2 ms)
+    ✅ q2: What is in the Call Stack, from top to bottom?
+    ✅ q3: What tooltip appears when hovering over the third debug button?
+    ✅ q4: What is displayed in the console?
+    ✅ q5: The owner of the wallet with insufficient funds is?
+
+Test Suites: 1 passed, 1 total
+Tests:       5 passed, 5 total
+Snapshots:   0 total
+Time:        0.409 s, estimated 1 s
+Ran all test suites matching /\/Users\/majdjadalhaq\/Desktop\/Assignments-Cohort54\/.dist\/1-JavaScript\/Week3\/unit-tests\/ex5-wallet.test.js/i.
+No linting errors detected.
+No spelling errors detected.