Osman Elsahib- TDD

I.pass-tests/01-add-one/add-one.js

@@ -1 +1,3 @@
-module.exports = function(numbers) {};
+module.exports = function(numbers) {
+    return numbers.map((number)=> number +1)
+};

I.pass-tests/02-get-word-lengths/get-word-lengths.js

@@ -1,3 +1,5 @@
-var getWordLengths = function(someWords) {};
+var getWordLengths = function (someWords) {
+    return someWords.map(word => word.length)
+};
 
 module.exports = getWordLengths;

I.pass-tests/03-sum-numbers/add-numbers.js

@@ -1,2 +1,7 @@
-function addNumbers(numbers) {}
+function addNumbers(numbers) {
+    return numbers.reduce(function (a, b) {
+        return a + b;
+    }, 0);
+}
+
 module.exports = addNumbers;

I.pass-tests/04-find-needle/find-needle.js

@@ -1,3 +1,10 @@
-function findNeedle(words) {}
+function findNeedle(words) {
+    let result;
+    const lookForWords = ["needle", "plant"]
+    words.forEach((element, index) => {
+        lookForWords.includes(element) ? result = index : "Not found"
+    });
+    return result
+}
 
 module.exports = findNeedle;

I.pass-tests/05-factorial/factorial.js

@@ -8,6 +8,14 @@
 // calculate and return the factorial of int
 // note: factorial of 0 is 1
 
-function factorial(int) {}
+function factorial(int) {
+    // let result = 1, i = int;
+    if (int === 0 || int === 1)
+        return 1;
+    for (var i = int - 1; i >= 1; i--) {
+        int *= i;
+    }
+    return int;
+}
 
 module.exports = factorial;

I.pass-tests/06-remove-middle/remove-middle.js

@@ -0,0 +1,7 @@
+function removeMiddle(array) {
+    if (array.length >= 3) {
+        let index = (array.length - 1) / 2
+        return array.splice(index, 1)
+    } else { return "Can't work with such short array" }
+}
+module.exports = removeMiddle;

I.pass-tests/07-second-largest/second-largest.js

@@ -0,0 +1,8 @@
+function secondLargest(array) {
+    let actualMax = Math.max(...array)
+    let maxIndex = array.indexOf(actualMax)
+    array.splice(maxIndex, 1)
+    return Math.max(...array)
+}
+
+module.exports = secondLargest;

I.pass-tests/08-get-average/get-average.js

@@ -1,3 +1,17 @@
 // the input is an array of numbers and strings
 // return the average of all the numbers
 // be sure to exclude the strings
+function getAverage(array) {
+    let numArray = [];
+    array.forEach(element => {
+        if (typeof element == "number") {
+            numArray.push(element)
+        }
+    });
+    let sum = numArray.reduce(function (a, b) {
+        return a + b;
+    }, 0);
+    return sum / numArray.length
+
+}
+module.exports = getAverage;

I.pass-tests/09-car-sales/car-sales.js

@@ -0,0 +1,24 @@
+function carSales(array) {
+    let result = {}, ford = 0, honda = 0, rover = 0, toyota = 0;
+    array.forEach(element => {
+        if (element.make === "Ford") {
+            ford = ford + element.price
+        } else if (element.make === "Honda") {
+            honda = honda + element.price
+        } else if (element.make === "Land Rover") {
+            rover = rover + element.price
+            debugger
+        } else {
+            toyota = toyota + element.price
+        }
+
+    });
+    result.Ford = ford
+    result.Honda = honda
+    result["Land Rover"] = rover
+    result.Toyota = toyota
+
+    return result
+
+}
+module.exports = carSales;

I.pass-tests/10-paint-ford/paint-cars.js

@@ -0,0 +1,7 @@
+function paintCars(array, color) {
+    // I didn't understand what is required to be done so I hardcoded the index
+    newarray = [...array]
+    newarray[0] = { ...newarray[0], colour: color }
+    return newarray
+}
+module.exports = paintCars;

I.pass-tests/11-cities-formatter/cities.js

@@ -0,0 +1,4 @@
+function cities(array, func) {
+    return array.map(item => func(item))
+}
+module.exports = cities;

I.pass-tests/ES-6-practice/es6.js

@@ -1,27 +1,26 @@
 // Turn this function into an arrow function
-function arrow() {
-  return "es6 is awesome!";
-}
+const arrow = () => { return "es6 is awesome!" };
+
 
 //Use the es6 syntax for default parameter
-function defaultParameter(name) {
-  var name = name || "sam";
+function defaultParameter(name = "sam") {
   return name;
 }
 
 // Use the spread operator to combine arr1 and arr2
 function combineArrays(arr1, arr2) {
-  return arr1.concat(arr2);
+  arr1 = [...arr1, ...arr2]
+  return arr1;
 }
 
 //use destructuring to return the object's cyf property
-function destructuring(obj) {
-  return obj.cyf;
+function destructuring({ cyf }) {
+  return cyf;
 }
 
 // use template literal to return a string with the sum of a and b
 function templateString(a, b) {
-  return "The sum is equal to " + (a + b).toString();
+  return `The sum is equal to ${(a + b).toString()}`;
 }
 
 module.exports = {

I.pass-tests/ES-6-practice/es6.test.js

@@ -4,7 +4,7 @@ const {
   combineArrays,
   destructuring,
   templateString
-} = require("./index");
+} = require("./es6");
 
 test("function arrow is instance of a function", () => {
   expect(arrow).toBeInstanceOf(Function);

II.write-tests/01-greet-people/greet-people.js

@@ -1,11 +1,11 @@
 function greetPeople(people) {
   var greeting = "Hello ";
-
-  people.forEach(function(person) {
-    greeting = greeting + person;
+  var myarray = []
+  people.forEach(function (person) {
+    myarray.push(greeting + person);
   });
 
-  return greeting;
+  return myarray;
 }
 
 module.exports = greetPeople;

II.write-tests/01-greet-people/greet-people.test.js

@@ -1,5 +1,12 @@
-test("print list of names prefixed with Hello", function() {
+const greetPeople = require("./greet-people.js")
+
+test("print list of names prefixed with Hello", function () {
   // Arrange
+  const mentors = ['Irina', 'Ashleigh', 'Etza'];
+  const expected = ['Hello Irina', 'Hello Ashleigh', 'Hello Etza'];
   // Act
+  const result = greetPeople(mentors)
+
   // Assert
+  expect(result).toEqual(expected)
 });

II.write-tests/02-remove-vowels/remove-vowels.js

@@ -3,17 +3,17 @@ function removeVowels(word) {
 
   var result = [];
 
-  characters.forEach(function(character) {
+  characters.forEach(function (character) {
     if (
-      character === "a" ||
-      character === "o" ||
-      character === "i" ||
-      character === "e" ||
-      character === "u"
+      character.toLowerCase() === "a" ||
+      character.toLowerCase() === "o" ||
+      character.toLowerCase() === "i" ||
+      character.toLowerCase() === "e" ||
+      character.toLowerCase() === "u"
     ) {
-      result.push(character);
+      result.push("");
     } else {
-      result.push("_");
+      result.push(character);
     }
   });
 

II.write-tests/02-remove-vowels/remove-vowels.test.js

@@ -1,5 +1,10 @@
-test("remove vowels from word", function() {
+const removeVowels = require("./remove-vowels.js")
+
+test("remove vowels from word", function () {
   // Arrange
+  const expected = "sml"
   // Act
+  const result = removeVowels('samuel');
   // Assert
+  expect(result).toEqual(expected)
 });

II.write-tests/03-remove-vowels-from-array/remove-vowels-in-array.test.js

@@ -1,7 +1,13 @@
-test("remove vowels from all words in array", function() {
+const removeVowelsForWords = require("./remove-vowels-in-array")
+
+test("remove vowels from all words in array", function () {
   // Arrange
+  const expected = ["rn", "tz", "Dnl"]
+  const input = ["Irina", "Etza", "Daniel"]
   // Act
+  const result = removeVowelsForWords(input)
   // Assert
+  expect(result).toEqual(expected)
 });
 
 // example

II.write-tests/04-numbers-greater-than-10/numbersGreaterThan10.test.js

@@ -1,6 +1,13 @@
 var largerThanTen = require("./numbersGreaterThan10");
 
-test("Get numbers greater than 10", function() {});
+test("Get numbers greater than 10", function () {
+    const input = [4, 10, 32, 9, 21],
+        expected = [32, 21],
+        result = largerThanTen(input)
+
+    expect(result).toEqual(expected)
+});
+
 
 // input: [4, 10, 32, 9, 21];
 // expected output: [32, 21];

II.write-tests/05-get-second-third-smallest/get-second-third.test.js

@@ -1,3 +1,12 @@
+const getSecondThird = require("./get-second-third")
+
+test("Move the second to the third position", function () { 
+    const input = [90, 5, 11, 8, 6],
+    expected = [6, 8],
+    result = getSecondThird(input)
+
+    expect(result).toEqual(expected)
+});
 // example
 // input = [90, 5, 11, 8, 6];
 // expected output = [6, 8];

II.write-tests/06-largest-number/largest-number.test.js

@@ -1,3 +1,10 @@
+const largestNumber = require("./largest-number")
+test("Return the largest Number", function () {
+    const input = [3, 21, 88, 4, 36],
+        expected = 88,
+        result = largestNumber(input)
+    expect(result).toEqual(expected)
+})
 // example
 // input: [3, 21, 88, 4, 36];
 // expected: 88;

II.write-tests/07-get-even-numbers/get-even-numbers.test.js

@@ -1,3 +1,12 @@
+const getEvenNumbers = require("./get-even-numbers")
+
+test("the even number", function () {
+    const input = [22, 13, 73, 82, 4],
+        expected = [22, 82, 4],
+        result = getEvenNumbers(input)
+    expect(result).toEqual(expected)
+})
+
 // example
 // input: [22, 13, 73, 82, 4];
 // expected: [22, 82, 4];

II.write-tests/09-test-async/async-1.test.js

@@ -1,12 +1,12 @@
 const getRepos = require("./async-1");
 
-test("gets a list of repositories names", function() {
+test("gets a list of repositories names", function () {
   // arrange
   var url = "https://api.github.com/users/kabaros/repos";
   // act
-  return getRepos(url).then(function(result) {
+  return getRepos(url).then(function (result) {
     // assert
-    expect(result).toContain("js-exercises");
-    expect(result).toContain("dom-ajax-repo");
+    expect(result).toContain("master-reference");
+    expect(result).toContain("sword");
   });
 });