refactor: return a Promise instead of relying on callback parameter (#…

…289)
all-contributors · Oct 4, 2020 · d44b70a · d44b70a
1 parent f1d576f
commit d44b70a
Showing 1 changed file with 16 additions and 12 deletions.
diff --git a/src/init/__tests__/add-contributors-list.js b/src/init/__tests__/add-contributors-list.js
@@ -30,20 +30,24 @@ test('create contributors section if content is empty', () => {
   expect(result).toMatchSnapshot()
 })
 
-test('README exists', done => {
-  const file = 'README.md'
-  ensureFileExists(file)
-    .then(data => expect(data).toStrictEqual(file))
-    .then(_ => done())
+test('README exists', () => {
+  return new Promise(done => {
+    const file = 'README.md'
+    ensureFileExists(file)
+      .then(data => expect(data).toStrictEqual(file))
+      .then(_ => done())
+  })
 })
 
-test("LOREM doesn't exists", done => {
-  const file = 'LOREM.md'
-  ensureFileExists(file).then(data => {
-    expect(data).toStrictEqual(file)
-    return unlink(file, err => {
-      if (err) throw err
-      done()
+test("LOREM doesn't exists", () => {
+  return new Promise(done => {
+    const file = 'LOREM.md'
+    ensureFileExists(file).then(data => {
+      expect(data).toStrictEqual(file)
+      return unlink(file, err => {
+        if (err) throw err
+        done()
+      })
     })
   })
 })