Extract createGitignore from create-files

Author: philnash

src/create-twilio-function.js

@@ -3,9 +3,9 @@ const {
   createDirectory,
   createEnvFile,
   createExampleFunction,
-  createGitignore,
   createPackageJSON
 } = require('./create-twilio-function/create-files');
+const createGitignore = require('./create-twilio-function/create-gitignore');
 const {
   installDependencies
 } = require('./create-twilio-function/install-dependencies');

src/create-twilio-function/create-files.js

@@ -1,10 +1,8 @@
 const fs = require('fs');
-const path = require('path');
 const { promisify } = require('util');
 const mkdir = promisify(fs.mkdir);
 const open = promisify(fs.open);
 const write = promisify(fs.write);
-const readFile = promisify(fs.readFile);
 
 function createDirectory(path, dirName) {
   return mkdir(path + '/' + dirName);
@@ -50,16 +48,9 @@ AUTH_TOKEN=${authToken}`;
   return createFile(fullPath, content);
 }
 
-async function createGitignore(dirPath) {
-  const fullPath = `${dirPath}/.gitignore`;
-  const content = await readFile(`${__dirname}/../../templates/.gitignore`);
-  return createFile(fullPath, content);
-}
-
 module.exports = {
   createDirectory,
   createPackageJSON,
   createExampleFunction,
-  createEnvFile,
-  createGitignore
+  createEnvFile
 };

src/create-twilio-function/create-files.test.js

@@ -2,8 +2,7 @@ const {
   createPackageJSON,
   createDirectory,
   createExampleFunction,
-  createEnvFile,
-  createGitignore
+  createEnvFile
 } = require('./create-files');
 const fs = require('fs');
 const { promisify } = require('util');
@@ -110,25 +109,3 @@ describe('createEnvFile', () => {
     }
   });
 });
-
-describe('createGitignore', () => {
-  test('it creates a new .gitignore file', async () => {
-    await createGitignore('./scratch');
-    const file = await stat('./scratch/.gitignore');
-    expect(file.isFile());
-    const contents = await readFile('./scratch/.gitignore', {
-      encoding: 'utf-8'
-    });
-    expect(contents).toMatch('*.log');
-  });
-
-  test('it rejects if there is already a .gitignore file', async () => {
-    fs.closeSync(fs.openSync('./scratch/.gitignore', 'w'));
-    expect.assertions(1);
-    try {
-      await createGitignore('./scratch');
-    } catch (e) {
-      expect(e.toString()).toMatch('file already exists');
-    }
-  });
-});

src/create-twilio-function/create-gitignore.js

@@ -0,0 +1,15 @@
+const fs = require('fs');
+const { promisify } = require('util');
+const readFile = promisify(fs.readFile);
+const open = promisify(fs.open);
+const write = promisify(fs.write);
+
+async function createGitignore(dirPath) {
+  const fullPath = `${dirPath}/.gitignore`;
+  const content = await readFile(`${__dirname}/../../templates/.gitignore`);
+  return open(fullPath, 'wx').then(fd => {
+    return write(fd, content);
+  });
+}
+
+module.exports = createGitignore;

src/create-twilio-function/create-gitignore.test.js

@@ -0,0 +1,42 @@
+// const nock = require('nock');
+const createGitignore = require('./create-gitignore');
+const { promisify } = require('util');
+const rimraf = promisify(require('rimraf'));
+const fs = require('fs');
+const mkdir = promisify(fs.mkdir);
+const readFile = promisify(fs.readFile);
+const stat = promisify(fs.stat);
+
+beforeAll(async () => {
+  await rimraf('./scratch');
+});
+
+beforeEach(async () => {
+  await mkdir('./scratch');
+});
+
+afterEach(async () => {
+  await rimraf('./scratch');
+});
+
+describe('createGitignore', () => {
+  test('it creates a new .gitignore file', async () => {
+    await createGitignore('./scratch');
+    const file = await stat('./scratch/.gitignore');
+    expect(file.isFile());
+    const contents = await readFile('./scratch/.gitignore', {
+      encoding: 'utf-8'
+    });
+    expect(contents).toMatch('*.log');
+  });
+
+  test('it rejects if there is already a .gitignore file', async () => {
+    fs.closeSync(fs.openSync('./scratch/.gitignore', 'w'));
+    expect.assertions(1);
+    try {
+      await createGitignore('./scratch');
+    } catch (e) {
+      expect(e.toString()).toMatch('file already exists');
+    }
+  });
+});