From 20c1e4fbb46bb5233bad10592d692af531d21c7d Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 14:35:56 -0500 Subject: [PATCH 1/8] Agregando Prueba de unidad para la funcion filtrar explorers por stack para la clase explorerService --- .gitignore | 2 ++ test/services/ExplorerService.test.js | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/.gitignore b/.gitignore index 3c3629e6..6fde799b 100644 --- a/.gitignore +++ b/.gitignore @@ -1 +1,3 @@ node_modules +*.json +!package.json \ No newline at end of file diff --git a/test/services/ExplorerService.test.js b/test/services/ExplorerService.test.js index 4313f192..b96d2fc0 100644 --- a/test/services/ExplorerService.test.js +++ b/test/services/ExplorerService.test.js @@ -6,5 +6,10 @@ describe("Tests para ExplorerService", () => { const explorersInNode = ExplorerService.filterByMission(explorers, "node"); expect(explorersInNode.length).toBe(1); }); + test('Requerimiento 2: Mostrar todos los explorers por stack',()=>{ + const explorers = [{name:'ajolote1',stack:['node','java']},{name:'ajolote2',stack:['node','java']},{name:'ajolote3',stack:'node'},{name:'ajolote4',stack:'java'}]; + const explorerInNode = ExplorerService.filterByStack(explorers,'node'); + expect(explorerInNode.name).toContain('ajolote1') + }) }); From c60cae1f069d4a75b2696502f492406a6ef9bf91 Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 14:49:28 -0500 Subject: [PATCH 2/8] Agregando cambios a la prueba automatizada en Windows --- .github/workflows/test.yml | 38 ++++++++++++++++++++++++++++++-------- 1 file changed, 30 insertions(+), 8 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index f5666317..59d4c1cc 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,11 +1,33 @@ -name: Run Tests in my project every push on GitHub - -on: [push, pull_request] - +name: Jest +on: push jobs: - build: + test: runs-on: ubuntu-latest steps: - - uses: actions/checkout@v1 - - name: Run Jest - uses: stefanoeb/jest-action@1.0.3 + - uses: actions/checkout@v2 + - name: Setup Node.js + uses: actions/setup-node@v1 + with: + node-version: "12" + + # Speed up subsequent runs with caching + - name: Cache node modules + uses: actions/cache@v2 + env: + cache-name: cache-node-modules + with: + # npm cache files are stored in `~/.npm` on Linux/macOS + path: ~/.npm + key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }} + restore-keys: | + ${{ runner.os }}-build-${{ env.cache-name }}- + ${{ runner.os }}-build- + ${{ runner.os }}- + + # Install required deps for action + - name: Install Dependencies + run: npm install + + # Finally, run our tests + - name: Run the tests + run: npm test \ No newline at end of file From dc65f4ca408c744d23e316064625a0e97d2e50c9 Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 14:52:19 -0500 Subject: [PATCH 3/8] Agregando cambios al Github Actions --- package.json | 18 +++++++++++++++--- 1 file changed, 15 insertions(+), 3 deletions(-) diff --git a/package.json b/package.json index 119436d2..65456584 100644 --- a/package.json +++ b/package.json @@ -1,10 +1,10 @@ { "name": "fizzbuzz", "version": "1.0.0", - "description": "", + "description": "1. Instalar dependencia:", "main": "index.js", "scripts": { - "test": "node ./node_modules/.bin/jest", + "test": "node --experimental-vm-modules ./node_modules/jest/bin/jest", "linter": "node ./node_modules/eslint/bin/eslint.js", "linter-fix": "node ./node_modules/eslint/bin/eslint.js . --fix", "server": "node ./lib/server.js" @@ -18,5 +18,17 @@ }, "dependencies": { "express": "^4.17.3" - } + }, + "directories": { + "lib": "lib", + "test": "test" + }, + "repository": { + "type": "git", + "url": "git+https://github.com/MoisesMendozaS01/fizzbuzz.git" + }, + "bugs": { + "url": "https://github.com/MoisesMendozaS01/fizzbuzz/issues" + }, + "homepage": "https://github.com/MoisesMendozaS01/fizzbuzz#readme" } From 48b472ec2f68370e22fbb9cb508e08d42a3f733b Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 15:35:48 -0500 Subject: [PATCH 4/8] Agregando funcion filtrar explorers por stack en la clase explorerServices --- lib/services/ExplorerService.js | 4 ++++ test/services/ExplorerService.test.js | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/services/ExplorerService.js b/lib/services/ExplorerService.js index 9fef9574..18743d22 100644 --- a/lib/services/ExplorerService.js +++ b/lib/services/ExplorerService.js @@ -15,6 +15,10 @@ class ExplorerService { const explorersUsernames = explorersByMission.map((explorer) => explorer.githubUsername); return explorersUsernames; } + static filterByStack(explorers,stack){ + const explorersByStack = explorers.filter((explorer)=>explorer.stack.includes(stack)); + return explorersByStack + } } diff --git a/test/services/ExplorerService.test.js b/test/services/ExplorerService.test.js index b96d2fc0..737bfe28 100644 --- a/test/services/ExplorerService.test.js +++ b/test/services/ExplorerService.test.js @@ -9,7 +9,7 @@ describe("Tests para ExplorerService", () => { test('Requerimiento 2: Mostrar todos los explorers por stack',()=>{ const explorers = [{name:'ajolote1',stack:['node','java']},{name:'ajolote2',stack:['node','java']},{name:'ajolote3',stack:'node'},{name:'ajolote4',stack:'java'}]; const explorerInNode = ExplorerService.filterByStack(explorers,'node'); - expect(explorerInNode.name).toContain('ajolote1') + expect(explorerInNode).not.toBeUndefined() }) }); From d765be2bfbc401883b000688777036a203044d63 Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 15:43:58 -0500 Subject: [PATCH 5/8] Agregando Prueba de unidad para llamar a la funcion get explorers by stack de la clase ExplorerController --- test/controllers/ExplorerController.test.js | 10 ++++++++++ 1 file changed, 10 insertions(+) create mode 100644 test/controllers/ExplorerController.test.js diff --git a/test/controllers/ExplorerController.test.js b/test/controllers/ExplorerController.test.js new file mode 100644 index 00000000..1ebe66b3 --- /dev/null +++ b/test/controllers/ExplorerController.test.js @@ -0,0 +1,10 @@ +const { TestWatcher } = require('jest'); +const ExplorerController = require('./../../../fizzbuzz/lib/controllers/ExplorerController'); + +describe('Unit Test for ExplorerController',()=>{ + test('Parte 1:Prueba de unidad para llamar la funcion filtrar por stack',()=>{ + const ExplorersInNodeJavascript = ExplorerController.getExplorersByStack('javascript'); + + expect(ExplorersInNodeJavascript).not.toBeUndefined(); + }); +}) \ No newline at end of file From c5d808d2889b642b1a32d6fb8e0024402c179869 Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 18:03:37 -0500 Subject: [PATCH 6/8] Agregando funcion GetExplorersByStack en la clase ExplorerController --- lib/controllers/ExplorerController.js | 4 ++++ lib/services/ExplorerService.js | 4 ++-- test/services/ExplorerService.test.js | 2 +- 3 files changed, 7 insertions(+), 3 deletions(-) diff --git a/lib/controllers/ExplorerController.js b/lib/controllers/ExplorerController.js index 85590a6d..e3d1ccfb 100644 --- a/lib/controllers/ExplorerController.js +++ b/lib/controllers/ExplorerController.js @@ -21,6 +21,10 @@ class ExplorerController{ const explorers = Reader.readJsonFile("explorers.json"); return ExplorerService.getAmountOfExplorersByMission(explorers, mission); } + static getExplorersByStack(stack){ + const explorers = Reader.readJsonFile("explorers.json"); + return ExplorerService.filterByStack(explorers,stack); + } } module.exports = ExplorerController; diff --git a/lib/services/ExplorerService.js b/lib/services/ExplorerService.js index 18743d22..5fba3ca8 100644 --- a/lib/services/ExplorerService.js +++ b/lib/services/ExplorerService.js @@ -15,8 +15,8 @@ class ExplorerService { const explorersUsernames = explorersByMission.map((explorer) => explorer.githubUsername); return explorersUsernames; } - static filterByStack(explorers,stack){ - const explorersByStack = explorers.filter((explorer)=>explorer.stack.includes(stack)); + static filterByStack(explorers,stacks){ + const explorersByStack = explorers.filter((explorer)=>explorer.stacks.includes(stacks)); return explorersByStack } diff --git a/test/services/ExplorerService.test.js b/test/services/ExplorerService.test.js index 737bfe28..a47f78c8 100644 --- a/test/services/ExplorerService.test.js +++ b/test/services/ExplorerService.test.js @@ -7,7 +7,7 @@ describe("Tests para ExplorerService", () => { expect(explorersInNode.length).toBe(1); }); test('Requerimiento 2: Mostrar todos los explorers por stack',()=>{ - const explorers = [{name:'ajolote1',stack:['node','java']},{name:'ajolote2',stack:['node','java']},{name:'ajolote3',stack:'node'},{name:'ajolote4',stack:'java'}]; + const explorers = [{name:'ajolote1',stacks:['node','java']},{name:'ajolote2',stacks:['node','java']},{name:'ajolote3',stacks:'node'},{name:'ajolote4',stacks:'java'}]; const explorerInNode = ExplorerService.filterByStack(explorers,'node'); expect(explorerInNode).not.toBeUndefined() }) From 51187e6b7b167cc7c4e6b1400995163db8e82b51 Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 18:10:42 -0500 Subject: [PATCH 7/8] Agregando EndPoint /v1/explorers/stack/:mission que imprime una lista de explorers por Stack --- lib/server.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/server.js b/lib/server.js index 9860b437..b3a93134 100644 --- a/lib/server.js +++ b/lib/server.js @@ -31,7 +31,11 @@ app.get("/v1/fizzbuzz/:score", (request, response) => { const fizzbuzzTrick = ExplorerController.applyFizzbuzz(score); response.json({score: score, trick: fizzbuzzTrick}); }); - +app.get("/v1/explorers/stack/:mission", (request, response) => { + const mission = request.params.mission; + const usersTrick = ExplorerController.getExplorersByStack(mission); + response.json(usersTrick); +}); app.listen(port, () => { console.log(`FizzBuzz API in localhost:${port}`); }); From d6ad94e9b1d45a13cabd4e4e199633711146bfa1 Mon Sep 17 00:00:00 2001 From: = <=> Date: Wed, 18 May 2022 18:22:49 -0500 Subject: [PATCH 8/8] Agregando dependencia Linter y corrigiendo errores --- .eslintrc.js | 3 +- lib/services/ExplorerService.js | 2 +- package-lock.json | 90 ++++++++++----------- package.json | 2 +- test/controllers/ExplorerController.test.js | 11 ++- test/services/ExplorerService.test.js | 10 +-- 6 files changed, 58 insertions(+), 60 deletions(-) diff --git a/.eslintrc.js b/.eslintrc.js index 95273503..3c2879d0 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -3,14 +3,13 @@ module.exports = { "browser": true, "commonjs": true, "es2021": true, - "jest": true + "jest":true }, "extends": "eslint:recommended", "parserOptions": { "ecmaVersion": "latest" }, "rules": { - "no-unused-vars": "off", indent: ["error", 4], "linebreak-style": ["error", "unix"], quotes: ["error", "double"], diff --git a/lib/services/ExplorerService.js b/lib/services/ExplorerService.js index 5fba3ca8..ff94cd84 100644 --- a/lib/services/ExplorerService.js +++ b/lib/services/ExplorerService.js @@ -17,7 +17,7 @@ class ExplorerService { } static filterByStack(explorers,stacks){ const explorersByStack = explorers.filter((explorer)=>explorer.stacks.includes(stacks)); - return explorersByStack + return explorersByStack; } } diff --git a/package-lock.json b/package-lock.json index bb4f2aae..bd0fc832 100644 --- a/package-lock.json +++ b/package-lock.json @@ -12,7 +12,7 @@ "express": "^4.17.3" }, "devDependencies": { - "eslint": "^8.14.0", + "eslint": "^8.15.0", "jest": "^27.5.1" } }, @@ -567,19 +567,19 @@ "dev": true }, "node_modules/@eslint/eslintrc": { - "version": "1.2.2", - "resolved": "https://registry.npmjs.org/@eslint/eslintrc/-/eslintrc-1.2.2.tgz", - "integrity": "sha512-lTVWHs7O2hjBFZunXTZYnYqtB9GakA1lnxIf+gKq2nY5gxkkNi/lQvveW6t8gFdOHTg6nG50Xs95PrLqVpcaLg==", + "version": "1.2.3", + "resolved": "https://registry.npmjs.org/@eslint/eslintrc/-/eslintrc-1.2.3.tgz", + "integrity": "sha512-uGo44hIwoLGNyduRpjdEpovcbMdd+Nv7amtmJxnKmI8xj6yd5LncmSwDa5NgX/41lIFJtkjD6YdVfgEzPfJ5UA==", "dev": true, "dependencies": { "ajv": "^6.12.4", "debug": "^4.3.2", - "espree": "^9.3.1", + "espree": "^9.3.2", "globals": "^13.9.0", "ignore": "^5.2.0", "import-fresh": "^3.2.1", "js-yaml": "^4.1.0", - "minimatch": "^3.0.4", + "minimatch": "^3.1.2", "strip-json-comments": "^3.1.1" }, "engines": { @@ -593,9 +593,9 @@ "dev": true }, "node_modules/@eslint/eslintrc/node_modules/globals": { - "version": "13.13.0", - "resolved": "https://registry.npmjs.org/globals/-/globals-13.13.0.tgz", - "integrity": "sha512-EQ7Q18AJlPwp3vUDL4mKA0KXrXyNIQyWon6T6XQiBQF0XHvRsiCSrWmmeATpUzdJN2HhWZU6Pdl0a9zdep5p6A==", + "version": "13.15.0", + "resolved": "https://registry.npmjs.org/globals/-/globals-13.15.0.tgz", + "integrity": "sha512-bpzcOlgDhMG070Av0Vy5Owklpv1I6+j96GhUI7Rh7IzDCKLzboflLrrfqMu8NquDbiR4EOQk7XzJwqVJxicxog==", "dev": true, "dependencies": { "type-fest": "^0.20.2" @@ -1094,9 +1094,9 @@ } }, "node_modules/acorn": { - "version": "8.7.0", - "resolved": "https://registry.npmjs.org/acorn/-/acorn-8.7.0.tgz", - "integrity": "sha512-V/LGr1APy+PXIwKebEWrkZPwoeoF+w1jiOBUmuxuiUIaOHtob8Qc9BTrYo7VuI5fR8tqsy+buA2WFooR5olqvQ==", + "version": "8.7.1", + "resolved": "https://registry.npmjs.org/acorn/-/acorn-8.7.1.tgz", + "integrity": "sha512-Xx54uLJQZ19lKygFXOWsscKUbsBZW0CPykPhVQdhIeIwrbPmJzqeASDInc8nKBnp/JT6igTs82qPXz069H8I/A==", "dev": true, "bin": { "acorn": "bin/acorn" @@ -1911,12 +1911,12 @@ } }, "node_modules/eslint": { - "version": "8.14.0", - "resolved": "https://registry.npmjs.org/eslint/-/eslint-8.14.0.tgz", - "integrity": "sha512-3/CE4aJX7LNEiE3i6FeodHmI/38GZtWCsAtsymScmzYapx8q1nVVb+eLcLSzATmCPXw5pT4TqVs1E0OmxAd9tw==", + "version": "8.15.0", + "resolved": "https://registry.npmjs.org/eslint/-/eslint-8.15.0.tgz", + "integrity": "sha512-GG5USZ1jhCu8HJkzGgeK8/+RGnHaNYZGrGDzUtigK3BsGESW/rs2az23XqE0WVwDxy1VRvvjSSGu5nB0Bu+6SA==", "dev": true, "dependencies": { - "@eslint/eslintrc": "^1.2.2", + "@eslint/eslintrc": "^1.2.3", "@humanwhocodes/config-array": "^0.9.2", "ajv": "^6.10.0", "chalk": "^4.0.0", @@ -1927,7 +1927,7 @@ "eslint-scope": "^7.1.1", "eslint-utils": "^3.0.0", "eslint-visitor-keys": "^3.3.0", - "espree": "^9.3.1", + "espree": "^9.3.2", "esquery": "^1.4.0", "esutils": "^2.0.2", "fast-deep-equal": "^3.1.3", @@ -1943,7 +1943,7 @@ "json-stable-stringify-without-jsonify": "^1.0.1", "levn": "^0.4.1", "lodash.merge": "^4.6.2", - "minimatch": "^3.0.4", + "minimatch": "^3.1.2", "natural-compare": "^1.4.0", "optionator": "^0.9.1", "regexpp": "^3.2.0", @@ -2120,13 +2120,13 @@ } }, "node_modules/espree": { - "version": "9.3.1", - "resolved": "https://registry.npmjs.org/espree/-/espree-9.3.1.tgz", - "integrity": "sha512-bvdyLmJMfwkV3NCRl5ZhJf22zBFo1y8bYh3VYb+bfzqNB4Je68P2sSuXyuFquzWLebHpNd2/d5uv7yoP9ISnGQ==", + "version": "9.3.2", + "resolved": "https://registry.npmjs.org/espree/-/espree-9.3.2.tgz", + "integrity": "sha512-D211tC7ZwouTIuY5x9XnS0E9sWNChB7IYKX/Xp5eQj3nFXhqmiUDB9q27y76oFl8jTg3pXcQx/bpxMfs3CIZbA==", "dev": true, "dependencies": { - "acorn": "^8.7.0", - "acorn-jsx": "^5.3.1", + "acorn": "^8.7.1", + "acorn-jsx": "^5.3.2", "eslint-visitor-keys": "^3.3.0" }, "engines": { @@ -5451,19 +5451,19 @@ "dev": true }, "@eslint/eslintrc": { - "version": "1.2.2", - "resolved": "https://registry.npmjs.org/@eslint/eslintrc/-/eslintrc-1.2.2.tgz", - "integrity": "sha512-lTVWHs7O2hjBFZunXTZYnYqtB9GakA1lnxIf+gKq2nY5gxkkNi/lQvveW6t8gFdOHTg6nG50Xs95PrLqVpcaLg==", + "version": "1.2.3", + "resolved": "https://registry.npmjs.org/@eslint/eslintrc/-/eslintrc-1.2.3.tgz", + "integrity": "sha512-uGo44hIwoLGNyduRpjdEpovcbMdd+Nv7amtmJxnKmI8xj6yd5LncmSwDa5NgX/41lIFJtkjD6YdVfgEzPfJ5UA==", "dev": true, "requires": { "ajv": "^6.12.4", "debug": "^4.3.2", - "espree": "^9.3.1", + "espree": "^9.3.2", "globals": "^13.9.0", "ignore": "^5.2.0", "import-fresh": "^3.2.1", "js-yaml": "^4.1.0", - "minimatch": "^3.0.4", + "minimatch": "^3.1.2", "strip-json-comments": "^3.1.1" }, "dependencies": { @@ -5474,9 +5474,9 @@ "dev": true }, "globals": { - "version": "13.13.0", - "resolved": "https://registry.npmjs.org/globals/-/globals-13.13.0.tgz", - "integrity": "sha512-EQ7Q18AJlPwp3vUDL4mKA0KXrXyNIQyWon6T6XQiBQF0XHvRsiCSrWmmeATpUzdJN2HhWZU6Pdl0a9zdep5p6A==", + "version": "13.15.0", + "resolved": "https://registry.npmjs.org/globals/-/globals-13.15.0.tgz", + "integrity": "sha512-bpzcOlgDhMG070Av0Vy5Owklpv1I6+j96GhUI7Rh7IzDCKLzboflLrrfqMu8NquDbiR4EOQk7XzJwqVJxicxog==", "dev": true, "requires": { "type-fest": "^0.20.2" @@ -5895,9 +5895,9 @@ } }, "acorn": { - "version": "8.7.0", - "resolved": "https://registry.npmjs.org/acorn/-/acorn-8.7.0.tgz", - "integrity": "sha512-V/LGr1APy+PXIwKebEWrkZPwoeoF+w1jiOBUmuxuiUIaOHtob8Qc9BTrYo7VuI5fR8tqsy+buA2WFooR5olqvQ==", + "version": "8.7.1", + "resolved": "https://registry.npmjs.org/acorn/-/acorn-8.7.1.tgz", + "integrity": "sha512-Xx54uLJQZ19lKygFXOWsscKUbsBZW0CPykPhVQdhIeIwrbPmJzqeASDInc8nKBnp/JT6igTs82qPXz069H8I/A==", "dev": true }, "acorn-globals": { @@ -6511,12 +6511,12 @@ } }, "eslint": { - "version": "8.14.0", - "resolved": "https://registry.npmjs.org/eslint/-/eslint-8.14.0.tgz", - "integrity": "sha512-3/CE4aJX7LNEiE3i6FeodHmI/38GZtWCsAtsymScmzYapx8q1nVVb+eLcLSzATmCPXw5pT4TqVs1E0OmxAd9tw==", + "version": "8.15.0", + "resolved": "https://registry.npmjs.org/eslint/-/eslint-8.15.0.tgz", + "integrity": "sha512-GG5USZ1jhCu8HJkzGgeK8/+RGnHaNYZGrGDzUtigK3BsGESW/rs2az23XqE0WVwDxy1VRvvjSSGu5nB0Bu+6SA==", "dev": true, "requires": { - "@eslint/eslintrc": "^1.2.2", + "@eslint/eslintrc": "^1.2.3", "@humanwhocodes/config-array": "^0.9.2", "ajv": "^6.10.0", "chalk": "^4.0.0", @@ -6527,7 +6527,7 @@ "eslint-scope": "^7.1.1", "eslint-utils": "^3.0.0", "eslint-visitor-keys": "^3.3.0", - "espree": "^9.3.1", + "espree": "^9.3.2", "esquery": "^1.4.0", "esutils": "^2.0.2", "fast-deep-equal": "^3.1.3", @@ -6543,7 +6543,7 @@ "json-stable-stringify-without-jsonify": "^1.0.1", "levn": "^0.4.1", "lodash.merge": "^4.6.2", - "minimatch": "^3.0.4", + "minimatch": "^3.1.2", "natural-compare": "^1.4.0", "optionator": "^0.9.1", "regexpp": "^3.2.0", @@ -6664,13 +6664,13 @@ "dev": true }, "espree": { - "version": "9.3.1", - "resolved": "https://registry.npmjs.org/espree/-/espree-9.3.1.tgz", - "integrity": "sha512-bvdyLmJMfwkV3NCRl5ZhJf22zBFo1y8bYh3VYb+bfzqNB4Je68P2sSuXyuFquzWLebHpNd2/d5uv7yoP9ISnGQ==", + "version": "9.3.2", + "resolved": "https://registry.npmjs.org/espree/-/espree-9.3.2.tgz", + "integrity": "sha512-D211tC7ZwouTIuY5x9XnS0E9sWNChB7IYKX/Xp5eQj3nFXhqmiUDB9q27y76oFl8jTg3pXcQx/bpxMfs3CIZbA==", "dev": true, "requires": { - "acorn": "^8.7.0", - "acorn-jsx": "^5.3.1", + "acorn": "^8.7.1", + "acorn-jsx": "^5.3.2", "eslint-visitor-keys": "^3.3.0" } }, diff --git a/package.json b/package.json index 65456584..27f8cfef 100644 --- a/package.json +++ b/package.json @@ -13,7 +13,7 @@ "author": "", "license": "ISC", "devDependencies": { - "eslint": "^8.14.0", + "eslint": "^8.15.0", "jest": "^27.5.1" }, "dependencies": { diff --git a/test/controllers/ExplorerController.test.js b/test/controllers/ExplorerController.test.js index 1ebe66b3..9580070a 100644 --- a/test/controllers/ExplorerController.test.js +++ b/test/controllers/ExplorerController.test.js @@ -1,10 +1,9 @@ -const { TestWatcher } = require('jest'); -const ExplorerController = require('./../../../fizzbuzz/lib/controllers/ExplorerController'); +const ExplorerController = require("./../../../fizzbuzz/lib/controllers/ExplorerController"); -describe('Unit Test for ExplorerController',()=>{ - test('Parte 1:Prueba de unidad para llamar la funcion filtrar por stack',()=>{ - const ExplorersInNodeJavascript = ExplorerController.getExplorersByStack('javascript'); +describe("Unit Test for ExplorerController",()=>{ + test("Parte 1:Prueba de unidad para llamar la funcion filtrar por stack",()=>{ + const ExplorersInNodeJavascript = ExplorerController.getExplorersByStack("javascript"); expect(ExplorersInNodeJavascript).not.toBeUndefined(); }); -}) \ No newline at end of file +}); \ No newline at end of file diff --git a/test/services/ExplorerService.test.js b/test/services/ExplorerService.test.js index a47f78c8..8139a689 100644 --- a/test/services/ExplorerService.test.js +++ b/test/services/ExplorerService.test.js @@ -6,10 +6,10 @@ describe("Tests para ExplorerService", () => { const explorersInNode = ExplorerService.filterByMission(explorers, "node"); expect(explorersInNode.length).toBe(1); }); - test('Requerimiento 2: Mostrar todos los explorers por stack',()=>{ - const explorers = [{name:'ajolote1',stacks:['node','java']},{name:'ajolote2',stacks:['node','java']},{name:'ajolote3',stacks:'node'},{name:'ajolote4',stacks:'java'}]; - const explorerInNode = ExplorerService.filterByStack(explorers,'node'); - expect(explorerInNode).not.toBeUndefined() - }) + test("Requerimiento 2: Mostrar todos los explorers por stack",()=>{ + const explorers = [{name:"ajolote1",stacks:["node","java"]},{name:"ajolote2",stacks:["node","java"]},{name:"ajolote3",stacks:"node"},{name:"ajolote4",stacks:"java"}]; + const explorerInNode = ExplorerService.filterByStack(explorers,"node"); + expect(explorerInNode).not.toBeUndefined(); + }); });