Merge pull request #63 from davidparsson/refactor/remove-date-format

Format dates without a dependency

package.json

@@ -41,7 +41,6 @@
     "time-grunt": "^1.2.1"
   },
   "dependencies": {
-    "date-format": "4.0.3",
     "lodash": "^4.17.21",
     "make-dir": "^3.1.0",
     "xmlbuilder": "^15.1.1"

spec/e2e_spec.js

@@ -28,7 +28,7 @@ describe('JUnit Report builder', function () {
     return result.trim();
   };
 
-  const reportWith = (content, testSuitesProperties) => '<?xml version="1.0" encoding="UTF-8"?>\n' + content;
+  const reportWith = (content) => '<?xml version="1.0" encoding="UTF-8"?>\n' + content;
 
   it('should produce a report identical to the expected one', function () {
     builder.testCase().className('root.test.Class1');

src/test_group.js

@@ -1,6 +1,5 @@
 // @ts-check
 var _ = require('lodash');
-var formatDate = require('date-format').asString;
 var { TestNode } = require('./test_node');
 
 class TestGroup extends TestNode {
@@ -20,7 +19,7 @@ class TestGroup extends TestNode {
    */
   timestamp(timestamp) {
     if (_.isDate(timestamp)) {
-      this._attributes.timestamp = formatDate('yyyy-MM-ddThh:mm:ss', timestamp);
+      this._attributes.timestamp = this.formatDate(timestamp);
     } else {
       this._attributes.timestamp = timestamp;
     }

src/test_node.js

@@ -104,6 +104,29 @@ class TestNode {
     return element;
   }
 
+  /**
+   * @protected
+   * @param {Date} date
+   * @returns {string}
+   */
+  formatDate(date) {
+    const pad = (num) => (num < 10 ? '0' : '') + num;
+
+    return (
+      date.getFullYear() +
+      '-' +
+      pad(date.getMonth() + 1) +
+      '-' +
+      pad(date.getDate()) +
+      'T' +
+      pad(date.getHours()) +
+      ':' +
+      pad(date.getMinutes()) +
+      ':' +
+      pad(date.getSeconds())
+    );
+  }
+
   /**
    * @protected
    * @param {import('xmlbuilder').XMLElement} element