From 2ee9ec77c10baad683db9740da1cecd7756c1d02 Mon Sep 17 00:00:00 2001 From: Ivan Huzhva Date: Thu, 4 Sep 2025 11:20:09 +0200 Subject: [PATCH 1/2] add task solution --- .github/workflows/test.yml-template | 23 ++++++++++ package-lock.json | 9 ++-- package.json | 2 +- src/app.js | 69 ++++++++++++++++++++++++++++- 4 files changed, 97 insertions(+), 6 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 2a93237..475d3d5 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.1", "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.1", + "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-2.1.1.tgz", + "integrity": "sha512-Tf97p/jZ/ZRsQSPGcZf2FpvxgUCl8DiUOsiDFHj7HBN8gMK0iZOBQEtyqcFsauVUAvPP8Ayo8cAiC12MMp45iQ==", "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 f8c126f..52183c7 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.1", "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 0d15e7b..448360b 100644 --- a/src/app.js +++ b/src/app.js @@ -1 +1,68 @@ -// write code here +/* eslint-disable no-console */ +const fs = require('fs'); +const path = require('path'); + +const args = process.argv.slice(2); + +function moveAFile(src, dest) { + if (args.length !== 2 || !src || !dest) { + console.error(new Error('Two valid arguments must be entered')); + + return; + } + + if (args.some((a) => a.startsWith('-'))) { + console.error(new Error('Invalid arguments: flags are not allowed')); + + return; + } + + const filePath = path.resolve(src); + + const fileStats = fs.statSync(filePath, { throwIfNoEntry: false }); + + if (!fileStats || !fileStats.isFile()) { + console.error( + new Error('Source file does not exist or is not a regular file.'), + ); + + return; + } + + let fullPath; + + try { + const destStats = fs.statSync(dest, { throwIfNoEntry: false }); + + if (destStats && destStats.isDirectory()) { + fullPath = path.join(dest, path.basename(src)); + } else { + fullPath = dest; + } + + const parentDir = path.dirname(fullPath); + const parentStats = fs.statSync(parentDir, { throwIfNoEntry: false }); + + if (!parentStats || !parentStats.isDirectory()) { + console.error( + new Error( + 'Parent directory does not exist or is not a directory: ' + parentDir, + ), + ); + + return; + } + + if (path.resolve(fullPath) === filePath) { + // source и destination совпадают + return; + } + + fs.renameSync(filePath, fullPath); + console.log(`File moved successfully to ${fullPath}`); + } catch (err) { + console.error(err); + } +} + +moveAFile(args[0], args[1]); From ff4382b776e561481f0607e7a281eef67505ff7e Mon Sep 17 00:00:00 2001 From: Ivan Huzhva Date: Thu, 4 Sep 2025 11:42:52 +0200 Subject: [PATCH 2/2] add changes --- src/app.js | 55 ++++++++++++++++++++++++++++++++++++------------------ 1 file changed, 37 insertions(+), 18 deletions(-) diff --git a/src/app.js b/src/app.js index 448360b..c1286f1 100644 --- a/src/app.js +++ b/src/app.js @@ -32,29 +32,48 @@ function moveAFile(src, dest) { let fullPath; try { - const destStats = fs.statSync(dest, { throwIfNoEntry: false }); - - if (destStats && destStats.isDirectory()) { + if (dest.endsWith('/') || dest.endsWith('\\')) { + if (!fs.existsSync(dest)) { + console.error( + new Error('Destination directory does not exist: ' + dest), + ); + + return; + } + + if (!fs.statSync(dest).isDirectory()) { + console.error( + new Error('Destination path is not a directory: ' + dest), + ); + + return; + } fullPath = path.join(dest, path.basename(src)); } else { - fullPath = dest; - } - - const parentDir = path.dirname(fullPath); - const parentStats = fs.statSync(parentDir, { throwIfNoEntry: false }); - - if (!parentStats || !parentStats.isDirectory()) { - console.error( - new Error( - 'Parent directory does not exist or is not a directory: ' + parentDir, - ), - ); - - return; + const destStats = fs.statSync(dest, { throwIfNoEntry: false }); + + if (destStats && destStats.isDirectory()) { + fullPath = path.join(dest, path.basename(src)); + } else { + fullPath = dest; + + const parentDir = path.dirname(fullPath); + const parentStats = fs.statSync(parentDir, { throwIfNoEntry: false }); + + if (!parentStats || !parentStats.isDirectory()) { + console.error( + new Error( + 'Parent directory does not exist or is not a directory: ' + + parentDir, + ), + ); + + return; + } + } } if (path.resolve(fullPath) === filePath) { - // source и destination совпадают return; }