i_wonder 0.0.6 → 0.0.7

data/README.rdoc

@@ -2,7 +2,7 @@
 
 Flexible Analytics, and Testing for every corner of your rails application. I read "The Lean Startup" and got inspired. There were a few good solutions, but none of them quite did what I was looking for. 
 
-Eventually I want it to be a drop-in replacement for Google analytics. You should also check out Vanity (http://vanity.labnotes.org/) as a solution for post of your analytics needs.
+Eventually I want it to be a drop-in replacement for Google analytics. Vanity (http://vanity.labnotes.org/) is a good alternative depending on your analytics needs.
 
 More information can be found at https://github.com/forrest/i_wonder/wiki
 

data/app/controllers/i_wonder/ab_tests_controller.rb

@@ -18,6 +18,7 @@ module IWonder
       @ab_test = AbTest.new(params[:ab_test])
 
       if @ab_test.save
+        AbTesting::Loader.save_ab_tests
         redirect_to @ab_test, :notice => "Successfully created ABTest"
       else
         render "new"
@@ -33,6 +34,7 @@ module IWonder
       @ab_test = AbTest.find(params[:id])
 
       if @ab_test.update_attributes(params[:ab_test])
+        AbTesting::Loader.save_ab_tests
         redirect_to @ab_test, :notice => "Successfully updated ABTest"
       else
         render "edit"

data/app/models/i_wonder/ab_test.rb

@@ -1,12 +1,12 @@
 module IWonder
   class AbTest < ActiveRecord::Base
-    attr_accessible :name, :sym, :description, :ab_test_goals_attributes, :test_applies_to, :test_group_names
+    attr_accessible :name, :sym, :description, :ab_test_goals_attributes, :test_applies_to, :test_group_names, :options, :test_group_data
     serialize :options, Hash
     serialize :test_group_data, Hash
     
-    has_many :test_group_memberships, :dependent => :destroy
+    has_many :test_group_memberships, :dependent => :destroy, :foreign_key => "ab_test_sym", :primary_key => :sym
 
-    has_many :ab_test_goals, :dependent => :destroy
+    has_many :ab_test_goals, :dependent => :destroy, :foreign_key => "ab_test_sym", :primary_key => :sym
     accepts_nested_attributes_for :ab_test_goals, :allow_destroy => true
     
     hash_accessor :options, :test_group_names, :type => :array, :reject_blanks => true
@@ -28,7 +28,17 @@ module IWonder
         errors.add(:base, "Must have atleast one goal")
       end
     end
-   
+      
+    after_save :save_to_file
+    def save_to_file
+      AbTesting::Loader.save_ab_test(self)
+    end
+      
+    after_destroy :remove_file
+    def remove_file
+      AbTesting::Loader.remove_file_for(self)
+    end
+      
     def started?
       test_group_memberships.count > 0
     end
@@ -63,7 +73,30 @@ module IWonder
       
       scoped_groups_with_goal_events.count
     end
+
+    def from_xml(xml)
+      [super]
+      
+      # make sure it's not blank. Then make sure it's a hash. Then Make sure it's symbolized.
+      self.options ||= {}
+      self.test_group_data ||= {}
+      self.options = {} if self.options.is_a?(String)
+      self.test_group_data = {} if self.test_group_data.is_a?(String)
+      self.options.symbolize_keys!
+      self.test_group_data.symbolize_keys!
+      
+      self.ab_test_goals.each(&:mark_for_destruction)
+      
+      hash = Hash.from_xml(xml)
+      hash["ab_test"]["ab_test_goals"].each{|ab_test_goal_hash|
+        ab_test_goal_hash["options"].symbolize_keys!
+        self.ab_test_goals.build(ab_test_goal_hash)
+      }
+    end
     
+    def started_at
+      test_group_memberships.minimum(:created_at)
+    end
     
   private
     

data/app/models/i_wonder/ab_test_goal.rb

@@ -1,8 +1,9 @@
 module IWonder
   class AbTestGoal < ActiveRecord::Base
-    belongs_to :ab_test
+    attr_accessible :goal_type, :event_type, :page_view_controller, :page_view_action, :options
+    
+    belongs_to :ab_test, :foreign_key => "ab_test_sym", :primary_key => :sym
     
-    attr_accessible :goal_type, :event_type, :page_view_controller, :page_view_action
     serialize :options, Hash
     hash_accessor :options, :goal_type, :default => "Event"
     hash_accessor :options, :event_type
@@ -32,7 +33,7 @@ module IWonder
         end
       end
       
-      sc.where("i_wonder_events.created_at > ?", self.ab_test.created_at) # event had to come after goal
+      sc.where("i_wonder_events.created_at > ?", self.ab_test.started_at) # event had to come after goal
     end
     
     def to_s

data/app/models/i_wonder/test_group_membership.rb

@@ -1,12 +1,12 @@
 module IWonder
   class TestGroupMembership < ActiveRecord::Base
-    belongs_to :ab_test
+    belongs_to :ab_test, :foreign_key => "ab_test_sym", :primary_key => :sym
     
     validates_presence_of :test_group_name, :on => :create, :message => "can't be blank"
     validates_inclusion_of :member_type, :in => %w( account user session ), :on => :create
     validates_presence_of :member_id, :on => :create, :message => "can't be blank"
     
-    validates_uniqueness_of :ab_test_id, :on => :create, :message => "must be unique for this member", :scope => [:member_type, :member_id]
+    validates_uniqueness_of :ab_test_sym, :on => :create, :message => "must be unique for this member", :scope => [:member_type, :member_id]
     
   end
 end

data/app/views/i_wonder/ab_tests/_ab_test_groups.html.erb

@@ -3,7 +3,11 @@
 <h4>Test Groups</h4>
 
 <ul id="ab_test_groups">
-	<%= render :partial => "ab_test_group_row", :collection => f.object.test_group_names, :as => :test_group_name %>
+	<% if f.object.test_group_names.empty? %> 
+		<%= render :partial => "ab_test_group_row", :collection => ["", ""], :as => :test_group_name %>
+	<% else %>
+		<%= render :partial => "ab_test_group_row", :collection => f.object.test_group_names, :as => :test_group_name %>
+	<% end %>
 </ul>
 
 <br style="clear:both"/>

data/app/views/i_wonder/ab_tests/_form.html.erb

@@ -27,11 +27,13 @@
 	</div>
 	
 	
-	<div class="field">
-		<%= f.label :sym, "Code reference symbol" %>
-		<%= f.text_field :sym, :size => 10 %>
-		<span class="description">letters numbers and underscores. NO spaces!</span>
-	</div>
+	<% if @ab_test.new_record? %>
+		<div class="field">
+			<%= f.label :sym, "Code reference symbol" %>
+			<%= f.text_field :sym, :size => 10 %>
+			<span class="description">letters numbers and underscores. NO spaces!</span>
+		</div>
+	<% end %>
 	
 	<hr/>
 	

data/app/views/i_wonder/ab_tests/index.html.erb

@@ -15,6 +15,8 @@
 	</ul>
 <% end %>
 
-<p>
-	<%= button_to "Create a new ABTest", url_for(:action => :new), :method => :get%>
-</p>
+<% unless Rails.env.production? %>
+	<p>
+		<%= button_to "Create a new ABTest", url_for(:action => :new), :method => :get%>
+	</p>
+<% end %>

data/app/views/i_wonder/ab_tests/show.html.erb

@@ -1,9 +1,13 @@
 <div class="show">
-	<p>
-		<%= link_to "Edit", edit_ab_test_path(@ab_test) %>
-		|
-		<%= link_to "Delete", @ab_test, :method => :delete, :confirm => "Are you sure?" %>
-	</p>
+	<% if Rails.env.production? %>
+		<span class="description">No Modifying tests in production</span>
+	<% else %>
+		<p>
+			<%= link_to "Edit", edit_ab_test_path(@ab_test) %>
+			|
+			<%= link_to "Delete", @ab_test, :method => :delete, :confirm => "Are you sure?" %>
+		</p>
+	<% end %>
 	
 	<h2>I wonder <%= @ab_test.name%>?</h2>
 	<p class="description">

data/db/migrate/20111022230720_i_wonder_migrations.rb

@@ -73,16 +73,16 @@ class IWonderMigrations < ActiveRecord::Migration
     add_index :i_wonder_ab_tests, :sym
     
     create_table :i_wonder_ab_test_goals do |t|
-      t.integer :ab_test_id
+      t.string :ab_test_sym
       t.text :options # serialized
     end
-    add_index :i_wonder_ab_test_goals, :ab_test_id
+    add_index :i_wonder_ab_test_goals, :ab_test_sym
     
     
     
     
     create_table :i_wonder_test_group_memberships do |t|
-      t.integer :ab_test_id
+      t.string :ab_test_sym
       t.string :test_group_name
       
       t.string :member_type
@@ -90,7 +90,7 @@ class IWonderMigrations < ActiveRecord::Migration
 
       t.timestamps
     end
-    add_index :i_wonder_test_group_memberships, :ab_test_id
+    add_index :i_wonder_test_group_memberships, :ab_test_sym
     add_index :i_wonder_test_group_memberships, :test_group_name
     add_index :i_wonder_test_group_memberships, [:member_type, :member_id], :name => "i_wonder_test_group_memberships_member"
    

data/lib/i_wonder/ab_testing/action_controller_mixins.rb

@@ -11,10 +11,9 @@ module IWonder
       def which_test_group?(event_sym)
         ab_test = IWonder::AbTest.find_by_sym(event_sym.to_s)
         return false unless ab_test
-        ab_test.which_test_group?(self)        
+        ab_test.which_test_group?(self)
       end
 
-
     end
   end
 end

data/lib/i_wonder/ab_testing/loader.rb

@@ -0,0 +1,54 @@
+module IWonder
+  module AbTesting
+    class Loader
+      I_WONDER_DATA_DIR = Rails.root.join("i_wonder").to_s
+      AB_TEST_DATA_DIR = File.join(I_WONDER_DATA_DIR,"ab_tests").to_s
+      
+      def self.save_ab_test(ab_test)
+        Dir.mkdir(I_WONDER_DATA_DIR) unless File.exists?(I_WONDER_DATA_DIR) 
+        Dir.mkdir(AB_TEST_DATA_DIR) unless File.exists?(AB_TEST_DATA_DIR)
+        
+        File.open(file_name(ab_test.sym), 'w') do |file|
+          file.write ab_test.to_xml(:except => [:id, :created_at, :updated_at], :include => :ab_test_goals)
+        end
+      end
+      
+      def self.load_all
+        Dir.open(AB_TEST_DATA_DIR) do |dir|
+          dir.each{|file_name|
+            if file_name =~ /(.+).xml/
+              load_sym($1)
+            end
+          }
+        end
+      end
+      
+      def self.load_sym(ab_test_sym)
+        if File.exists?(file_name(ab_test_sym))
+          File.open(file_name(ab_test_sym), 'r') do |file|
+            
+            ab_test = AbTest.find_by_sym(ab_test_sym)
+            ab_test ||= AbTest.new
+            
+            ab_test.from_xml(file.read)
+            
+            ab_test.save!
+          end          
+        end
+      end
+    
+      def self.remove_file_for(ab_test)
+        if File.exists?(file_name(ab_test.sym))
+          File.delete(file_name(ab_test.sym))
+        end
+      end
+    
+    private
+    
+      def self.file_name(sym)
+        AB_TEST_DATA_DIR + "/#{sym}.xml"
+      end
+    
+    end
+  end
+end

data/lib/i_wonder/engine.rb

@@ -1,8 +1,13 @@
 module IWonder
   class Engine < Rails::Engine
     # This is for the mounting the engine ===========
-    
     isolate_namespace IWonder
+    
+    initializer "i_wonder.loading_tests" do |app|
+      ActiveRecord::Base.send :include, HashAccessor # this is to avoid load order issues from the required gem
+      AbTesting::Loader.load_all
+    end
+    
   end
 end
 

data/lib/i_wonder/version.rb

@@ -1,3 +1,3 @@
 module IWonder
-  VERSION = "0.0.6"
+  VERSION = "0.0.7"
 end

data/lib/i_wonder.rb

@@ -16,6 +16,7 @@ module IWonder
   
   module AbTesting
     autoload :ActionControllerMixins, 'i_wonder/ab_testing/action_controller_mixins'
+    autoload :Loader, 'i_wonder/ab_testing/loader'
   end
 
 

data/test/dummy/db/schema.rb

@@ -35,11 +35,11 @@ ActiveRecord::Schema.define(:version => 20111023231947) do
   add_index "delayed_jobs", ["priority", "run_at"], :name => "delayed_jobs_priority"
 
   create_table "i_wonder_ab_test_goals", :force => true do |t|
-    t.integer "ab_test_id"
-    t.text    "options"
+    t.string "ab_test_sym"
+    t.text   "options"
   end
 
-  add_index "i_wonder_ab_test_goals", ["ab_test_id"], :name => "index_i_wonder_ab_test_goals_on_ab_test_id"
+  add_index "i_wonder_ab_test_goals", ["ab_test_sym"], :name => "index_i_wonder_ab_test_goals_on_ab_test_sym"
 
   create_table "i_wonder_ab_tests", :force => true do |t|
     t.string   "name"
@@ -114,7 +114,7 @@ ActiveRecord::Schema.define(:version => 20111023231947) do
   end
 
   create_table "i_wonder_test_group_memberships", :force => true do |t|
-    t.integer  "ab_test_id"
+    t.string   "ab_test_sym"
     t.string   "test_group_name"
     t.string   "member_type"
     t.string   "member_id"
@@ -122,7 +122,7 @@ ActiveRecord::Schema.define(:version => 20111023231947) do
     t.datetime "updated_at"
   end
 
-  add_index "i_wonder_test_group_memberships", ["ab_test_id"], :name => "index_i_wonder_test_group_memberships_on_ab_test_id"
+  add_index "i_wonder_test_group_memberships", ["ab_test_sym"], :name => "index_i_wonder_test_group_memberships_on_ab_test_sym"
   add_index "i_wonder_test_group_memberships", ["member_type", "member_id"], :name => "i_wonder_test_group_memberships_member"
   add_index "i_wonder_test_group_memberships", ["test_group_name"], :name => "index_i_wonder_test_group_memberships_on_test_group_name"