fix(main): preformat publish time before slug

voxpelli · Jul 24, 2015 · b8c7813 · b8c7813
1 parent affb0f7
commit b8c7813
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 3 deletions.
diff --git a/index.js b/index.js
@@ -169,10 +169,10 @@ Formatter.prototype.preFormat = function (data) {
     }
   });
 
-  var slug = this._formatSlug(data);
+  data.properties.published = [data.properties.published && data.properties.published[0] ? new Date(data.properties.published[0]) : new Date()];
 
+  var slug = this._formatSlug(data);
   data.properties.slug = slug ? [slug] : [];
-  data.properties.published = [data.properties.published && data.properties.published[0] ? new Date(data.properties.published[0]) : new Date()];
 
   data.derived = {};
 

diff --git a/test/formatter.spec.js b/test/formatter.spec.js
@@ -238,7 +238,7 @@ describe('Formatter', function () {
       formatter._formatSlug(baseMicroformatData).should.equal('foo-bar-abc');
     });
 
-    it('should ulimately fall back to publish time', function () {
+    it('should ultimately fall back to publish time', function () {
       delete baseMicroformatData.properties.name;
       formatter._formatSlug(baseMicroformatData).should.equal('52441');
     });
@@ -393,6 +393,18 @@ describe('Formatter', function () {
         .that.deep.equals(['hello world']);
     });
 
+    it('should be able to base slug on default publish times', function () {
+      delete baseMicroformatData.properties.name;
+      delete baseMicroformatData.properties.published;
+      delete baseMicroformatData.properties.slug;
+
+      return formatter.preFormat(baseMicroformatData)
+        .should.eventually
+        .have.deep.property('properties.slug')
+        .that.deep.equals(['51600']);
+    });
+
+
   });
 
   describe('_formatFilesSlug', function () {