cli: write tests for init command, add rimraf, add .npmignore

Author: raphamorim

.npmignore

@@ -0,0 +1,2 @@
+__fixtures__
+__tests__

cli/__fixtures__/package/package.json

@@ -0,0 +1,3 @@
+{
+  "name": "package"
+}

cli/__tests__/init-test.js

@@ -0,0 +1,62 @@
+const { spawnSync } = require('child_process')
+const path = require('path')
+const rimraf = require('rimraf')
+
+const scriptPath = path.resolve(__dirname, '../index.js')
+const packagePath = path.resolve(__dirname, '../__fixtures__/package')
+const emptyPackagePath = path.resolve(__dirname, '../__fixtures__/emptyPackage')
+
+require('./to-run-successfully-matcher')
+
+describe('[CLI] init', () => {
+  describe('react-tv init', () => {
+    let slug = 'react-tv'
+
+    beforeEach((done) => rimraf(path.resolve(packagePath, slug), done))
+    afterEach((done) => rimraf(path.resolve(packagePath, slug), done))
+
+    it('should create react-tv folder', () => {
+      const subject = spawnSync('node', [
+        scriptPath,
+        'init'
+      ], { cwd: packagePath })
+
+      const createdAppPath = path.resolve(packagePath, slug)
+      const appInfo = require(path.resolve(createdAppPath, 'webos/appinfo.json'))
+
+      expect(appInfo.id).toEqual('react.tv.app')
+      expect(appInfo.title).toEqual('package')
+      expect(subject).toRunSuccessfully()
+    })
+
+    it('should fail when package.json not exists', () => {
+      const subject = spawnSync('node', [
+        scriptPath,
+        'init'
+      ], { cwd: emptyPackagePath })
+
+      expect(subject).not.toRunSuccessfully()
+    })
+  })
+
+  describe('react-tv init <appName>', () => {
+    const appName = 'russell-crowe'
+
+    beforeEach((done) => rimraf(path.resolve(emptyPackagePath, appName), done))
+    afterEach((done) => rimraf(path.resolve(emptyPackagePath, appName), done))
+
+    it('should create custom app', () => {
+      const subject = spawnSync('node', [
+        scriptPath,
+        'init',
+        appName
+      ], { cwd: emptyPackagePath })
+
+      const createdAppPath = path.resolve(emptyPackagePath, appName)
+      const createdPackageJson = require(path.resolve(createdAppPath, 'package.json'))
+
+      expect(subject).toRunSuccessfully()
+      expect(createdPackageJson.name).toEqual('russell-crowe')
+    })
+  })
+})

cli/__tests__/to-run-successfully-matcher.js

@@ -0,0 +1,17 @@
+expect.extend({
+  toRunSuccessfully(received) {
+    const pass = received.status === 0
+
+    if (pass) {
+      return {
+        message: () => `expected exit code not to be 0`,
+        pass: true,
+      }
+    } else {
+      return {
+        message: () => `expected exit code to be 0, was ${received.status} with error: \n${received.output}`,
+        pass: false,
+      }
+    }
+  }
+})

cli/shared/index.js

@@ -31,41 +31,52 @@ function createReactTVApp(appName) {
 
   if (appName) {
     appPath = `${appPath}/${appName}`;
-    fs.copySync(customApp, path.resolve(appPath));
-    replace({
-      regex: '{{REACTTVAPP}}',
-      replacement: appName,
-      paths: [appName],
-      recursive: true,
-      silent: true,
-    });
+    try {
+      fs.copySync(customApp, path.resolve(appPath));
+      replace({
+        regex: '{{REACTTVAPP}}',
+        replacement: appName,
+        paths: [appName],
+        recursive: true,
+        silent: true,
+      });
+    } catch(e) {
+      return process.exit(1);
+    }
 
     debug('Done! 📺  ⭐');
-    return;
+    return process.exit(0);
   }
 
   if (fs.existsSync(packageJson)) {
     existentProject = true;
     appName = require(packageJson).name;
   } else {
-    return debug('package.json not founded');
+    debug('package.json not founded');
+    return process.exit(1);
   }
 
   if (!appName) {
-    return debug('package.json {name} property not exists');
+    debug('package.json {name} property not exists');
+    return process.exit(1);
   }
 
   appPath = `${appPath}/react-tv`;
-  fs.copySync(appTemplatePath, appPath);
-  replace({
-    regex: '{{REACTTVAPP}}',
-    replacement: appName,
-    paths: ['./react-tv'],
-    recursive: true,
-    silent: true,
-  });
+  try {
+    fs.copySync(appTemplatePath, appPath);
+    replace({
+      regex: '{{REACTTVAPP}}',
+      replacement: appName,
+      paths: ['./react-tv'],
+      recursive: true,
+      silent: true,
+    });
+  } catch(e) {
+    return process.exit(1);
+  }
 
   debug('Done! 📺  ⭐');
+  process.exit(0);
 }
 
 module.exports = {

package.json

@@ -14,15 +14,15 @@
   ],
   "scripts": {
     "build": "node scripts/rollup/build.js",
-    "build-all": "NODE_ENV=PROD node scripts/rollup/build.js",
-    "prepublishOnly": "yarn test && yarn build-all",
-    "prettier-stat": "node ./scripts/prettier/index",
-    "test": "yarn prettier-stat && yarn flow && yarn jest",
-    "jest": "jest",
-    "jest-watch": "jest --watch",
+    "build:all": "NODE_ENV=PROD node scripts/rollup/build.js",
+    "prepublishOnly": "yarn test && yarn build:all",
+    "prettier:stat": "node ./scripts/prettier/index",
+    "test": "yarn prettier:stat && yarn flow && yarn jest:ci",
+    "jest": "jest --testPathIgnorePatterns \"(__fixtures__|matcher.js)\"",
+    "jest:ci": "jest --testPathIgnorePatterns \"(__fixtures__|matcher.js)\" --ci --runInBand",
     "flow": "flow",
     "prettier": "node ./scripts/prettier/index.js write-changed",
-    "prettier-all": "node ./scripts/prettier/index.js write"
+    "prettier:all": "node ./scripts/prettier/index.js write"
   },
   "dependencies": {
     "chalk": "^2.1.0",
@@ -44,6 +44,7 @@
     "react": "^16.0.0",
     "react-fiber-types": "file:src/renderer/types",
     "react-reconciler": "^0.6.0",
+    "rimraf": "^2.6.2",
     "rollup": "^0.50.0",
     "rollup-plugin-babel": "^2.7.1",
     "rollup-plugin-commonjs": "^8.0.2",
@@ -82,7 +83,9 @@
     "webos"
   ],
   "engines": {
-    "node": ">=6"
+    "node": ">=6",
+    "npm": ">=3",
+    "yarn": ">=0.22"
   },
   "author": "Raphael Amorim <[email protected]>",
   "license": "MIT",
@@ -92,7 +95,8 @@
   "homepage": "https://github.com/raphamorim/react-tv#readme",
   "jest": {
     "roots": [
-      "<rootDir>/src"
+      "<rootDir>/src",
+      "<rootDir>/cli"
     ],
     "moduleDirectories": [
       "node_modules",

yarn.lock

@@ -2706,7 +2706,7 @@ right-align@^0.1.1:
   dependencies:
     align-text "^0.1.1"
 
-rimraf@2, rimraf@^2.5.1, rimraf@^2.6.1:
+rimraf@2, rimraf@^2.5.1, rimraf@^2.6.1, rimraf@^2.6.2:
   version "2.6.2"
   resolved "https://registry.yarnpkg.com/rimraf/-/rimraf-2.6.2.tgz#2ed8150d24a16ea8651e6d6ef0f47c4158ce7a36"
   dependencies: