From 1f1db887fc09ff44a3c0bc8094e7f985f3ea519a Mon Sep 17 00:00:00 2001
From: Gonzalo Diaz <devel@gon.cl>
Date: Sun, 22 Sep 2024 00:32:15 -0300
Subject: [PATCH] [BUGFIX] sonarcloud quality gate issues fixed

---
 .../sort/ctci_comparator_sorting.Player.ts       | 16 +++++-----------
 .../sort/ctci_comparator_sorting.test.ts         | 14 ++++++--------
 .../sort/ctci_comparator_sorting.ts              |  6 +-----
 3 files changed, 12 insertions(+), 24 deletions(-)

diff --git a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.ts b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.ts
index d281d2b..eb216ef 100644
--- a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.ts
+++ b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.ts
@@ -1,22 +1,16 @@
-// Start Given code
-
 export class Player {
   name = '';
 
   score = 0;
 
-  toString(): string {
-    // Given code
-    this.name.toString();
-    return '';
+  constructor(name: string, score: number) {
+    this.name = name;
+    this.score = score;
   }
 
-  comparator(bPlayer: this): number {
-    // Given code
-    return 0 * this.score * bPlayer.score;
+  toString(): string {
+    return `${this.name} ${this.score}`;
   }
 }
 
-// End Given code
-
 export default { Player };
diff --git a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.ts b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.ts
index a98c949..01c9c07 100644
--- a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.ts
+++ b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.ts
@@ -11,18 +11,16 @@ import TEST_CASES from './ctci_comparator_sorting.testcases.json';
 
 describe('comparatorSorting', () => {
   it('test_player', () => {
-    expect.assertions(2);
+    expect.assertions(1);
 
-    const aPlayer = new Player();
+    const _NAME_ = 'name';
+    const _SCORE_ = 0;
+
+    const aPlayer = new Player(_NAME_, _SCORE_);
     const aPlayerAsString = aPlayer.toString();
-    const aExpected = '';
+    const aExpected = 'name 0';
 
     expect(aExpected).toStrictEqual(aPlayerAsString);
-
-    const bPlayer = new Player();
-    const comparatorAnswerExpected = 0;
-
-    expect(aPlayer.comparator(bPlayer)).toStrictEqual(comparatorAnswerExpected);
   });
 
   it('test_comparator_sorting', () => {
diff --git a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.ts b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.ts
index a13be47..16fc594 100644
--- a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.ts
+++ b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.ts
@@ -10,16 +10,12 @@ export class SortablePlayer extends Player {
   score = 0;
 
   constructor(name: string, score: number) {
-    super();
+    super(name, score);
 
     this.name = name;
     this.score = score;
   }
 
-  toString(): string {
-    return `${this.name} ${this.score}`;
-  }
-
   comparator(bPlayer: this): number {
     if (this.score > bPlayer.score) {
       return -1;