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..10897bf 100644 --- a/src/app.js +++ b/src/app.js @@ -1 +1,47 @@ +/* eslint-disable no-console */ 'use strict'; + +const fs = require('fs'); +const source = process.argv[2]; +const dest = process.argv[3]; + +if (!source) { + console.error('Please provide correct source path'); + process.exit(0); +} + +if (!dest) { + console.error('Please provide correct destination path'); + process.exit(0); +} + +if (source === dest) { + process.exit(0); +} + +try { + const stats = fs.statSync(source); + + if (!stats.isFile()) { + console.error('Source path is not a file'); + process.exit(0); + } + + if (fs.existsSync(dest)) { + const destStats = fs.statSync(dest); + + if (destStats.isDirectory()) { + console.error('Destination path is a directory'); + process.exit(0); + } + } + + const data = fs.readFileSync(source); + + fs.writeFileSync(dest, data); + + console.log(`File copied from "${source}" to "${dest}" path successfully`); +} catch (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);