Respond to code comments: Add description, use const and rename test.

Author: jhorbulyk

component.json

@@ -39,6 +39,8 @@
       }
     },
     "write_attachment": {
+      "description":
+      "Multiple incoming events can be combined into one CSV file with the write CSV action.  See https://github.com/elasticio/csv-component/ for additional documentation.",
       "main": "./lib/actions/write.js",
       "title": "Write CSV attachment",
       "fields": {

lib/actions/write.js

@@ -7,6 +7,8 @@ const _ = require('lodash');
 const messages = require('elasticio-node').messages;
 const client = require('elasticio-rest-node')();
 
+const TIMEOUT_BETWEEN_EVENTS = 10000;
+
 let stringifier;
 let signedUrl;
 let timeout;
@@ -72,7 +74,7 @@ exports.process = function ProcessAction(msg, cfg) {
 
             yield exports.init(cfg);
         });
-    }, 10000);
+    }, TIMEOUT_BETWEEN_EVENTS);
 
     let row = msg.body;
     if (cfg.writer.columns) {

spec/read.spec.js

@@ -37,8 +37,8 @@ describe('CSV Read component', function CsvReadComponentTests() {
     }
 
 
-    it('empty body', function emptyBodyTest(done) {
-        runTest(csv.process, {}, {}, function emptyBodyTestAssertions(runner) {
+    it('should handle empty body', function handleEmptyBodyTest(done) {
+        runTest(csv.process, {}, {}, function handleEmptyBodyTestAssertions(runner) {
             expect(runner.data.length).to.equal(0);
             expect(runner.errors.length).to.equal(1);
             expect(runner.snapshot).to.be.undefined;