traffic_mansion 0.0.16 → 0.0.17

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8a4e05cf8d2d6d40c3e987b8d11a38a95e3eb1a1
-  data.tar.gz: c3982ed383e534acf243439a2e16bce2bce8d670
+  metadata.gz: 6437be2faa31736634daf27280d51bdf320565f8
+  data.tar.gz: ab887e60ee77d25a7ecaeb0eb41c23d4d2f6addf
 SHA512:
-  metadata.gz: 8943f9f2c4901ae0d748dbf76869db31151151f3caa59a2144e7c1e9c7f94e62be70ac78c259ac067a6b24115d40d285aef63b11ecb15d2ba51f314b68cfff15
-  data.tar.gz: 1e202381df9c51c6e3618f0f4d5eb4c9f0afbdc414098f9a6599f76bc43b560bb23e34944e1fca317e468695d2902ebc538ab4d3f83a16fd220d13dbcf8f7f96
+  metadata.gz: 7a4676cb050c6539846aa0996278aa54db7a7ccc5754e92e969bfca92cea412647aef0c5214772bf0f408edafe0042d779099c6606372ba70f3cc3f7914e4a32
+  data.tar.gz: 36908c2d01215069578f34d45af73314430a5648c79f2b6c70377d28515e6a3e5ad37302498094cc1bca26e6c1bbe6c963c333dce8acde4b4645e9b8d8247732

data/lib/traffic_mansion/queue.rb

@@ -35,7 +35,9 @@ module TrafficMansion
         "remote_addr"  => remote_ip,
         "http_referer" => referer
       }
-      TrafficMansion::Queue::add "signup", parent_event_id, event_data
+      event = TrafficMansion::Queue::add "signup", parent_event_id, event_data
+      TrafficMansion::Queue::update_user_id_for_visit parent_event_id, user_id
+      event
     end
 
     def self.add_sale user_id, payment_id, amount, product_code, remote_ip, referer, timestamp = Time.now.to_i
@@ -91,6 +93,25 @@ module TrafficMansion
       TrafficMansion::Queue.where(:user_id => user_id, :type => "signup").first
     end
 
+    def self.update_user_id_for_visit visit_event_id, user_id
+      visit = TrafficMansion::Queue.where(:id => visit_event_id).first
+      if visit
+        visit.set :user_id, user_id
+      end
+    end
+
+    def self.find_user_tracked_data user_id
+      visit = TrafficMansion::Queue.where(:type => "visit", :user_id => user_id, :processed => true).first
+      if visit && visit.result.present?
+        return visit.result
+      end
+      nil
+    end
+
+    def self.find_not_processed
+      where(:processed => false)
+    end
+
     def process
       if !parent? && parent_event_id.nil?
         update_parent_event_id

data/lib/traffic_mansion/version.rb

@@ -1,3 +1,3 @@
 module TrafficMansion
-  VERSION = "0.0.16"
+  VERSION = "0.0.17"
 end

data/spec/queue_spec.rb

@@ -34,45 +34,51 @@ describe TrafficMansion::Queue do
   end
 
   describe "self.add_signup" do
-    let(:parent_event_id) { "campaign_id" }
-    let(:user_id)         { "user_id" }
-    let(:remote_ip)       { "remote_ip" }
-    let(:referer)         { "referer" }
+    let!(:parent_event) { TrafficMansion::Queue.add "visit" }
+    let(:user_id)       { "user_id" }
+    let(:remote_ip)     { "remote_ip" }
+    let(:referer)       { "referer" }
 
     it "persists the signup" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer
       TrafficMansion::Queue.last.type.should eql "signup"
     end
 
     it "persists the parent event id" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer
-      TrafficMansion::Queue.last.parent_event_id.should eql parent_event_id
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer
+      TrafficMansion::Queue.last.parent_event_id.should eql parent_event.id.to_s
     end
 
     it "persists the user id" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer
       TrafficMansion::Queue.last.user_id.should eql user_id
     end
 
     it "persists the user id" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer
       TrafficMansion::Queue.last.data["user_id"].should eql user_id
     end
 
     it "persists the remote ip" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer
       TrafficMansion::Queue.last.data["remote_addr"].should eql remote_ip
     end
 
     it "persists the referer" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer
       TrafficMansion::Queue.last.data["http_referer"].should eql referer
     end
 
     it "persists the timestamp" do
-      TrafficMansion::Queue.add_signup parent_event_id, user_id, remote_ip, referer, 69
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer, 69
       TrafficMansion::Queue.last.data["timestamp"].should eql 69
     end
+
+    it "updates the user id of the parent visit" do
+      TrafficMansion::Queue.add_signup parent_event.id, user_id, remote_ip, referer, 69
+      parent_event.reload
+      parent_event.user_id.should eql user_id
+    end
   end
 
   describe "self.add_sale" do
@@ -258,6 +264,55 @@ describe TrafficMansion::Queue do
     end
   end
 
+  describe "self.update_user_id_for_visit" do
+    let!(:event) { TrafficMansion::Queue.add "visit"}
+
+    it "updates the event's user id with the given one" do
+      TrafficMansion::Queue.update_user_id_for_visit event.id, "user_id"
+      event.reload
+      event.user_id.should eql "user_id"
+    end
+  end
+
+  describe "self.find_user_tracked_data" do
+    context "when there is a processed visit event that contains a successful TM result" do
+      let(:result) { {"result" => "result"} }
+
+      before :each do
+        TrafficMansion::Queue.create :type => "visit", :user_id => "user_id", :processed => true, :result => result
+      end
+
+      it "returns the TM result" do
+        TrafficMansion::Queue.find_user_tracked_data("user_id").should eql result
+      end
+    end
+    context "when there is a processed visit event that does not contain a successful TM result" do
+      let(:result) { {} }
+
+      before :each do
+        TrafficMansion::Queue.create :type => "visit", :user_id => "user_id", :processed => true, :result => result
+      end
+
+      it "returns nil" do
+        TrafficMansion::Queue.find_user_tracked_data("user_id").should be_nil
+      end
+    end
+    context "when there is no processed visit event" do
+      it "returns nil" do
+        TrafficMansion::Queue.find_user_tracked_data("user_id").should be_nil
+      end
+    end
+  end
+
+  describe "self.find_not_processed" do
+    let!(:processed_event)     { TrafficMansion::Queue.create :type => "visit", :processed => true }
+    let!(:not_processed_event) { TrafficMansion::Queue.add "visit"}
+
+    it "returns the non processed events back" do
+      TrafficMansion::Queue.find_not_processed.to_a.should eql [not_processed_event]
+    end
+  end
+
   describe :process do
     context "when it is a visit event" do
       let!(:event) { TrafficMansion::Queue.add "visit"}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: traffic_mansion
 version: !ruby/object:Gem::Version
-  version: 0.0.16
+  version: 0.0.17
 platform: ruby
 authors:
 - AgileWings
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-06 00:00:00.000000000 Z
+date: 2013-08-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler