fixed lint errors, added unit test and removed async/await

Author: Brno Sartori

src/client-unit.js

@@ -572,6 +572,35 @@ describe('browserbox unit tests', () => {
     })
   })
 
+  describe('#deleteMailbox', () => {
+    beforeEach(() => {
+      sinon.stub(br, 'exec')
+    })
+
+    it('should call DELETE with a string payload', () => {
+      br.exec.withArgs({
+        command: 'DELETE',
+        attributes: ['mailboxname']
+      }).returns(Promise.resolve())
+
+      return br.deleteMailbox('mailboxname').then(() => {
+        expect(br.exec.callCount).to.equal(1)
+      })
+    })
+
+    it('should call mutf7 encode the argument', () => {
+      // From RFC 3501
+      br.exec.withArgs({
+        command: 'DELETE',
+        attributes: ['~peter/mail/&U,BTFw-/&ZeVnLIqe-']
+      }).returns(Promise.resolve())
+
+      return br.deleteMailbox('~peter/mail/\u53f0\u5317/\u65e5\u672c\u8a9e').then(() => {
+        expect(br.exec.callCount).to.equal(1)
+      })
+    })
+  })
+
   describe.skip('#listMessages', () => {
     beforeEach(() => {
       sinon.stub(br, 'exec')

src/client.js

@@ -359,7 +359,7 @@ export default class Client {
     }
   }
 
-	/**
+  /**
    * Delete a mailbox with the given path.
    *
    * DELETE details:
@@ -371,13 +371,9 @@ export default class Client {
    * @returns {Promise}
    *     Promise resolves if mailbox was deleted.
    */
-  async createMailbox (path) {
+  deleteMailbox (path) {
     this.logger.debug('Deleting mailbox', path, '...')
-    try {
-      await this.exec({ command: 'DELETE', attributes: [imapEncode(path)] })
-    } catch (err) {
-      throw err
-    }
+    return this.exec({ command: 'DELETE', attributes: [imapEncode(path)] })
   }
 
   /**