From e542c353739ce5da67f9b398724f66c8d9ac93be Mon Sep 17 00:00:00 2001
From: endritibra <endrit.ibraimimax@live.com>
Date: Wed, 1 Jun 2022 03:13:52 +0200
Subject: [PATCH] Ex.5 Done

---
 05_sumAll/sumAll.js      | 16 +++++++++++++++-
 05_sumAll/sumAll.spec.js | 10 +++++-----
 2 files changed, 20 insertions(+), 6 deletions(-)

diff --git a/05_sumAll/sumAll.js b/05_sumAll/sumAll.js
index 00880c7..fb9b0fa 100644
--- a/05_sumAll/sumAll.js
+++ b/05_sumAll/sumAll.js
@@ -1,4 +1,18 @@
-const sumAll = function() {
+const sumAll = function(min,max) {
+	if(!Number.isInteger(min) || !Number.isInteger(max)) return "ERROR";
+if(min<0 || max<0) return "ERROR";
+if(min>max){
+const temp=min;
+min=max;
+max=temp;
+}
+let sum=0;
+for(let i=min;i<max+1;i++){
+sum+=i;
+}
+return sum;
+	
+
 
 };
 
diff --git a/05_sumAll/sumAll.spec.js b/05_sumAll/sumAll.spec.js
index 1a9fb7c..a171e5f 100644
--- a/05_sumAll/sumAll.spec.js
+++ b/05_sumAll/sumAll.spec.js
@@ -4,19 +4,19 @@ describe('sumAll', () => {
   test('sums numbers within the range', () => {
     expect(sumAll(1, 4)).toEqual(10);
   });
-  test.skip('works with large numbers', () => {
+  test('works with large numbers', () => {
     expect(sumAll(1, 4000)).toEqual(8002000);
   });
-  test.skip('works with larger number first', () => {
+  test('works with larger number first', () => {
     expect(sumAll(123, 1)).toEqual(7626);
   });
-  test.skip('returns ERROR with negative numbers', () => {
+  test('returns ERROR with negative numbers', () => {
     expect(sumAll(-10, 4)).toEqual('ERROR');
   });
-  test.skip('returns ERROR with non-number parameters', () => {
+  test('returns ERROR with non-number parameters', () => {
     expect(sumAll(10, "90")).toEqual('ERROR');
   });
-  test.skip('returns ERROR with non-number parameters', () => {
+  test('returns ERROR with non-number parameters', () => {
     expect(sumAll(10, [90, 1])).toEqual('ERROR');
   });
 });