Merge pull request #7 from Constructor-io/noch/path-fix

Add semicolon

Author: jjl014

spec/006-cookies.js

@@ -17,10 +17,10 @@ describe('ConstructorioID', function () {
   });
 
   describe('set_cookie', function () {
-    it('should create a cookie', function () {
+    it('should create a cookie with a valid path', function () {
       var session = new ConstructorioID();
-      session.set_cookie('mewantcookie', 'meeatcookie');
-      expect(global.document.cookie).to.match(/mewantcookie=meeatcookie/);
+      var cookieData =  session.set_cookie('mewantcookie', 'meeatcookie');
+      expect(cookieData).to.match(/mewantcookie=meeatcookie; expires=.*; path=\//);
     });
   });
 

src/constructorio-id.js

@@ -42,11 +42,14 @@
   ConstructorioID.prototype.set_cookie = function (name, value) {
     if (!this.on_node && this.persist) {
       var expires = new Date(Date.now() + this.cookie_days_to_live * 24 * 60 * 60 * 1000);
-      var cookie_data = name + '=' + value + '; expires=' + expires.toUTCString() + ' path=/';
+      var cookie_data = name + '=' + value + '; expires=' + expires.toUTCString() + '; path=/';
       if (this.cookie_domain) {
         cookie_data += '; domain=' + this.cookie_domain;
       }
       document.cookie = cookie_data;
+
+      // For testing purposes
+      return cookie_data;
     }
   };