From 6af764f88834ac2c6d6679ef8fa5ad8a568acc87 Mon Sep 17 00:00:00 2001 From: Yevhenii Kaliaiev Date: Tue, 13 Jan 2026 14:11:03 +0200 Subject: [PATCH 1/2] added solution to the task --- .github/workflows/test.yml-template | 23 ++++++++++++++++++ package-lock.json | 9 +++---- package.json | 2 +- src/app.js | 37 +++++++++++++++++++++++++++++ 4 files changed, 66 insertions(+), 5 deletions(-) create mode 100644 .github/workflows/test.yml-template diff --git a/.github/workflows/test.yml-template b/.github/workflows/test.yml-template new file mode 100644 index 0000000..bb13dfc --- /dev/null +++ b/.github/workflows/test.yml-template @@ -0,0 +1,23 @@ +name: Test + +on: + pull_request: + branches: [ master ] + +jobs: + build: + + runs-on: ubuntu-latest + + strategy: + matrix: + node-version: [20.x] + + steps: + - uses: actions/checkout@v2 + - name: Use Node.js ${{ matrix.node-version }} + uses: actions/setup-node@v1 + with: + node-version: ${{ matrix.node-version }} + - run: npm install + - run: npm test diff --git a/package-lock.json b/package-lock.json index 10f9f16..b7c274c 100644 --- a/package-lock.json +++ b/package-lock.json @@ -12,7 +12,7 @@ "devDependencies": { "@faker-js/faker": "^8.4.1", "@mate-academy/eslint-config": "latest", - "@mate-academy/scripts": "^1.8.6", + "@mate-academy/scripts": "^2.1.3", "eslint": "^8.57.0", "eslint-plugin-jest": "^28.6.0", "eslint-plugin-node": "^11.1.0", @@ -1484,10 +1484,11 @@ } }, "node_modules/@mate-academy/scripts": { - "version": "1.8.6", - "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-1.8.6.tgz", - "integrity": "sha512-b4om/whj4G9emyi84ORE3FRZzCRwRIesr8tJHXa8EvJdOaAPDpzcJ8A0sFfMsWH9NUOVmOwkBtOXDu5eZZ00Ig==", + "version": "2.1.3", + "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-2.1.3.tgz", + "integrity": "sha512-a07wHTj/1QUK2Aac5zHad+sGw4rIvcNl5lJmJpAD7OxeSbnCdyI6RXUHwXhjF5MaVo9YHrJ0xVahyERS2IIyBQ==", "dev": true, + "license": "MIT", "dependencies": { "@octokit/rest": "^17.11.2", "@types/get-port": "^4.2.0", diff --git a/package.json b/package.json index 4e4705d..1640507 100644 --- a/package.json +++ b/package.json @@ -18,7 +18,7 @@ "devDependencies": { "@faker-js/faker": "^8.4.1", "@mate-academy/eslint-config": "latest", - "@mate-academy/scripts": "^1.8.6", + "@mate-academy/scripts": "^2.1.3", "eslint": "^8.57.0", "eslint-plugin-jest": "^28.6.0", "eslint-plugin-node": "^11.1.0", diff --git a/src/app.js b/src/app.js index ad9a93a..aa69a81 100644 --- a/src/app.js +++ b/src/app.js @@ -1 +1,38 @@ +/* eslint-disable no-console */ 'use strict'; + +const fs = require('fs'); +const path = process.argv[2]; +const copyPath = process.argv[3]; + +if (!path || !copyPath) { + console.error('Error: Please provide source and destination paths'); +} + +if (path === copyPath) { + console.error('Error: Source and destination paths are the same'); +} + +try { + const stats = fs.statSync(path); + + if (!stats.isFile()) { + console.error('Error: Source path is not a file'); + } + + if (fs.existsSync(copyPath)) { + const destStats = fs.statSync(copyPath); + + if (destStats.isDirectory()) { + console.error('Error: Destination path is a directory'); + } + } + + const data = fs.readFileSync(path); + + fs.writeFileSync(copyPath, data); + + console.log(`File copied from "${path}" to "${copyPath}" successfully`); +} catch (err) { + console.error(`Error: ${err}`); +} From fb70f961f2b6e2d1626659d27e36909245af3aa1 Mon Sep 17 00:00:00 2001 From: Yevhenii Kaliaiev Date: Tue, 13 Jan 2026 14:28:23 +0200 Subject: [PATCH 2/2] added improvements according to AI comments --- src/app.js | 39 ++++++++++++++++++++++++--------------- tests/copyFiles.test.js | 1 + 2 files changed, 25 insertions(+), 15 deletions(-) diff --git a/src/app.js b/src/app.js index aa69a81..10897bf 100644 --- a/src/app.js +++ b/src/app.js @@ -2,37 +2,46 @@ 'use strict'; const fs = require('fs'); -const path = process.argv[2]; -const copyPath = process.argv[3]; +const source = process.argv[2]; +const dest = process.argv[3]; -if (!path || !copyPath) { - console.error('Error: Please provide source and destination paths'); +if (!source) { + console.error('Please provide correct source path'); + process.exit(0); } -if (path === copyPath) { - console.error('Error: Source and destination paths are the same'); +if (!dest) { + console.error('Please provide correct destination path'); + process.exit(0); +} + +if (source === dest) { + process.exit(0); } try { - const stats = fs.statSync(path); + const stats = fs.statSync(source); if (!stats.isFile()) { - console.error('Error: Source path is not a file'); + console.error('Source path is not a file'); + process.exit(0); } - if (fs.existsSync(copyPath)) { - const destStats = fs.statSync(copyPath); + if (fs.existsSync(dest)) { + const destStats = fs.statSync(dest); if (destStats.isDirectory()) { - console.error('Error: Destination path is a directory'); + console.error('Destination path is a directory'); + process.exit(0); } } - const data = fs.readFileSync(path); + const data = fs.readFileSync(source); - fs.writeFileSync(copyPath, data); + fs.writeFileSync(dest, data); - console.log(`File copied from "${path}" to "${copyPath}" successfully`); + console.log(`File copied from "${source}" to "${dest}" path successfully`); } catch (err) { - console.error(`Error: ${err}`); + console.error(err.message); + process.exit(0); } diff --git a/tests/copyFiles.test.js b/tests/copyFiles.test.js index 7b90dbd..2f61e98 100644 --- a/tests/copyFiles.test.js +++ b/tests/copyFiles.test.js @@ -97,6 +97,7 @@ describe('File Copy', () => { ); expect(stderr.length).toBeGreaterThan(0); + expect( fs.existsSync(path.join(directoryPath, path.basename(sourceFile))), ).toBe(false);