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..e6f7aef 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": "^1.9.12", "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": "1.9.12", + "resolved": "https://registry.npmjs.org/@mate-academy/scripts/-/scripts-1.9.12.tgz", + "integrity": "sha512-/OcmxMa34lYLFlGx7Ig926W1U1qjrnXbjFJ2TzUcDaLmED+A5se652NcWwGOidXRuMAOYLPU2jNYBEkKyXrFJA==", "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..c356e31 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": "^1.9.12", "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..383f996 100644 --- a/src/app.js +++ b/src/app.js @@ -1 +1,60 @@ -// write code here +/* eslint-disable no-param-reassign */ +/* eslint-disable no-shadow */ +/* eslint-disable no-console */ +'use strict'; + +const fs = require('fs'); +const path = require('path'); + +const moveFiles = (source, destination) => { + if (!source) { + throw new Error('Source must be provided'); + } + + if (!destination) { + throw new Error('Destination must be provided'); + } + + const resolvedSource = path.resolve(source); + const resolvedDestination = path.resolve(destination); + + if (resolvedSource === resolvedDestination) { + return; + } + + if (!fs.existsSync(source)) { + throw new Error(`${source} doesn't exist, check the file path`); + } + + const sourceStats = fs.statSync(source); + + if (!sourceStats.isFile()) { + throw new Error('Only files can be moved'); + } + + const destStats = fs.existsSync(destination) && fs.statSync(destination); + + if (destStats && destStats.isDirectory()) { + const fileName = path.basename(source); + + destination = path.join(destination, fileName); + } + + const parentDir = path.dirname(destination); + + if (!fs.existsSync(parentDir)) { + throw new Error(`Destination directory ${parentDir} doesn't exist`); + } + + fs.renameSync(source, destination); + + console.log(`File moved from ${source} to ${destination}`); +}; + +const [source, destination] = process.argv.slice(2); + +try { + moveFiles(source, destination); +} catch (err) { + console.error(err.message); +}