Manchester | 26-ITP-May | Szidonia Bodo | Sprint 3 | Implementing | Practice TDD | Dead code

Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js

@@ -15,7 +15,19 @@
 // execute the code to ensure all tests pass.
 
 function getAngleType(angle) {
-  // TODO: Implement this function
+  if (angle > 0 && angle < 90) {
+    return "Acute angle";
+  } else if (angle === 90) {
+    return "Right angle";
+  } else if (angle > 90 && angle < 180) {
+    return "Obtuse angle";
+  } else if (angle === 180) {
+    return "Straight angle";
+  } else if (angle > 180 && angle < 360) {
+    return "Reflex angle";
+  } else {
+    return "Invalid angle";
+  }
 }
 
 // The line below allows us to load the getAngleType function into tests in other files.
@@ -34,4 +46,30 @@ function assertEquals(actualOutput, targetOutput) {
 // TODO: Write tests to cover all cases, including boundary and invalid cases.
 // Example: Identify Right Angles
 const right = getAngleType(90);
+const acute = getAngleType(66);
+const obtuse = getAngleType(109);
+const straight = getAngleType(180);
+const reflex = getAngleType(222);
+const invalid = getAngleType(366);
+const boundary = getAngleType(0);
+const invalidBoundary = getAngleType(360)
+const acuteBoundary = getAngleType(1)
+const acuteSecondBoundary = getAngleType(89)
+const obtuseBoundary = getAngleType(91)
+const reflexBoundary = getAngleType(181)
+
+
+
 assertEquals(right, "Right angle");
+assertEquals(acute, "Acute angle");
+assertEquals(obtuse, "Obtuse angle");
+assertEquals(straight, "Straight angle");
+assertEquals(reflex, "Reflex angle")
+assertEquals(invalid, "Invalid angle")
+assertEquals(invalidBoundary, "Invalid angle")
+assertEquals(acuteBoundary, "Acute angle")
+assertEquals(acuteSecondBoundary, "Acute angle")
+assertEquals(obtuseBoundary, "Obtuse angle")
+assertEquals(reflexBoundary, "Reflex angle")
+
+

Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js

@@ -11,7 +11,11 @@
 // execute the code to ensure all tests pass.
 
 function isProperFraction(numerator, denominator) {
-  // TODO: Implement this function
+  if (numerator <= denominator && numerator > 0) {
+    return true;
+  } else {
+    return false;
+  }
 }
 
 // The line below allows us to load the isProperFraction function into tests in other files.
@@ -31,3 +35,10 @@ function assertEquals(actualOutput, targetOutput) {
 
 // Example: 1/2 is a proper fraction
 assertEquals(isProperFraction(1, 2), true);
+assertEquals(isProperFraction(0, 4), false);
+assertEquals(isProperFraction(5, 5), true);
+assertEquals(isProperFraction(-8, 2), false);
+assertEquals(isProperFraction(4, 0), false);
+assertEquals(isProperFraction(0, 0), false);
+assertEquals(isProperFraction(-2, -2), false)
+

Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js

@@ -22,7 +22,16 @@
 // execute the code to ensure all tests pass.
 
 function getCardValue(card) {
-  // TODO: Implement this function
+  let removeSuit = card.slice(0, -1);
+  if (removeSuit === "A") {
+    return 11;
+  } else if (removeSuit === "J" || removeSuit === "Q" ||removeSuit === "K") {
+    return 10;
+  } else if (removeSuit > 1 && removeSuit < 11) {
+    return Number(removeSuit);
+  } else {
+    throw new Error("Error")
+  }
 }
 
 // The line below allows us to load the getCardValue function into tests in other files.
@@ -40,15 +49,27 @@ function assertEquals(actualOutput, targetOutput) {
 // TODO: Write tests to cover all outcomes, including throwing errors for invalid cards.
 // Examples:
 assertEquals(getCardValue("9♠"), 9);
+assertEquals(getCardValue("A♠"), 11);
+assertEquals(getCardValue("J♦"), 10);
+assertEquals(getCardValue("7♥"), 7);
+assertEquals(getCardValue("A♥"), 11);
+assertEquals(getCardValue("8♥"), 8);
 
 // Handling invalid cards
 try {
   getCardValue("invalid");
-
   // This line will not be reached if an error is thrown as expected
   console.error("Error was not thrown for invalid card 😢");
 } catch (e) {
   console.log("Error thrown for invalid card 🎉");
 }
 
 // What other invalid card cases can you think of?
+try {
+  getCardValue("15♥");
+  // This line will not be reached if an error is thrown as expected
+  console.error("Error was not thrown for invalid card 😢");
+} catch (e) {
+  console.log("Error thrown for invalid card 🎉");
+}
+

Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js

@@ -14,7 +14,33 @@ test(`should return "Acute angle" when (0 < angle < 90)`, () => {
 });
 
 // Case 2: Right angle
+test(`should return "Right angle" when (angle = 90)`, () => {
+  // Test various acute angles, including boundary cases
+  expect(getAngleType(90)).toEqual("Right angle");
+});
 // Case 3: Obtuse angles
+test(`should return "Obtuse angle" when (angle < 180, angle > 90)`, () => {
+  // Test various acute angles, including boundary cases
+  expect(getAngleType(91)).toEqual("Obtuse angle");
+  expect(getAngleType(179)).toEqual("Obtuse angle");
+  expect(getAngleType(99)).toEqual("Obtuse angle");
+});
 // Case 4: Straight angle
+test(`should return "Straight angle" when (angle = 180)`, () => {
+  // Test various acute angles, including boundary cases
+  expect(getAngleType(180)).toEqual("Straight angle");
+});
 // Case 5: Reflex angles
+test(`should return "Reflex angle" when (angle < 360, angle > 180)`, () => {
+  // Test various acute angles, including boundary cases
+  expect(getAngleType(189)).toEqual("Reflex angle");
+  expect(getAngleType(359)).toEqual("Reflex angle");
+  expect(getAngleType(199)).toEqual("Reflex angle");
+});
 // Case 6: Invalid angles
+test(`should return "Invalid angle" when (angle > 361, angle < 0)`, () => {
+  // Test various acute angles, including boundary cases
+  expect(getAngleType(505)).toEqual("Invalid angle");
+  expect(getAngleType(0)).toEqual("Invalid angle");
+  expect(getAngleType(699)).toEqual("Invalid angle");
+});

Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js

@@ -8,3 +8,18 @@ const isProperFraction = require("../implement/2-is-proper-fraction");
 test(`should return false when denominator is zero`, () => {
   expect(isProperFraction(1, 0)).toEqual(false);
 });
+test(`should return true when denominator is smaller or equal to the numerator`,() =>{
+  expect(isProperFraction(2, 4)).toEqual(true);
+  expect(isProperFraction(3, 3)).toEqual(true);
+  expect(isProperFraction(5, 10)).toEqual(true);
+  expect(isProperFraction(2, 2)).toEqual(true);
+});
+
+test(`should return false when denominator is bigger than the numerator`, () => {
+  expect(isProperFraction(5, -2)).toEqual(false);
+  expect(isProperFraction(-1, 0)).toEqual(false);
+  expect(isProperFraction(12, 4)).toEqual(false);
+
+});
+
+

Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js

@@ -9,6 +9,29 @@ test(`Should return 11 when given an ace card`, () => {
   expect(getCardValue("A♠")).toEqual(11);
 });
 
+test(`Should return 10 when given a Joker card`, () => {
+  expect(getCardValue("J♥")).toEqual(10);
+});
+
+test(`Should return 2 when given the 2 card`, () => {
+  expect(getCardValue("2♠")).toEqual(2);
+});
+
+test(`Should return 10 when given a King card`, () => {
+  expect(getCardValue("K♠")).toEqual(10);
+});
+
+test(`Should return 5 when given the 5 card`, () => {
+  expect(getCardValue("5♠")).toEqual(5);
+});
+
+test(`Should return 8 when gine the 8 card`, () => {
+  expect(getCardValue("8♠")).toEqual(8);
+});
+
+test(`Should return 10 when given the Queen card`, () => {
+  expect(getCardValue("Q♠")).toEqual(10);
+});
 // Suggestion: Group the remaining test data into these categories:
 //   Number Cards (2-10)
 //   Face Cards (J, Q, K)
@@ -17,4 +40,8 @@ test(`Should return 11 when given an ace card`, () => {
 // To learn how to test whether a function throws an error as expected in Jest,
 // please refer to the Jest documentation:
 // https://jestjs.io/docs/expect#tothrowerror
-
+test(`Should not return any card should throw an error message`, () => {
+  expect(() => {
+    getCardValue("QQ♥");
+  }).toThrow();
+});

Sprint-3/2-practice-tdd/count.js

@@ -1,5 +1,12 @@
 function countChar(stringOfCharacters, findCharacter) {
-  return 5
+  let countCharacter = 0;
+  for (let i = 0; i < stringOfCharacters.length; i++){
+    if (stringOfCharacters.charAt(i) === findCharacter){
+      countCharacter += 1;
+    } 
+  }
+  return countCharacter
 }
 
+
 module.exports = countChar;

Sprint-3/2-practice-tdd/count.test.js

@@ -22,3 +22,31 @@ test("should count multiple occurrences of a character", () => {
 // And a character `char` that does not exist within `str`.
 // When the function is called with these inputs,
 // Then it should return 0, indicating that no occurrences of `char` were found.
+
+test("should count multiple occurrences of a character", () => {
+  const str = "banana";
+  const char = "n";
+  const count = countChar(str, char);
+  expect(count).toEqual(2);
+});
+
+test("should count multiple occurrences of a character", () => {
+  const str = "appleTree";
+  const char = "e";
+  const count = countChar(str, char);
+  expect(count).toEqual(3);
+});
+
+test("should count multiple occurrences of a character", () => {
+  const str = "strawberry";
+  const char = "r";
+  const count = countChar(str, char);
+  expect(count).toEqual(3);
+});
+
+test("should count multiple occurrences of a character", () => {
+  const str = "ghost";
+  const char = "o";
+  const count = countChar(str, char);
+  expect(count).toEqual(1);
+});

Sprint-3/2-practice-tdd/get-ordinal-number.js

@@ -1,5 +1,18 @@
 function getOrdinalNumber(num) {
-  return "1st";
+  if ( typeof num !==  "number" ){
+    throw new Error('Not a number')
+  }
+
+  if (num % 10 === 1 &&  num % 100 !== 11) {
+    return `${num}st`;
+  } else if (num % 10 === 2 && num % 100 !== 12 ) {
+    return `${num}nd`;
+  } else if (num % 10 === 3 && num % 100 !== 13) {
+    return `${num}rd`;
+  } else {
+
+    return `${num}th`;
+  }
 }
 
 module.exports = getOrdinalNumber;

Sprint-3/2-practice-tdd/get-ordinal-number.test.js

@@ -17,4 +17,37 @@ test("should append 'st' for numbers ending with 1, except those ending with 11"
   expect(getOrdinalNumber(1)).toEqual("1st");
   expect(getOrdinalNumber(21)).toEqual("21st");
   expect(getOrdinalNumber(131)).toEqual("131st");
+  expect(getOrdinalNumber(1101)).toEqual("1101st");
+});
+
+test("should append 'nd' for numbers ending with 2, except those ending with 12", () => {
+  expect(getOrdinalNumber(2)).toEqual("2nd");
+  expect(getOrdinalNumber(22)).toEqual("22nd");
+  expect(getOrdinalNumber(132)).toEqual("132nd");
+  expect(getOrdinalNumber(22222)).toEqual("22222nd");
+});
+
+test("should append 'rd' for numbers ending with 3, except those ending with 13", () => {
+  expect(getOrdinalNumber(3)).toEqual("3rd");
+  expect(getOrdinalNumber(23)).toEqual("23rd");
+  expect(getOrdinalNumber(133)).toEqual("133rd");
+  expect(getOrdinalNumber(33333)).toEqual("33333rd");
+});
+
+test("should append 'th' for numbers ending with 11, 12, 13", () => {
+  expect(getOrdinalNumber(11)).toEqual("11th");
+  expect(getOrdinalNumber(12)).toEqual("12th");
+  expect(getOrdinalNumber(13)).toEqual("13th");
+  expect(getOrdinalNumber(111)).toEqual("111th");
+  expect(getOrdinalNumber(112)).toEqual("112th");
+  expect(getOrdinalNumber(113)).toEqual("113th");
+  expect(getOrdinalNumber(11111)).toEqual("11111th");
+  expect(getOrdinalNumber(11112)).toEqual("11112th");
+  expect(getOrdinalNumber(11113)).toEqual("11113th");
+});
+
+test("should throw an error message", () => {
+  expect(() => {
+    getOrdinalNumber("Hello");
+  }).toThrow();
 });

Sprint-3/2-practice-tdd/repeat-str.js

@@ -1,7 +1,13 @@
-function repeatStr() {
-  // Your implementation of this function must *not* call String.prototype.repeat (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat).
-  // The goal is to re-implement that function, not to use it.
-  return "hellohellohello";
+function repeatStr(str, count) {
+  if (count < 0) {
+    throw new Error("Error");
+  }
+
+  let repeatedString = "";
+  for (let i = 0; i < count; i++) {
+    repeatedString += str;
+  }
+  return repeatedString;
 }
 
 module.exports = repeatStr;

Sprint-3/2-practice-tdd/repeat-str.test.js

@@ -20,13 +20,33 @@ test("should repeat the string count times", () => {
 // Given a target string `str` and a `count` equal to 1,
 // When the repeatStr function is called with these inputs,
 // Then it should return the original `str` without repetition.
+test("should repeat the string count times", () => {
+  const str = "what";
+  const count = 1;
+  const repeatedStr = repeatStr(str, count);
+  expect(repeatedStr).toEqual("what");
+});
 
 // Case: Handle count of 0:
 // Given a target string `str` and a `count` equal to 0,
 // When the repeatStr function is called with these inputs,
 // Then it should return an empty string.
 
+test("should return an empty string", () => {
+  const str = "hello";
+  const count = 0;
+  const repeatedStr = repeatStr(str, count);
+  expect(repeatedStr).toEqual("");
+});
+
 // Case: Handle negative count:
 // Given a target string `str` and a negative integer `count`,
 // When the repeatStr function is called with these inputs,
 // Then it should throw an error, as negative counts are not valid.
+test("should throw an error message", () => {
+  const str = "hello";
+  const count = -3;
+  expect(() => {
+    repeatStr(str, count);
+  }).toThrow();
+});

Sprint-3/3-dead-code/exercise-1.js

@@ -5,9 +5,11 @@ let testName = "Jerry";
 const greeting = "hello";
 
 function sayHello(greeting, name) {
-  const greetingStr = greeting + ", " + name + "!";
+  //const greetingStr = greeting + ", " + name + "!"; 
+  //  - removing this line as this is redundant code and duplicated inside the return 
   return `${greeting}, ${name}!`;
-  console.log(greetingStr);
+  //console.log(greetingStr); 
+  // - removing this line because after return the console.log will not execute
 }
 
 testName = "Aman";