feat: sendHostname can works after 7 days

Author: Sohee Lee

src/js/request.js

@@ -9,6 +9,19 @@ var object = require('./object');
 var collection = require('./collection');
 var type = require('./type');
 
+/**
+ * Check if the date has passed 7 days
+ * @param {number} date - milliseconds
+ * @returns {boolean}
+ * @ignore
+ */
+function isExpired(date) {
+    var now = new Date().getTime();
+    var ms7days = 7 * 24 * 60 * 60 * 1000;
+
+    return now - date > ms7days;
+}
+
 /**
  * Send hostname on DOMContentLoaded.
  * To prevent hostname set tui.usageStatistics to false.
@@ -21,18 +34,20 @@ function sendHostname(applicationId) {
     var hitType = 'event';
     var trackingId = 'UA-115377265-9';
     var applicationKeyForStorage = 'TOAST UI ' + applicationId + ' for ' + hostname + ': Statistics';
-    var alreadySentForThisApplication = window.localStorage.getItem(applicationKeyForStorage);
+    var date = window.localStorage.getItem(applicationKeyForStorage);
 
-    // skip only if the flag is defined and is set to false explicitly
-    if ((!type.isUndefined(window.tui) && window.tui.usageStatistics === false)
-        || alreadySentForThisApplication) {
+    // skip if the flag is defined and is set to false explicitly
+    if (!type.isUndefined(window.tui) && window.tui.usageStatistics === false) {
         return;
     }
 
-    if (!alreadySentForThisApplication) {
-        window.localStorage.setItem(applicationKeyForStorage, true);
+    // skip if not pass seven days old
+    if (date && !isExpired(date)) {
+        return;
     }
 
+    window.localStorage.setItem(applicationKeyForStorage, new Date().getTime());
+
     setTimeout(function() {
         if (document.readyState === 'interactive' || document.readyState === 'complete') {
             imagePing(url, {

test/request.test.js

@@ -29,6 +29,7 @@ describe('module:request', function() {
             // can not spy on imagePing. spy on appendChild instead.
             spyOn(document.body, 'appendChild');
             spyOn(document.body, 'removeChild');
+            spyOn(localStorage, 'getItem').and.returnValue(null);
         });
 
         it('should call appendChild', function(done) {
@@ -51,4 +52,46 @@ describe('module:request', function() {
             }, 1500);
         });
     });
+
+    describe('sendHostname with localstorage', function() {
+        beforeEach(function() {
+            window.tui = window.tui || {};
+
+            // can not spy on imagePing. spy on appendChild instead.
+            spyOn(document.body, 'appendChild');
+            spyOn(document.body, 'removeChild');
+        });
+
+        it('should not call appendChild within 7 days', function(done) {
+            var now = new Date().getTime();
+            var ms6days = 6 * 24 * 60 * 60 * 1000;
+
+            spyOn(localStorage, 'getItem').and.returnValue(now - ms6days);
+
+            window.tui.usageStatistics = true;
+
+            request.sendHostname('editor');
+
+            setTimeout(function() {
+                expect(document.body.appendChild).not.toHaveBeenCalled();
+                done();
+            }, 1500);
+        });
+
+        it('should call appendChild after 7 days', function(done) {
+            var now = new Date().getTime();
+            var ms8days = 8 * 24 * 60 * 60 * 1000;
+
+            spyOn(localStorage, 'getItem').and.returnValue(now - ms8days);
+
+            window.tui.usageStatistics = true;
+
+            request.sendHostname('editor');
+
+            setTimeout(function() {
+                expect(document.body.appendChild).toHaveBeenCalled();
+                done();
+            }, 1500);
+        });
+    });
 });