From c202bb4a004d4b358ede3faa7e63122886533b88 Mon Sep 17 00:00:00 2001
From: Salih <salihyildirimnorge82@gmail.com>
Date: Thu, 16 Jul 2020 21:03:37 +0200
Subject: [PATCH] solved sumAll.js

---
 sumAll/sumAll.js      | 25 +++++++++++++++++++++++--
 sumAll/sumAll.spec.js | 10 +++++-----
 2 files changed, 28 insertions(+), 7 deletions(-)

diff --git a/sumAll/sumAll.js b/sumAll/sumAll.js
index 4030fe8..8e1eb6e 100644
--- a/sumAll/sumAll.js
+++ b/sumAll/sumAll.js
@@ -1,5 +1,26 @@
-const sumAll = function() {
-
+const sumAll = function(number1,number2) {
+    if (number1<0 || number2<0 ){
+        return 'ERROR';
+    }
+    else if (typeof number1 != 'number' || typeof number2 != 'number'){ 
+       return 'ERROR';
+    }
+    else {       
+        if (number2>number1){
+            let sum=0;
+            for (let index = number1; index <= number2; index++) {
+                sum=sum+index;
+            }
+            return sum;
+        }
+        else{
+            let sum=0;
+            for (let index = number2; index <= number1; index++) {
+                sum=sum+index;
+            }
+            return sum;
+        }   
+    }    
 }
 
 module.exports = sumAll
diff --git a/sumAll/sumAll.spec.js b/sumAll/sumAll.spec.js
index 520adc1..2a584a6 100644
--- a/sumAll/sumAll.spec.js
+++ b/sumAll/sumAll.spec.js
@@ -4,19 +4,19 @@ describe('sumAll', function() {
   it('sums numbers within the range', function() {
     expect(sumAll(1, 4)).toEqual(10);
   });
-  xit('works with large numbers', function() {
+  it('works with large numbers', function() {
     expect(sumAll(1, 4000)).toEqual(8002000);
   });
-  xit('works with larger number first', function() {
+  it('works with larger number first', function() {
     expect(sumAll(123, 1)).toEqual(7626);
   });
-  xit('returns ERROR with negative numbers', function() {
+  it('returns ERROR with negative numbers', function() {
     expect(sumAll(-10, 4)).toEqual('ERROR');
   });
-  xit('returns ERROR with non-number parameters', function() {
+  it('returns ERROR with non-number parameters', function() {
     expect(sumAll(10, "90")).toEqual('ERROR');
   });
-  xit('returns ERROR with non-number parameters', function() {
+  it('returns ERROR with non-number parameters', function() {
     expect(sumAll(10, [90, 1])).toEqual('ERROR');
   });
 });