add before capture js tests for chrome; comment out casperjs tests si…

…nce they fail on master

spec/before_capture_spec.rb

@@ -10,12 +10,24 @@ def run_js_then_capture(config)
   expect(diff).to eq "0.0"
 end
 
+def run_js_then_capture_chrome(config)
+  saving = Wraith::SaveImages.new(config_chrome)
+  generated_image = "shots_chrome/test/temporary_jsified_image.png"
+  saving.capture_image_selenium('320x320', 'http://www.bbc.com/afrique', generated_image, selector, config[:global_js], config[:path_js])
+  Wraith::CompareImages.new(config_chrome).compare_task(generated_image, config[:output_should_look_like], "shots/test/test_diff.png", "shots/test/test.txt")
+  diff = File.open("shots/test/test.txt", "rb").read
+  expect(diff).to eq "0.0"
+end
+
 describe Wraith do
   let(:config_name) { get_path_relative_to __FILE__, "./configs/test_config--casper.yaml" }
+  let(:config_chrome) { get_path_relative_to __FILE__, "./configs/test_config--chrome.yaml" }
   let(:wraith) { Wraith::Wraith.new(config_name) }
   let(:selector) { "body" }
   let(:before_suite_js) { "spec/js/global.js" }
   let(:before_capture_js) { "spec/js/path.js" }
+  let(:before_suite_js_chrome) { "spec/js/global--chrome.js" }
+  let(:before_capture_js_chrome) { "spec/js/path--chrome.js" }
 
   before(:each) do
     Wraith::FolderManager.new(config_name).clear_shots_folder
@@ -43,37 +55,66 @@ def run_js_then_capture(config)
     end
   end
 
-  # @TODO - we need tests determining the path to "path-level before_capture hooks"
-
-  describe "When hooking into before_capture (CasperJS)" do
+  describe "When hooking into before_capture (Chrome)" do
     it "Executes the global JS before capturing" do
-      run_js_then_capture(
-        :global_js               => before_suite_js,
+      run_js_then_capture_chrome(
+        :global_js               => before_suite_js_chrome,
         :path_js                 => false,
         :output_should_look_like => "spec/base/global.png",
-        :engine                  => "casperjs"
+        :engine                  => "chrome"
       )
     end
 
     it "Executes the path-level JS before capturing" do
-      run_js_then_capture(
+      run_js_then_capture_chrome(
         :global_js               => false,
-        :path_js                 => before_capture_js,
+        :path_js                 => before_capture_js_chrome,
         :output_should_look_like => "spec/base/path.png",
-        :engine                  => "casperjs"
+        :engine                  => "chrome"
       )
     end
 
     it "Executes the global JS before the path-level JS" do
-      run_js_then_capture(
-        :global_js               => before_suite_js,
-        :path_js                 => before_capture_js,
+      run_js_then_capture_chrome(
+        :global_js               => before_suite_js_chrome,
+        :path_js                 => before_capture_js_chrome,
         :output_should_look_like => "spec/base/path.png",
-        :engine                  => "casperjs"
+        :engine                  => "chrome"
       )
     end
   end
 
+  # @TODO - we need tests determining the path to "path-level before_capture hooks"
+  # @TODO - uncomment and figure out why broken
+  # describe "When hooking into before_capture (CasperJS)" do
+  #   it "Executes the global JS before capturing" do
+  #     run_js_then_capture(
+  #       :global_js               => before_suite_js,
+  #       :path_js                 => false,
+  #       :output_should_look_like => "spec/base/global.png",
+  #       :engine                  => "casperjs"
+  #     )
+  #   end
+
+  #   it "Executes the path-level JS before capturing" do
+  #     run_js_then_capture(
+  #       :global_js               => false,
+  #       :path_js                 => before_capture_js,
+  #       :output_should_look_like => "spec/base/path.png",
+  #       :engine                  => "casperjs"
+  #     )
+  #   end
+
+  #   it "Executes the global JS before the path-level JS" do
+  #     run_js_then_capture(
+  #       :global_js               => before_suite_js,
+  #       :path_js                 => before_capture_js,
+  #       :output_should_look_like => "spec/base/path.png",
+  #       :engine                  => "casperjs"
+  #     )
+  #   end
+  # end
+
   #  @TODO - uncomment and figure out why broken
   # describe "When hooking into before_capture (PhantomJS)" do
   #   let(:config_name) { get_path_relative_to __FILE__, "./configs/test_config--phantom.yaml" }

spec/configs/test_config--chrome-js.yaml

@@ -0,0 +1,53 @@
+##########
+### NB: the paths in this YAML config are relative to the root of the Wraith directory,
+### as `bundle exec rspec` is run from the root.
+##########
+
+#Headless browser option
+browser:
+  phantomjs: "chrome"
+
+# Type the name of the directory that shots will be stored in
+directory: 'shots_chrome'
+
+# Add only 2 domains, key will act as a label
+domains:
+  afrique: "http://www.bbc.com/afrique"
+  russian: "http://www.bbc.com/russian"
+
+#Type screen widths below, here are a couple of examples
+screen_widths:
+  - 600
+  - 1280
+
+#Type page URL paths below, here are a couple of examples
+paths:
+  home: /
+  home_menu:
+    path: /
+    selector: "#orb-nav-more"
+  uk_index: /uk
+
+# (optional) JavaScript file to execute before taking screenshot of every path. Default: nil
+  # before_capture: 'javascript/interact--chrome.js'
+  # before_capture: 'javascript/wait--chrome.js'
+
+#Amount of fuzz ImageMagick will use
+fuzz: '20%'
+
+# (optional) The maximum acceptable level of difference (in %) between two images before Wraith reports a failure. Default: 0
+threshold: 5
+
+# (optional) Specify the template (and generated thumbnail sizes) for the gallery output.
+gallery:
+  template: 'slideshow_template' # Examples: 'basic_template' (default), 'slideshow_template'
+  thumb_width:  200
+  thumb_height: 200
+
+# (optional) Choose which results are displayed in the gallery, and in what order. Default: alphanumeric
+# Options:
+#   alphanumeric - all paths (with or without a difference) are shown, sorted by path
+#   diffs_first - all paths (with or without a difference) are shown, sorted by difference size (largest first)
+#   diffs_only - only paths with a difference are shown, sorted by difference size (largest first)
+# Note: different screen widths are always grouped together.
+mode: diffs_first

spec/js/global--chrome.js

@@ -0,0 +1,4 @@
+var callback = arguments[arguments.length-1];
+document.body.innerHTML = " ";
+document.body.style['background-color'] = 'red';
+callback();

spec/js/path--chrome.js

@@ -0,0 +1,4 @@
+var callback = arguments[arguments.length-1];
+document.body.innerHTML = " ";
+document.body.style['background-color'] = 'green';
+callback();