dom_glancy 0.1.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2c84851470a93ba9650102b600020afe44d0e015
-  data.tar.gz: c76baf99580a6abd4080152d24748419bb584dbc
+  metadata.gz: 2652a65dec1c5c98ff34e5c08d53fee7f2d37491
+  data.tar.gz: f6a4fcfef1372c84781e2af456e85f2c27e79cd1
 SHA512:
-  metadata.gz: e2734c33ad4d12c6a97c94b7c683e7e243406c20f7c4e0c14d5da809ee501635f295158c3529d2d3ea76939c78799dfe57ec83a7de161d931a08cf62a2fa78c8
-  data.tar.gz: ac4244455dc406913eb676a4e9f2c32f9868d1d3175bdf82b602aa0ffe7d78c0f876a76f5cb9f8db5872a7304e6e621d32000f791f1b6996669be636095067ea
+  metadata.gz: bfb04f669881c5e6c30f05a4015b93ecfd5f712a207ef02a67b880e297ea9fcf6a0044a4606fc2655d98f07734d84200ee97d640a93dc1c2af37f86fe19ad39a
+  data.tar.gz: 066fe6d51d913d860d17b5140a2c43adc8f9275b48eebbb51c3ce387a3579f758213b2cf0208259c72a62acb36aa302277a1951cd66df066ecd332e609a0a8e2

data/.gitignore

@@ -19,3 +19,4 @@ tmp
 *.DS_store
 test/test_app/log
 test/test_app/tmp
+test/test_app/db/*.sqlite3

data/.travis.yml

@@ -4,12 +4,10 @@ rvm:
   - "2.0.0"
   - "2.1.0"
   - "2.1.2"
-env:
-  - DB=sqlite
 before_script:
   - export DISPLAY=:99.0
   - sh -e /etc/init.d/xvfb start
 script:
   - bundle exec rake
 addons:
-  firefox: "19.0.1"
+  firefox: "31.0"

data/CHANGELOG.md

@@ -0,0 +1,5 @@
+## 1.0.0
+
+- Change configuration syntax
+- Convert gem to use DomGlancy instances
+- Make page mappings account for scrollbar width by widening browser accordingly

data/Gemfile

@@ -10,7 +10,6 @@ gem 'minitest-reporters'
 gem 'awesome_print'
 gem 'mocha'
 
-gem 'selenium-webdriver', '2.42.0'
+gem 'selenium-webdriver', '2.44.0'
 gem 'capybara', '2.2.1'
 gem 'ae_page_objects', '1.0.0'
-

data/README.md

@@ -10,7 +10,7 @@ _you know. like taking a quick glance at the DOM, but make it sound kinda like T
 
 ## Installation
 
-Add this line to your application's Gemfile:
+Add this line to your application's Gemfile in the test and development groups:
 
     gem 'dom_glancy'
 
@@ -24,7 +24,74 @@ Or install it yourself as:
 
 ## Usage
 
-Don't know yet. I'll let you know when i figure something out.
+Using [Capybara](http://jnicklas.github.io/capybara/), visit a page to test. Use dom_glancy's `page_map_same?` to analyze the page.
+
+### Setup
+
+Before the current page can be mapped and the comparison performed, the dom_glancy library must be configured with some location parameters:
+
+    DomGlancy.configure do |c|
+      c.master_file_location  = '/path/to/master/files'
+      c.current_file_location = '/path/to/current/files'
+      c.diff_file_location    = '/path/to/difference/files'
+    end
+
+This setup can be done in a test helper; however, if it is done in a rails initializer the helper route `/dom_glancy` is added to the application and can be used to analyze the results and bless files easily.
+
+### Perform Comparison
+
+The return value of `page_map_same?('page_name_key')` is an array with the first element being a boolean representing same TRUE or FALSE. the second element is an message describing the differences and a suggestion on what to do next. examples of messages can be found below.
+
+    def test_responsive
+      # do some browser setup so the page is displayed like you want to test.
+      # for example, set the browser width to 635.
+
+      # visit the page however you like. we like PageObjects around here.
+
+      jack_ryan = DomGlancy::DomGlancy.new
+      same, msg = jack_ryan.page_map_same?('page_name_key')       # the dom_glancy magic!
+
+      assert same, msg                                            # use whatever assertion library you like.
+
+      # interact with the page. maybe expand or edit something.
+      same, msg = jack_ryan.page_map_same?('page_name_key__edit')
+
+      assert same, msg
+    end
+
+_in both this example and the error messages, the test name of `page_name_key` is used. this string needs to be unique to every test as it is used to look for master files, generate current map files, and produce a difference file at the completion of the comparison._
+
+### Error Messages
+
+The following error message would have been returned if the pages were not the same. Specifically, 2 new dom elements were found that were not in the master file, 1 element that was in the master file but not on the current page, and 35 elements were found to have changed from master to current.
+
+    ------- DOM Comparison Failure ------
+    Elements not in master: 2
+    Elements not in current: 1
+    Changed elements: 35
+    Files:
+    	current: /path/to/current/files/page_name_key.yaml
+    	master: /path/to/master/files/page_name_key_master.yaml
+    	difference: /path/to/difference/files/page_name_key_diff.html
+    Bless this current data set:
+    	cp /path/to/current/files/page_name_key.yaml /path/to/master/files/page_name_key_master.yaml
+    -------------------------------------
+
+The following error message would be returned if there was no master file found matching the test parameters.
+
+    ------- DOM Comparison Failure ------
+    Master file does not exist. To make a new master from
+    the current page, use this command:
+      cp /path/to/current/files/page_name_key.yaml /path/to/master/files/page_name_key_master.yaml
+    -------------------------------------
+
+Notice that in both cases there is a line of code that can be used to copy the current map file to the master file location and name it appropriately. in these examples, that line is `cp /path/to/current/maps/page_name_key.yaml /path/to/master/maps/page_name_key_master.yaml`.
+
+### DOM Glancy Visualizer
+
+If the dom_glancy gem is included in the application's development environment, then the route `/dom_glancy` can be visited to help deal with new files and test failures.
+
+For this to work, the initialization of dom_glancy must be done in an initializer or some other way that is included when the environment is loaded.
 
 ## Contributing
 

data/app/assets/javascripts/application.js

@@ -1 +1,7 @@
-
+function checkByParent(aId, aChecked) {
+    var collection = document.getElementById(aId).getElementsByTagName('INPUT');
+    for (var x=0; x<collection.length; x++) {
+        if (collection[x].type.toUpperCase()=='CHECKBOX')
+            collection[x].checked = aChecked;
+    }
+}

data/app/assets/stylesheets/dom_glancy.css

@@ -52,15 +52,15 @@ a.kr:active  { text-decoration:none; color: #A3906D; }
 }
 
 .kr--ul {
+  line-height: 1.8em;
   list-style-type: none;
-  padding:0;
   margin:0;
+  padding: 20px 0;
 }
 
-
 .kr--dom_review_table td {
     padding: 1em;
-    border: 0px solid;
+    border: 0 solid;
     background-color: transparent;
     vertical-align: top;
     min-width: 250px;
@@ -132,3 +132,19 @@ a.kr:active  { text-decoration:none; color: #A3906D; }
     padding-left: 7px;
     padding-right: 7px;
 }
+
+.kr--button {
+    font-family: inherit;
+    font-size: 100%;
+    padding: .5em 1em;
+    color: #956C1F;
+    border: 1px solid #956C1F;
+    background-color: #ffffff;
+    text-decoration: none;
+    border-radius: 10px;
+}
+
+.kr--button-small {
+    border-radius: 5px;
+    font-size: 75%;
+}

data/app/controllers/dom_glancy_controller.rb

@@ -4,8 +4,8 @@ class DomGlancyController < DomGlancyApplicationController
   require 'kramdown'
 
   def index
-    @files = if DomGlancy.diff_file_location
-      Dir[File.join(DomGlancy.diff_file_location, "*_diff.html")].map{|f| File.basename(f)}
+    @files = if DomGlancy.configuration.diff_file_location
+      Dir[File.join(DomGlancy.configuration.diff_file_location, "*_diff.html")].map{|f| File.basename(f)}
     else
       []
     end
@@ -26,16 +26,16 @@ class DomGlancyController < DomGlancyApplicationController
   end
 
   def new
-    @files_current = Dir[File.join(DomGlancy.current_file_location, '*.yaml')].map { |f| File.basename(f).gsub('.yaml', '') }
-    @files_master  = Dir[File.join(DomGlancy.master_file_location,  '*.yaml')].map { |f| File.basename(f).gsub('.yaml', '') }
+    @files_current = Dir[File.join(DomGlancy.configuration.current_file_location, '*.yaml')].map { |f| File.basename(f).gsub('.yaml', '') }
+    @files_master  = Dir[File.join(DomGlancy.configuration.master_file_location,  '*.yaml')].map { |f| File.basename(f).gsub('.yaml', '') }
 
     @extra_files = @files_current.select { |f| !@files_master.include?("#{f}_master")}.sort
   end
 
   def make_master
     test_root = params[:file]
-    src = DomGlancy.current_filename(test_root)
-    dst = DomGlancy.master_filename(test_root)
+    src = DomGlancy.configuration.current_filename(test_root)
+    dst = DomGlancy.configuration.master_filename(test_root)
     FileUtils.cp src, dst
 
     redirect_to '/dom_glancy/new'
@@ -43,7 +43,7 @@ class DomGlancyController < DomGlancyApplicationController
 
   def delete_current
     test_root = params[:file]
-    src = DomGlancy.current_filename(test_root)
+    src = DomGlancy.configuration.current_filename(test_root)
     FileUtils.rm_rf src
 
     redirect_to '/dom_glancy/new'
@@ -81,8 +81,8 @@ class DomGlancyController < DomGlancyApplicationController
   end
 
   def clear
-    Dir[File.join(DomGlancy.diff_file_location, '*.yaml'), File.join(DomGlancy.diff_file_location, '*.html')].each { |f| FileUtils.rm_rf(f) }
-    Dir[File.join(DomGlancy.current_file_location, '*.yaml')].each { |f| FileUtils.rm_rf(f) }
+    Dir[File.join(DomGlancy.configuration.diff_file_location, '*.yaml'), File.join(DomGlancy.configuration.diff_file_location, '*.html')].each { |f| FileUtils.rm_rf(f) }
+    Dir[File.join(DomGlancy.configuration.current_file_location, '*.yaml')].each { |f| FileUtils.rm_rf(f) }
     redirect_to dom_glancy_path
   end
 
@@ -93,16 +93,16 @@ class DomGlancyController < DomGlancyApplicationController
     blessings.each do |blessing|
       base_test_name = blessing
 
-      src_yaml = DomGlancy.current_filename(base_test_name)
-      dst_yaml = DomGlancy.master_filename(base_test_name)
+      src_yaml = DomGlancy::DomGlancy.current_filename(base_test_name)
+      dst_yaml = DomGlancy::DomGlancy.master_filename(base_test_name)
 
       FileUtils.cp src_yaml, dst_yaml if File.exist?(src_yaml)
 
       if params['delete_on_bless'] == 'true'
-        files_to_remove  = [DomGlancy.diff_filename(base_test_name)]
-        files_to_remove << File.join(DomGlancy.diff_file_location, blessing + '__current_not_master__diff.yaml')
-        files_to_remove << File.join(DomGlancy.diff_file_location, blessing + '__master_not_current__diff.yaml')
-        files_to_remove << File.join(DomGlancy.diff_file_location, blessing + '__changed_master__diff.yaml')
+        files_to_remove  = [DomGlancy::DomGlancy.diff_filename(base_test_name)]
+        files_to_remove << File.join(DomGlancy.configuration.diff_file_location, blessing + '__current_not_master__diff.yaml')
+        files_to_remove << File.join(DomGlancy.configuration.diff_file_location, blessing + '__master_not_current__diff.yaml')
+        files_to_remove << File.join(DomGlancy.configuration.diff_file_location, blessing + '__changed_master__diff.yaml')
 
         files_to_remove.each { |f| FileUtils.rm_rf f }
       end

data/app/views/dom_glancy/index.html.erb

@@ -1,12 +1,17 @@
 <h2>difference files</h2>
-<% if @files.length > 0 %>
-  <p>file location: <%= DomGlancy.diff_file_location %></p>
-<% else %>
-  <p>There were no differences files found.</p>
-<% end %>
+<p id="js--file_location">file location: <%= DomGlancy.configuration.diff_file_location %></p>
+<form name="bessing" action="/dom_glancy/bless" method="POST">
+  <p id="js--input"><input type="checkbox" name="delete_on_bless" value='true' checked> Remove difference files on blessing</p>
+  <input class="kr--button kr--button-small" type="button" value="Check All" onclick="checkByParent('kr--files', true);">
+  <input class="kr--button kr--button-small" type="button" value="Uncheck All" onclick="checkByParent('kr--files', false);">
 
-<ul id="kr--files" class="kr--ul">
-  <% @files.each do |file| %>
-    <li class="kr--file"><%= link_to file, "/dom_glancy/show/#{file}", :class => 'kr' %></li>
-  <% end %>
-</ul>
+  <ul id="kr--files" class="kr--ul">
+    <% @files.each_with_index do |file, i| %>
+      <li class="kr--file">
+        <input type="checkbox" name="option_<%= i %>" value="<%= File.basename(file).gsub('_diff.html', '') %>">
+        <%= link_to File.basename(file).gsub('_diff.html', ''), "/dom_glancy/show/#{file}", :class => 'kr' %>
+      </li>
+    <% end %>
+  </ul>
+  <input type="submit" value="Bless Selected" class="kr--button">
+</form>

data/app/views/dom_glancy/path_config.html.erb

@@ -1,8 +1,8 @@
 <h2>Path Configuration</h2>
 <table class="kr">
   <tbody>
-  <tr><th>master</th><td id="js-config_master"><%= DomGlancy.master_file_location %></td></tr>
-  <tr><th>current</th><td id="js-config_current"><%= DomGlancy.current_file_location %></td></tr>
-  <tr><th>diff</th><td id="js-config_diffs"><%= DomGlancy.diff_file_location %></td></tr>
+  <tr><th>master</th><td id="js-config_master"><%= DomGlancy.configuration.master_file_location %></td></tr>
+  <tr><th>current</th><td id="js-config_current"><%= DomGlancy.configuration.current_file_location %></td></tr>
+  <tr><th>diff</th><td id="js-config_diffs"><%= DomGlancy.configuration.diff_file_location %></td></tr>
   </tbody>
 </table>

data/app/views/dom_glancy/show.html.erb

@@ -7,20 +7,20 @@
 <table class="kr--dom_review_table">
   <tr>
     <td>
-      <% contents = File.read(File.join(DomGlancy.diff_file_location, "#{@file_diff}")) %>
+      <% contents = File.read(File.join(DomGlancy.configuration.diff_file_location, "#{@file_diff}")) %>
       <%= raw(contents) %>
     </td>
     <td>
       <h2>in current dom, not master</h2>
-      <% elements = YAML::load_file(File.join(DomGlancy.diff_file_location, @file_set_not_master)) %>
+      <% elements = YAML::load_file(File.join(DomGlancy.configuration.diff_file_location, @file_set_not_master)) %>
       <%= render :partial => "shared/dom_set", :locals => {:elements => elements, :finder_id => 'js--not_master'} %>
 
       <h2>in master dom, not current</h2>
-      <% elements = YAML::load_file(File.join(DomGlancy.diff_file_location, @file_set_not_current)) %>
+      <% elements = YAML::load_file(File.join(DomGlancy.configuration.diff_file_location, @file_set_not_current)) %>
       <%= render :partial => "shared/dom_set", :locals => {:elements => elements, :finder_id => 'js--not_current'} %>
 
       <h2>changed</h2>
-      <% elements = YAML::load_file(File.join(DomGlancy.diff_file_location, @file_set_changed)) %>
+      <% elements = YAML::load_file(File.join(DomGlancy.configuration.diff_file_location, @file_set_changed)) %>
       <%= render :partial => "shared/dom_set", :locals => {:elements => elements, :finder_id => 'js--changed'} %>
 
     </td>

data/app/views/layouts/dom_glancy.html.erb

@@ -3,13 +3,13 @@
 <head>
   <title>DOM Glancy</title>
   <meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no"/>
-
-  <%= stylesheet_link_tag "noramlize.css", :media => "all" %>
-  <%= stylesheet_link_tag "720_grid.css", :media => "screen and (min-width: 720px)" %>
-  <%= stylesheet_link_tag "986_grid.css", :media => "screen and (min-width: 986px)" %>
-  <%= stylesheet_link_tag "1236_grid.css", :media => "screen and (min-width: 1236px)" %>
-
-  <%= stylesheet_link_tag "dom_glancy", :media => "all" %>
+  <%= javascript_include_tag 'application'
+  %>
+  <%= stylesheet_link_tag 'normalize.css', :media => 'all' %>
+  <%= stylesheet_link_tag '720_grid.css',  :media => 'screen and (min-width: 720px)' %>
+  <%= stylesheet_link_tag '986_grid.css',  :media => 'screen and (min-width: 986px)' %>
+  <%= stylesheet_link_tag '1236_grid.css', :media => 'screen and (min-width: 1236px)' %>
+  <%= stylesheet_link_tag 'dom_glancy',    :media => 'all' %>
 
   <%= csrf_meta_tags %>
 </head>

data/dom_glancy.gemspec

@@ -21,6 +21,6 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "bundler", "~> 1.3"
   spec.add_development_dependency "rake"
 
-  spec.add_dependency(%q<kramdown>, ["~> 1.1.0"])
-  spec.add_dependency 'capybara'
+  spec.add_dependency 'kramdown', '~> 1.1.0'
+  spec.add_dependency 'capybara', ['>= 1.0', '< 3']
 end

data/lib/dom_glancy/analysis.rb

@@ -1,141 +1,141 @@
 module DomGlancy
+  class DomGlancy
+    def analyze(master_data, current_data, test_root = nil)
+      output_hash = {}
 
-  def analyze(master_data, current_data, test_root = nil)
-    output_hash = {}
+      master_set  = master_data.to_set
+      current_set = current_data.to_set
 
-    master_set  = master_data.to_set
-    current_set = current_data.to_set
+      set_current_not_master = current_set - master_set
+      set_master_not_current = master_set  - current_set
+      set_changed_master = Set.new
 
-    set_current_not_master = current_set - master_set
-    set_master_not_current = master_set  - current_set
-    set_changed_master = Set.new
+      changed_element_pairs = []
+      if set_master_not_current.count > 0 || set_current_not_master.count > 0
 
-    changed_element_pairs = []
-    if set_master_not_current.count > 0 || set_current_not_master.count > 0
+        ok_pairs = pairs_that_are_close_enough(set_current_not_master, set_master_not_current)
 
-      ok_pairs = pairs_that_are_close_enough(set_current_not_master, set_master_not_current)
-
-      ok_pairs.each do |item1, item2|
-        set_current_not_master.delete(item1)
-        set_master_not_current.delete(item2)
-      end
+        ok_pairs.each do |item1, item2|
+          set_current_not_master.delete(item1)
+          set_master_not_current.delete(item2)
+        end
 
-      changed_element_pairs = get_set_of_same_but_different(set_current_not_master, set_master_not_current)
+        changed_element_pairs = get_set_of_same_but_different(set_current_not_master, set_master_not_current)
 
-      changed_element_pairs.each do |item1, item2|
-        set_current_not_master.delete(item1)
-        set_current_not_master.delete(item2)
+        changed_element_pairs.each do |item1, item2|
+          set_current_not_master.delete(item1)
+          set_current_not_master.delete(item2)
 
-        set_master_not_current.delete(item1)
-        set_master_not_current.delete(item2)
-      end
+          set_master_not_current.delete(item1)
+          set_master_not_current.delete(item2)
+        end
 
-      changed_element_pairs.select!{ |pair| !DOMElement.new(pair[0]).close_enough?(DOMElement.new(pair[1])) }
-      changed_element_pairs.each do |pair|
-        set_changed_master.add(pair.first)
+        changed_element_pairs.select!{ |pair| !DOMElement.new(pair[0]).close_enough?(DOMElement.new(pair[1])) }
+        changed_element_pairs.each do |pair|
+          set_changed_master.add(pair.first)
+        end
       end
-    end
 
-    all_same = set_current_not_master.count == 0 && set_master_not_current.count == 0 && changed_element_pairs.count == 0
+      all_same = set_current_not_master.count == 0 && set_master_not_current.count == 0 && changed_element_pairs.count == 0
 
-    create_diff_file(set_current_not_master, set_master_not_current, set_changed_master, test_root) if test_root && !all_same
+      create_diff_file(set_current_not_master, set_master_not_current, set_changed_master, test_root) if test_root && !all_same
 
-    output_hash[:not_in_master]  = set_current_not_master
-    output_hash[:not_in_current] = set_master_not_current
-    output_hash[:changed_element_pairs] = changed_element_pairs
-    output_hash[:same] = all_same
-    output_hash[:test_root] = test_root
-    output_hash
-  end
-
-  def make_svg(set_master_not_current, set_current_not_master, set_changed_master)
-    js_id = 0
-    set_master_not_current.each do |item|
-      item[:js_id] = js_id
-      js_id += 1
-    end
-    set_current_not_master.each do |item|
-      item[:js_id] = js_id
-      js_id += 1
-    end
-    set_changed_master.each do |item|
-      item[:js_id] = js_id
-      js_id += 1
+      output_hash[:not_in_master]  = set_current_not_master
+      output_hash[:not_in_current] = set_master_not_current
+      output_hash[:changed_element_pairs] = changed_element_pairs
+      output_hash[:same] = all_same
+      output_hash[:test_root] = test_root
+      output_hash
     end
 
-    rectangles = set_current_not_master.map  { |item| item.merge(format__not_in_master) }
-    rectangles << set_master_not_current.map { |item| item.merge(format__not_in_current) }
-    rectangles << set_changed_master.map     { |item| item.merge(format__same_but_different) }
-    rectangles.flatten!
+    def make_svg(set_master_not_current, set_current_not_master, set_changed_master)
+      js_id = 0
+      set_master_not_current.each do |item|
+        item[:js_id] = js_id
+        js_id += 1
+      end
+      set_current_not_master.each do |item|
+        item[:js_id] = js_id
+        js_id += 1
+      end
+      set_changed_master.each do |item|
+        item[:js_id] = js_id
+        js_id += 1
+      end
 
-    generate_svg(rectangles)
-  end
+      rectangles = set_current_not_master.map  { |item| item.merge(format__not_in_master) }
+      rectangles << set_master_not_current.map { |item| item.merge(format__not_in_current) }
+      rectangles << set_changed_master.map     { |item| item.merge(format__same_but_different) }
+      rectangles.flatten!
 
-  def create_diff_file(set_current_not_master, set_master_not_current, set_changed_master, test_root)
-    filename = DomGlancy.diff_filename(test_root)
-    svg = make_svg(set_current_not_master, set_master_not_current, set_changed_master)
-    File.open(filename, 'w') { |file| file.write(svg) }
-    save_set_info(test_root, 'current_not_master', set_current_not_master)
-    save_set_info(test_root, 'master_not_current', set_master_not_current)
-    save_set_info(test_root, 'changed_master', set_changed_master)
-  end
+      generate_svg(rectangles)
+    end
 
-  def save_set_info(test_root, suffix, data_set)
-    filename = File.join(DomGlancy.diff_file_location, "#{test_root}__#{suffix}__diff.yaml")
+    def create_diff_file(set_current_not_master, set_master_not_current, set_changed_master, test_root)
+      filename = DomGlancy.diff_filename(test_root)
+      svg = make_svg(set_current_not_master, set_master_not_current, set_changed_master)
+      File.open(filename, 'w') { |file| file.write(svg) }
+      save_set_info(test_root, 'current_not_master', set_current_not_master)
+      save_set_info(test_root, 'master_not_current', set_master_not_current)
+      save_set_info(test_root, 'changed_master', set_changed_master)
+    end
 
-    data_array = data_set.to_a
+    def save_set_info(test_root, suffix, data_set)
+      filename = File.join(::DomGlancy.configuration.diff_file_location, "#{test_root}__#{suffix}__diff.yaml")
 
-    File.open(filename, 'w') { |file| file.write(data_array.to_yaml) }
-  end
+      data_array = data_set.to_a
+
+      File.open(filename, 'w') { |file| file.write(data_array.to_yaml) }
+    end
 
-  def get_set_of_same_but_different(set1, set2)
-    same_but_different_pairs = []
-    set1.each do |item1|
-      element1 = DOMElement.new(item1)
-      set2.each do |item2|
-        element2 = DOMElement.new(item2)
-        if element1.same_element?(element2)
-          same_but_different_pairs << [item1, item2] #unless element1.close_enough?(element2)
+    def get_set_of_same_but_different(set1, set2)
+      same_but_different_pairs = []
+      set1.each do |item1|
+        element1 = DOMElement.new(item1)
+        set2.each do |item2|
+          element2 = DOMElement.new(item2)
+          if element1.same_element?(element2)
+            same_but_different_pairs << [item1, item2] #unless element1.close_enough?(element2)
+          end
         end
       end
+      same_but_different_pairs
     end
-    same_but_different_pairs
-  end
 
-  def pairs_that_are_close_enough(set1, set2)
-    ok_pairs = []
-    set1.each do |item1|
-      element1 = DOMElement.new(item1)
-      set2.each do |item2|
-        element2 = DOMElement.new(item2)
-        if element1.same_element?(element2) && element1.close_enough?(element2)
-          ok_pairs << [item1, item2]
+    def pairs_that_are_close_enough(set1, set2)
+      ok_pairs = []
+      set1.each do |item1|
+        element1 = DOMElement.new(item1)
+        set2.each do |item2|
+          element2 = DOMElement.new(item2)
+          if element1.same_element?(element2) && element1.close_enough?(element2)
+            ok_pairs << [item1, item2]
+          end
         end
       end
+      ok_pairs
     end
-    ok_pairs
-  end
 
-  def make_analysis_failure_report(analysis_data)
-    return '' if analysis_data[:same]
-
-    msg = ["\n------- DOM Comparison Failure ------"]
-    msg << "Elements not in master: #{analysis_data[:not_in_master].count}"
-    msg << "Elements not in current: #{analysis_data[:not_in_current].count}"
-    msg << "Changed elements: #{analysis_data[:changed_element_pairs].count}"
-    msg << "Files:"
-    msg << "\tcurrent: #{DomGlancy.current_filename(analysis_data[:test_root])}"
-    msg << "\tmaster: #{DomGlancy.master_filename(analysis_data[:test_root])}"
-    msg << "\tdifference: #{DomGlancy.diff_filename(analysis_data[:test_root])}"
-    msg << "Bless this current data set:"
-    msg << "\t#{blessing_copy_string(analysis_data[:test_root])}"
-    msg<< "-------------------------------------"
-
-    msg.join("\n")
-  end
+    def make_analysis_failure_report(analysis_data)
+      return '' if analysis_data[:same]
+
+      msg = ["\n------- DOM Comparison Failure ------"]
+      msg << "Elements not in master: #{analysis_data[:not_in_master].count}"
+      msg << "Elements not in current: #{analysis_data[:not_in_current].count}"
+      msg << "Changed elements: #{analysis_data[:changed_element_pairs].count}"
+      msg << "Files:"
+      msg << "\tcurrent: #{DomGlancy.current_filename(analysis_data[:test_root])}"
+      msg << "\tmaster: #{DomGlancy.master_filename(analysis_data[:test_root])}"
+      msg << "\tdifference: #{DomGlancy.diff_filename(analysis_data[:test_root])}"
+      msg << "Bless this current data set:"
+      msg << "\t#{blessing_copy_string(analysis_data[:test_root])}"
+      msg<< "-------------------------------------"
+
+      msg.join("\n")
+    end
 
-  def blessing_copy_string(test_root)
-    "cp #{DomGlancy.current_filename(test_root)} #{DomGlancy.master_filename(test_root)}"
+    def blessing_copy_string(test_root)
+      "cp #{DomGlancy.current_filename(test_root)} #{DomGlancy.master_filename(test_root)}"
+    end
   end
-
 end

data/lib/dom_glancy/configuration.rb

@@ -0,0 +1,22 @@
+module DomGlancy
+  class << self
+    attr_accessor :configuration
+  end
+
+  def self.configure
+    self.configuration ||= Configuration.new
+    yield(configuration)
+  end
+
+  class Configuration
+    attr_accessor :master_file_location
+    attr_accessor :diff_file_location
+    attr_accessor :current_file_location
+
+    def initialize
+      @master_file_location  = nil
+      @diff_file_location    = nil
+      @current_file_location = nil
+    end
+  end
+end