From 6019c78cc9acf6b299875ca6b9b893356c1f6430 Mon Sep 17 00:00:00 2001
From: Thuan Bui <9248622+10h30@users.noreply.github.com>
Date: Wed, 26 Feb 2025 09:55:45 +0900
Subject: [PATCH] Complete excercise 8

---
 08_calculator/calculator.js      | 24 +++++++++++++++++----
 08_calculator/calculator.spec.js | 36 ++++++++++++++++----------------
 2 files changed, 38 insertions(+), 22 deletions(-)

diff --git a/08_calculator/calculator.js b/08_calculator/calculator.js
index c22e8d2..81c3f71 100644
--- a/08_calculator/calculator.js
+++ b/08_calculator/calculator.js
@@ -1,25 +1,41 @@
 const add = function() {
-	
+  console.log(arguments)
+  let add = 0;
+  for (i = 0; i < arguments.length; i++) {
+      add += arguments[i]
+  }
+  return add
 };
 
 const subtract = function() {
-	
+  console.log(arguments)
+  return arguments[0] - arguments[1]
 };
 
 const sum = function() {
-	
+  console.log(arguments)
+	return arguments[0].reduce((total, arg) => total + arg,0)
 };
 
 const multiply = function() {
+  console.log(arguments)
+	return arguments[0].reduce((total, arg) => total * arg,1)
 
 };
 
 const power = function() {
+  console.log(arguments)
+  return arguments[0] ** arguments[1]
 	
 };
 
 const factorial = function() {
-	
+  console.log(arguments)
+  let factorial = 1;
+  for (i=1; i<= arguments[0]; i++) {
+    factorial *= i
+  }
+  return factorial;
 };
 
 // Do not edit below this line
diff --git a/08_calculator/calculator.spec.js b/08_calculator/calculator.spec.js
index 72152f7..b78188a 100644
--- a/08_calculator/calculator.spec.js
+++ b/08_calculator/calculator.spec.js
@@ -5,85 +5,85 @@ describe('add', () => {
     expect(calculator.add(0, 0)).toBe(0);
   });
 
-  test.skip('adds 2 and 2', () => {
+  test('adds 2 and 2', () => {
     expect(calculator.add(2, 2)).toBe(4);
   });
 
-  test.skip('adds positive numbers', () => {
+  test('adds positive numbers', () => {
     expect(calculator.add(2, 6)).toBe(8);
   });
 });
 
 describe('subtract', () => {
-  test.skip('subtracts numbers', () => {
+  test('subtracts numbers', () => {
     expect(calculator.subtract(10, 4)).toBe(6);
   });
 
-  test.skip('subtracts negative numbers', () => {
+  test('subtracts negative numbers', () => {
     expect(calculator.subtract(-10, -4)).toBe(-6);
   });
 
-  test.skip('subtracts numbers of mixed parity', () => {
+  test('subtracts numbers of mixed parity', () => {
     expect(calculator.subtract(-8, 7)).toBe(-15);
   });
 });
 
 describe('sum', () => {
-  test.skip('computes the sum of an empty array', () => {
+  test('computes the sum of an empty array', () => {
     expect(calculator.sum([])).toBe(0);
   });
 
-  test.skip('computes the sum of an array of one number', () => {
+  test('computes the sum of an array of one number', () => {
     expect(calculator.sum([7])).toBe(7);
   });
 
-  test.skip('computes the sum of an array of two numbers', () => {
+  test('computes the sum of an array of two numbers', () => {
     expect(calculator.sum([7, 11])).toBe(18);
   });
 
-  test.skip('computes the sum of an array of many numbers', () => {
+  test('computes the sum of an array of many numbers', () => {
     expect(calculator.sum([1, 3, 5, 7, 9])).toBe(25);
   });
 });
 
 describe('multiply', () => {
-  test.skip('multiplies two numbers', () => {
+  test('multiplies two numbers', () => {
     expect(calculator.multiply([2, 4])).toBe(8);
   });
 
-  test.skip('multiplies several numbers', () => {
+  test('multiplies several numbers', () => {
     expect(calculator.multiply([2, 4, 6, 8, 10, 12, 14])).toBe(645120);
   });
 });
 
 describe('power', () => {
-  test.skip('raises one number to the power of another number', () => {
+  test('raises one number to the power of another number', () => {
     expect(calculator.power(4, 3)).toBe(64); // 4 to third power is 64
   });
 
-  test.skip('raises one number to the power of a large number', () => {
+  test('raises one number to the power of a large number', () => {
     expect(calculator.power(3, 10)).toBe(59049); // 3 to tenth power is 59049
   });
 });
 
 describe('factorial', () => {
-  test.skip('computes the factorial of 0', () => {
+  test('computes the factorial of 0', () => {
     expect(calculator.factorial(0)).toBe(1); // 0! = 1
   });
 
-  test.skip('computes the factorial of 1', () => {
+  test('computes the factorial of 1', () => {
     expect(calculator.factorial(1)).toBe(1);
   });
 
-  test.skip('computes the factorial of 2', () => {
+  test('computes the factorial of 2', () => {
     expect(calculator.factorial(2)).toBe(2);
   });
 
-  test.skip('computes the factorial of 5', () => {
+  test('computes the factorial of 5', () => {
     expect(calculator.factorial(5)).toBe(120);
   });
 
-  test.skip('computes the factorial of 10', () => {
+  test('computes the factorial of 10', () => {
     expect(calculator.factorial(10)).toBe(3628800);
   });
 });