From 0a0a6452793b73913b1ca3fe5d7bd748327f77ba Mon Sep 17 00:00:00 2001
From: Gonzalo Diaz <devel@gon.cl>
Date: Tue, 24 Sep 2024 17:15:11 -0300
Subject: [PATCH 1/2] [BUGFIX] sonarcloud quality gate issues fixed:

The return value of "toString" must be used.
---
 .../sort/ctci_comparator_sorting.Player.js         | 12 +++++-------
 .../sort/ctci_comparator_sorting.js                |  4 ----
 .../sort/ctci_comparator_sorting.test.js           | 14 ++++++--------
 3 files changed, 11 insertions(+), 19 deletions(-)

diff --git a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.js b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.js
index 6d62acb..cfc6bef 100644
--- a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.js
+++ b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.Player.js
@@ -5,15 +5,13 @@ export class Player {
 
   score = 0;
 
-  toString() {
-    // Given code
-    this.name.toString();
-    return '';
+  constructor(name, score) {
+    this.name = name;
+    this.score = score;
   }
 
-  comparator(bPlayer) {
-    // Given code
-    return 0 * this.score * bPlayer.score;
+  toString() {
+    return `${this.name} ${this.score}`;
   }
 }
 
diff --git a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.js b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.js
index 7a5ada7..0ddfed6 100644
--- a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.js
+++ b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.js
@@ -16,10 +16,6 @@ export class SortablePlayer extends Player {
     this.score = score;
   }
 
-  toString() {
-    return `${this.name} ${this.score}`;
-  }
-
   comparator(bPlayer) {
     if (this.score > bPlayer.score) {
       return -1;
diff --git a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.js b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.js
index a626de1..762cc6c 100644
--- a/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.js
+++ b/src/hackerrank/interview_preparation_kit/sort/ctci_comparator_sorting.test.js
@@ -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', () => {

From e455d1afc6c17bb913c1824f6e5de27196d52ca0 Mon Sep 17 00:00:00 2001
From: Gonzalo Diaz <devel@gon.cl>
Date: Tue, 24 Sep 2024 17:16:01 -0300
Subject: [PATCH 2/2] [CONFIG] jest: dist/ directory added to exceptions.

---
 jest.config.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/jest.config.js b/jest.config.js
index 72ebd0a..45cda5e 100644
--- a/jest.config.js
+++ b/jest.config.js
@@ -159,7 +159,11 @@ const jestConfig = {
   testMatch: ['**/?(*.)+(spec|test).js?(x)'],
 
   // An array of regexp pattern strings that are matched against all test paths, matched tests are skipped
-  testPathIgnorePatterns: ['/node_modules/', '<rootDir>/coverage'],
+  testPathIgnorePatterns: [
+    '/node_modules/',
+    '<rootDir>/coverage',
+    '<rootDir>/dist'
+  ],
 
   // The regexp pattern or array of patterns that Jest uses to detect test files
   // testRegex: [],