effective_test_bot 0.4.9 → 0.4.10

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6cf4d62267c1a02ff2a0bd22303f3ee9910e16ba
-  data.tar.gz: 9c971d7950781d4195c2e13b0dd2af2dd4ebe7eb
+  metadata.gz: fce8a949abc11a081049b6bf71e9212533cd060a
+  data.tar.gz: 7c7ceda55bd5db3edb0fd1325a07d0ab27aeb475
 SHA512:
-  metadata.gz: e78e8b4eca28808362fd52db931bb236d26571a814d7dabf372a6c268899b853d9c2c2cc405c5b53013250854f692717e1d7cf1c9b822e13649fcbd19f3aeb65
-  data.tar.gz: 9ac142dffbfc8288e6ebe00d9335f4d7c5b8a69d470be8dd32503232e42859bebaa945688715ebc8d9083f0998a30f020e4a920217e6926c865b66e8331df27b
+  metadata.gz: ac832e73e5c7fe59e84d0c4a3dccb180d68432d962bb65b21c9527a14c56bda7b5ee8efbe60215d51b2e4f34418314d683a3d04e50337fbdd7f3424257ca502a
+  data.tar.gz: 9be90c3a4f5efba42943460588c9219468fc3b3f3d0c789695f96f64da9c9da44de7cdda6ce5ce65bfe2ca988d519ecb7e484de96163895c1e3349c15999b69e

data/lib/effective_profile_bot/heap_analyzer.rb

@@ -0,0 +1,39 @@
+# bundle exec derailed exec perf:heap
+# http://samsaffron.com/archive/2015/03/31/debugging-memory-leaks-in-ruby
+# HeapAnalyzer.new(ARGV[0]).analyze
+
+require 'json'
+
+module EffectiveProfileBot
+  class HeapAnalyzer
+    attr_accessor :data, :filename
+
+    def initialize(filename)
+      puts "Initialized with #{filename}"
+
+      @filename = filename
+      @data = []
+    end
+
+    def analyze
+      File.open(filename) do |f|
+        f.each_line do |line|
+          parsed = JSON.parse(line)
+          data << parsed
+        end
+
+      end
+
+      data.group_by { |row| row['generation'] }.sort{ |a, b| a[0].to_i <=> b[0].to_i }.each do |k, v|
+        puts "generation #{k} objects #{v.count}"
+      end
+
+      data.select { |row| row['generation'] == 95 }
+        .group_by { |row| "#{row["file"]}:#{row["line"]}" }
+        .sort { |a, b| b[1].count <=> a[1].count }.each do |k, v|
+          puts "#{k} * #{v.count}"
+      end
+
+    end
+  end
+end

data/lib/effective_test_bot/version.rb

@@ -1,3 +1,3 @@
 module EffectiveTestBot
-  VERSION = '0.4.9'.freeze
+  VERSION = '0.4.10'.freeze
 end

data/lib/tasks/effecitve_profile_bot_tasks.rake

@@ -0,0 +1,14 @@
+namespace :profile do
+  namespace :bot do
+    desc 'Runs the HeapAnalyzer on a dump file'
+    task :heap, [:filename] do |t, args|
+      if args[:filename].blank?
+        puts "Expected usage:  rake profile:bot:heap[my_heap.dump]"
+        exit
+      end
+
+      require 'effective_profile_bot/heap_analyzer'
+      EffectiveProfileBot::HeapAnalyzer.new(args[:filename]).analyze
+    end
+  end
+end

data/lib/tasks/effective_test_bot_tasks.rake

@@ -5,7 +5,8 @@ require 'rails/test_unit/sub_test_task'
 # rake test:bot TEST=documents#new
 # rake test:bot TEST=documents#new,documents#show
 # rake test:bot TOUR=true
-# rake test:bot TOUR=verbose
+# rake test:bot TOUR=verbose   # Prints out the animated gif patch after test is run
+# rake test:bot TOUR=extreme   # Makes a whole bunch of extra screenshots
 
 # rake test:bot:tour
 # rake test:bot:tour TEST=documents#new
@@ -16,9 +17,11 @@ require 'rails/test_unit/sub_test_task'
 # rake test:bot:tours
 # rake test:bot:tours TEST=documents
 
+# rake test:tour  # Not the bot, just regular minitest 'rake test'
+# rake test:tourv
 
 namespace :test do
-  desc 'Runs the effective_test_bot'
+  desc 'Runs the effective_test_bot automated test suite'
   task :bot do
     if ENV['TEST'].present?
       ENV['TEST_BOT_TEST'] = ENV['TEST']
@@ -28,6 +31,18 @@ namespace :test do
     Rake::Task['test:effective_test_bot'].invoke
   end
 
+  desc "Runs 'rake test' with effective_test_bot tour mode enabled"
+  task :tour do
+    ENV['TOUR'] ||= 'true'
+    Rake::Task['test'].invoke
+  end
+
+  desc "Runs 'rake test' with effective_test_bot verbose tour mode enabled"
+  task :tourv do
+    ENV['TOUR'] ||= 'verbose'
+    Rake::Task['test'].invoke
+  end
+
   namespace :bot do
     desc 'Runs effective_test_bot environment test'
     task :environment do
@@ -55,6 +70,7 @@ namespace :test do
 
     desc 'Prints all effective_test_bot animated gif tour file paths'
     task :tours do
+      present = false
       Dir['test/tours/*.gif'].each do |file|
         file = file.to_s
 
@@ -62,8 +78,11 @@ namespace :test do
           next unless file.include?(ENV['TEST'])
         end
 
+        present = true
         puts "\e[32m#{Rails.root + file}\e[0m" # 32 is green
       end
+
+      puts 'No effective_test_bot tours present.' unless present
     end
   end # /namespace bot
 

data/test/support/effective_test_bot_form_filler.rb

@@ -33,7 +33,6 @@ module EffectiveTestBotFormFiller
       tab.click()
       synchronize!
       save_test_bot_screenshot
-      @shared_max_fields = nil # Reset value_for_input_numeric_field() history
 
       within('div' + tab['href']) { fill_form_fields(fills) }
     end
@@ -67,19 +66,15 @@ module EffectiveTestBotFormFiller
       when 'input_text', 'input_email', 'input_password', 'input_tel', 'input_number', 'input_checkbox', 'input_radio', 'textarea'
         field.set(value_for_field(field, fills))
       when 'select'
-        if field['class'].to_s.include?('select2') # effective_select
-          if EffectiveTestBot.tour_mode_extreme?
-            page.execute_script("try { $('select##{field['id']}').select2('open'); } catch(e) {};")
-            save_test_bot_screenshot
-          end
+        if EffectiveTestBot.tour_mode_extreme? && field['class'].to_s.include?('select2') # select2
+          page.execute_script("try { $('select##{field['id']}').select2('open'); } catch(e) {};")
+          save_test_bot_screenshot
         end
 
         field.select(value_for_field(field, fills), match: :first)
 
-        if field['class'].to_s.include?('select2') # effective_select
-          if EffectiveTestBot.tour_mode_extreme?
-            page.execute_script("try { $('select##{field['id']}').select2('close'); } catch(e) {};")
-          end
+        if EffectiveTestBot.tour_mode_extreme? && field['class'].to_s.include?('select2')
+          page.execute_script("try { $('select##{field['id']}').select2('close'); } catch(e) {};")
         end
       when 'input_file'
         if field['class'].to_s.include?('asset-box-uploader-fileinput')
@@ -99,8 +94,12 @@ module EffectiveTestBotFormFiller
       end
     end
 
-    save_test_bot_screenshot
+    # Clear any value_for_field momoized values
+    @filled_numeric_fields = nil
+    @filled_password_fields = nil
+    @filled_radio_fields = nil
 
+    save_test_bot_screenshot
   end
 
   # Generates an appropriately pseudo-random value for the given field
@@ -115,10 +114,9 @@ module EffectiveTestBotFormFiller
 
     fill_value = fill_value_for_field(fills, attributes)
 
-    # If there is a predefined fill value for this field, return it here
-    # except for select fields which are treated differently, so we can match fill values on both the html text or value
-    # this edge case is implemented below
-    if fill_value.present? && !['select'].include?(field_name)
+    # If there is a predefined fill value for this field return it now
+    # except for select, checkbox and radio fields which we want to match by value or label
+    if fill_value.present? && !['select', 'input_checkbox', 'input_radio'].include?(field_name)
       return fill_value
     end
 
@@ -171,6 +169,25 @@ module EffectiveTestBotFormFiller
         Faker::Lorem.words(3).join(' ').capitalize
       end
 
+    when 'input_checkbox'
+      value_for_input_checkbox_field(field, fill_value)
+
+    when 'input_email'
+      Faker::Internet.email
+
+    when 'input_file'
+      "#{File.dirname(__FILE__)}/important_documents._test"
+
+    when 'input_number'
+      value_for_input_numeric_field(field, "input[type='number'][name$='[#{attribute}]']")
+
+    when 'input_password'
+      # Use the same password throughout a single test. Allows passwords and password_confirmations to match.
+      @filled_password_fields ||= Faker::Internet.password
+
+    when 'input_radio'
+      value_for_input_radio_field(field, fill_value)
+
     when 'select'
       if fill_value.present? # accept a value or text
         field.all('option:enabled').each do |option|
@@ -179,30 +196,55 @@ module EffectiveTestBotFormFiller
       end
 
       field.all('option:enabled').select { |option| option.value.present? }.sample.try(:text) || '' # Don't select an empty option
-    when 'input_number'
-      value_for_input_numeric_field(field, "input[type='number'][name$='[#{attribute}]']")
-    when 'input_email'
-      Faker::Internet.email
-    when 'input_password'
-      # Use the same password throughout a single test. Allows passwords and password_confirmations to match.
-      @test_bot_current_password ||= Faker::Internet.password
+
     when 'input_tel'
       d = 10.times.map { DIGITS.sample }
       d[0] + d[1] + d[2] + '-' + d[3] + d[4] + d[5] + '-' + d[6] + d[7] + d[8] + d[9]
+
     when 'textarea'
       Faker::Lorem.paragraph
-    when 'input_checkbox', 'input_radio'
-      if field['value'] == 'true'
-        true
-      elsif field['value'] == 'false'
-        false
-      else
+
+    else
+      raise "fill_value unsupported field type: #{field['type']}"
+    end
+  end
+
+  def value_for_input_checkbox_field(field, fill_value)
+    if fill_value.present?
+      fill_values = Array(fill_value)  # Allow an array of fill values to be passed
+      (fill_values.include?(field['value']) || fill_values.include?(field.find(:xpath, '..').text))
+    elsif field['value'] == 'true'
+      true
+    elsif field['value'] == 'false'
+      false
+    else
+      [true, false].sample
+    end
+  end
+
+  # The first time we run into a radio button, we definitely want to set it to TRUE so it's definitely selected
+  # Subsequent ones, we can randomly true/false
+  # Then if we run into something that has a fill_value, we definitely want to set that one to TRUE, and false the rest
+  def value_for_input_radio_field(field, fill_value)
+    @filled_radio_fields ||= {}
+
+    previous = @filled_radio_fields[field['name']]
+
+    retval =
+      if previous == true  # We've selected one of the options before
         [true, false].sample
+      elsif previous.kind_of?(String)  # We selected a previous option with a specific fill_value
+        false
+      else # We've never seen this radio field before
+        true
       end
-    when 'input_file'
-      "#{File.dirname(__FILE__)}/important_documents._test"
+
+    if fill_value.present? && (fill_value == field['value'] || fill_value == field.find(:xpath, '..').text)
+      @filled_radio_fields[field['name']] = fill_value
+      true
     else
-      raise "fill_value unsupported field type: #{field['type']}"
+      @filled_radio_fields[field['name']] ||= true
+      retval
     end
   end
 
@@ -219,10 +261,10 @@ module EffectiveTestBotFormFiller
 
     # So there's definitely 2+ fields that share the same max, named the same
     # We want the total value of all these fields to add upto the max single max value
-    @shared_max_fields ||= {}
-    @shared_max_fields[selector] ||= max
+    @filled_numeric_fields ||= {}
+    @filled_numeric_fields[selector] ||= max
 
-    available = @shared_max_fields[selector]
+    available = @filled_numeric_fields[selector]
 
     amount = if max.kind_of?(Float)
       (((max * 1000.0) / shared_max_fields.length.to_f).ceil() / 1000.0).round(2)
@@ -231,7 +273,7 @@ module EffectiveTestBotFormFiller
     end
     amount = [[amount, min].max, available].min
 
-    @shared_max_fields[selector] = (available - amount)
+    @filled_numeric_fields[selector] = (available - amount)
     amount
   end
 

data/test/support/effective_test_bot_screenshots_helper.rb

@@ -5,22 +5,18 @@ module EffectiveTestBotScreenshotsHelper
 
   # Creates a screenshot based on the current test and the order in this test.
   def save_test_bot_screenshot
-    return unless EffectiveTestBot.screenshots? && defined?(current_test)
-
-    full_path = current_test_temp_path + "/#{current_test_screenshot_id}.png"
-    page.save_screenshot(full_path)
+    return unless EffectiveTestBot.screenshots?
+    page.save_screenshot("#{current_test_temp_path}/#{current_test_screenshot_id}.png")
   end
 
   # This is run before every test
   # def before_setup
-  #   super
-  #   return unless (EffectiveTestBot.screenshots? && defined?(current_test))
   # end
 
   # This gets called after every test.  Minitest hook for plugin developers
   def after_teardown
     super
-    return unless EffectiveTestBot.screenshots? && defined?(current_test) && (@test_bot_screenshot_id || 0) > 0
+    return unless EffectiveTestBot.screenshots? && (@test_bot_screenshot_id || 0) > 0
 
     if !passed? && EffectiveTestBot.autosave_animated_gif_on_failure?
       save_test_bot_failure_gif
@@ -101,7 +97,7 @@ module EffectiveTestBotScreenshotsHelper
   # current_test_failure_path: destination for .gifs of failing tests
 
   def current_test_temp_path
-    @_current_test_temp_path ||= "#{Rails.root}/tmp/test_bot/#{current_test || 'none'}"
+    @_current_test_temp_path ||= "#{Rails.root}/tmp/test_bot/#{current_test_name}"
   end
 
   def current_test_failure_path
@@ -110,7 +106,7 @@ module EffectiveTestBotScreenshotsHelper
 
   def current_test_failure_filename
     # Match Capybara-screenshots format-ish
-    "#{current_test}_failure_#{Time.now.strftime('%Y-%m-%d-%H-%M-%S')}.gif"
+    "#{current_test_name}_failure_#{Time.now.strftime('%Y-%m-%d-%H-%M-%S')}.gif"
   end
 
   # Where the tour animated gif ends up
@@ -119,7 +115,11 @@ module EffectiveTestBotScreenshotsHelper
   end
 
   def current_test_tour_filename
-    "#{current_test}.gif"
+    "#{current_test_name}.gif"
+  end
+
+  def current_test_name
+    defined?(curent_test) ? current_test : (@NAME.to_s.presence || 'none')
   end
 
   private

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: effective_test_bot
 version: !ruby/object:Gem::Version
-  version: 0.4.9
+  version: 0.4.10
 platform: ruby
 authors:
 - Code and Effect
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-14 00:00:00.000000000 Z
+date: 2015-12-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -215,12 +215,14 @@ extensions: []
 extra_rdoc_files: []
 files:
 - app/helpers/effective_test_bot_controller_helper.rb
+- lib/effective_profile_bot/heap_analyzer.rb
 - lib/effective_test_bot/engine.rb
 - lib/effective_test_bot/version.rb
 - lib/effective_test_bot.rb
 - lib/generators/effective_test_bot/install_generator.rb
 - lib/generators/templates/effective_test_bot.rb
 - lib/generators/templates/test_helper.rb
+- lib/tasks/effecitve_profile_bot_tasks.rake
 - lib/tasks/effective_test_bot_tasks.rake
 - test/concerns/test_botable/base_dsl.rb
 - test/concerns/test_botable/crud_dsl.rb