lineup 0.5.2 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: fef12db42af79b96ca8998beae02319bf95d47aa
-  data.tar.gz: 080d552e625a913dd64a89fc55c2e5286833fe5f
+  metadata.gz: 4f3e24b0859c6f0c12ecf146e1564082f7f8a6a3
+  data.tar.gz: 08743eb9f69f2bd4b820c02a3799bd22e358acb5
 SHA512:
-  metadata.gz: d42427ce1bb9db21d587e286645f4e8cb60020d22687910f620337cc4c55492b231c588d767d0810030c882c30f48e195f2904ae2a41f59130e975cbce0e0045
-  data.tar.gz: 1601351d06b6149fd308ac935a13b65106e2985cd63a7f37c4c051a3b9b354e603d0ffbaa3058aebff958e6892d00101603459595c0978582810929ed3691674
+  metadata.gz: 259f123498f086d61fb790aec5b791d0a99b273f3eb7ee34aa3989b9a654c7cfe6b01f6bff25495f4227bff1c4104183fc92b468695474d1ecb51f15b540bf4b
+  data.tar.gz: 93f0f59e0ea444937fd85d0d9338be16bbfdb712186a4f69a1c8b2248a5b0a8c157c47c923be86931490b31bcc1b67814188ff44c4467d669f8f2c0cf5f37154

data/bin/lineup

@@ -4,20 +4,28 @@ require '../lib/lineup'
 
 puts("This is an example. And a benchmark.")
 
-lineup = Lineup::Screenshot.new('https://uhr.ptb.de')
-lineup.wait_for_asynchron_pages(1)
-lineup.use_phantomjs true
+lineup = Lineup::Screenshot.new('https://prelive.otto.de')
+lineup.urls("/,multimedia,moebel,damenmode,mittesten")
+lineup.cookies([{"name"=>"trackingDisabled","value"=>"true","secure"=>false,"path"=>"/","domain"=>".otto.de"},
+                {"name"=>"survey","value"=>"1","secure"=>false,"path"=>"/","domain"=>".otto.de"}])
+lineup.localStorage(["us_customerServiceWidget"=>"{'customerServiceWidgetNotificationHidden':{'value':true,'timestamp':1467723066080}}"])
+lineup.resolutions("600,800,1200")
+lineup.use_phantomjs false
+lineup.wait_for_asynchron_pages(2)
+
+puts("Taking base screenshots")
 start = Time.now
-puts("Taking screenshots")
 lineup.record_screenshot('base')
 screenshots = Time.now
-sleep(2)
-lineup.record_screenshot('nun')
-screenshots2 = Time.now - 2
+
+puts("Taking new screenshots")
+lineup.record_screenshot('new')
+screenshots2 = Time.now
+
 puts("Starting comparison")
-lineup.compare('base', 'nun')
+lineup.compare('base', 'new')
 lineup.save_json(".")
-compare = Time.now - 2
+compare = Time.now
 
 s_t = screenshots - start
 puts("Screenshots first run took #{s_t} seconds")

data/lib/controller/browser.rb

@@ -7,7 +7,7 @@ require_relative '../helper'
 
 class Browser
 
-  def initialize(baseurl, urls, resolutions, path, headless, wait, cookies = [])
+  def initialize(baseurl, urls, resolutions, path, headless, wait, cookies = [], localStorage = [])
     @absolute_image_path = path
     FileUtils.mkdir_p @absolute_image_path
     @baseurl = baseurl
@@ -15,7 +15,18 @@ class Browser
     @resolutions = resolutions
     @headless = headless
     @wait = wait
-    @cookies = cookies
+
+    if cookies
+      @cookies = cookies
+    else
+      @cookies = []
+    end
+
+    if localStorage
+      @localStorage = localStorage
+    else
+      @localStorage = []
+    end
   end
 
   def record(version)
@@ -53,14 +64,29 @@ class Browser
     @browser.driver.manage.window.resize_to(width, 1000)
 
     url = Helper.url(@baseurl, url)
-    @browser.goto url
-    if @cookies.any?
-      @browser.cookies.clear
-      @cookies.each do |cookie|
-        @browser.cookies.add(cookie[:name], cookie[:value], domain: cookie[:domain], path: cookie[:path], expires: Time.now + 7200, secure: cookie[:secure])
-      end
+
+    if @cookies.any? || @localStorage.any?
+      # load url first before setting cookies and/or localStorage values
       @browser.goto url
+
+      if @cookies.any?
+        @browser.cookies.clear
+        @cookies.each do |cookie|
+          @browser.cookies.add(cookie[:name], cookie[:value], domain: cookie[:domain], path: cookie[:path], expires: Time.now + 7200, secure: cookie[:secure])
+        end
+      end
+
+      if @localStorage.any?
+        @localStorage.each do |keyValue|
+          # Generate javascript for localStorage.setItem, escaping single quotes in key and value
+          stmt = "localStorage.setItem('" + keyValue.keys.first.gsub("'", "\\\\'") + "','" + keyValue.values.first.gsub("'", "\\\\'") + "')";
+          @browser.execute_script(stmt)
+        end
+      end
     end
+
+    @browser.goto url
+
     sleep @wait if @wait
     @browser.screenshot.save( File.expand_path(filename))
   end

data/lib/lineup.rb

@@ -230,6 +230,36 @@ module Lineup
     end
 
 
+    def localStorage(localStorage)
+
+      # a hash for localStorage Key Value pair can be set here. this is optional.
+      #
+      # e.g {'key': 'value'}
+      #
+      # if it is not nil it has to be an array:
+
+      @localStorage = []
+      if localStorage
+        localStorage.each do |keyValue|
+          # generate :symbol => "value" hash map from "symbol" => "value"
+          #keyValue = keyValue.inject({}) { |element, (symbol, value)| element[symbol.to_sym] = value; element }
+
+          #Validation
+          (raise "LocalStorage Key Value pair must be a hash of format
+                {'key'=>'value'}
+                " unless keyValue.is_a? Hash)
+
+          raise "LocalStorage Key Value pair must have exactly one key" unless (keyValue.keys.length) == 1
+          raise "LocalStorage Key Value pair must have exactly one value" unless (keyValue.values.length) == 1
+          raise "LocalStorage Key Value pair must have a string as key" unless (keyValue.keys.first).is_a? String
+          raise "LocalStorage Key Value pair must have a string as value" unless (keyValue.values.first).is_a? String
+
+          @localStorage.push(keyValue)
+        end
+      end
+
+    end
+
     def wait_for_asynchron_pages(wait)
 
       # if required a wait time in seconds can be set. This may be needed, if after pageload some
@@ -265,6 +295,7 @@ module Lineup
       wait_for_asynchron_pages(configuration["wait_for_asynchron_pages"])
       cookie_for_experiment(configuration["cookie_for_experiment"])
       cookies(configuration["cookies"])
+      localStorage(configuration["localStorage"])
 
       if @cookies
         cookies_merged= @cookies.inject([]) { |a, element| a << element.dup }
@@ -278,7 +309,7 @@ module Lineup
       # the method calls set the variables for the parameters, we return an array with all of them.
       # for the example above it is:
       # [["/multimedia", "/sport"], [600, 800, 1200], "~/images/", true, "#/images/diff"]
-      [@urls, @resolutions, @screenshots_path, @headless, @difference_path, @wait_for_asynchron_pages, cookies_merged]
+      [@urls, @resolutions, @screenshots_path, @headless, @difference_path, @wait_for_asynchron_pages, cookies_merged, @localStorage]
     end
 
 
@@ -303,7 +334,7 @@ module Lineup
         cookies_merged.push(@cookie_for_experiment)
       end
 
-      browser = Browser.new(@baseurl, @urls, @resolutions, @screenshots_path, @headless, @wait_for_asynchron_pages, cookies_merged)
+      browser = Browser.new(@baseurl, @urls, @resolutions, @screenshots_path, @headless, @wait_for_asynchron_pages, cookies_merged, @localStorage)
       # the only argument missing is if this is the "base" or "new" screenshot, this can be
       # passed as an argument. The value does not need to be "base" or "new", but can be anything
       browser.record(version)

data/lib/lineup/version.rb

@@ -1,8 +1,8 @@
 module Lineup
   class Version
     MAJOR = 0
-    MINOR = 5
-    PATCH = 2
+    MINOR = 6
+    PATCH = 0
 
     class << self
       def to_s

data/tests/rspec/lineup_spec.rb

@@ -6,21 +6,68 @@ require_relative '../../lib/lineup'
 
 describe '#screeshot_recorder' do
 
-  BASE_URL = 'https://www.google.de/'
+  BASE_URL = 'https://www.google.de'
   SCREENSHOTS = "#{Dir.pwd}/screenshots/"
 
   after(:each) { FileUtils.rmtree SCREENSHOTS }
 
+  it 'loads minimum configuration from a json file' do
+    # Given
+    file = "#{Dir.pwd}/test_configuration.json"
+    FileUtils.rm file if (File.exists? file)
+    json = '{"urls":"page1",
+             "resolutions": "13",
+             "filepath_for_images":"screenshots/path",
+             "use_phantomjs":true,
+             "difference_path":"screenshots/path/difference",
+             "wait_for_asynchron_pages":5
+             }'
+    save_json(json, file)
+
+    # When
+    lineup = Lineup::Screenshot.new(BASE_URL)
+
+    # Then
+    expect(
+        lineup.load_json_config(file)
+    ).to eq([['page1'], [13], 'screenshots/path', true, 'screenshots/path/difference', 5, [], []])
+
+    # cleanup:
+    FileUtils.rm file if (File.exists? file)
+  end
+
   it 'loads all configuration from a json file' do
     # Given
     file = "#{Dir.pwd}/test_configuration.json"
     FileUtils.rm file if (File.exists? file)
     json = '{"urls":"page1, page2",
              "resolutions":"13,42",
-             "filepath_for_images":"some/path",
+             "filepath_for_images":"screenshots/path",
              "use_phantomjs":true,
-             "difference_path":"some/difference/image/path",
-             "wait_for_asynchron_pages":5
+             "difference_path":"screenshots/path/difference",
+             "wait_for_asynchron_pages":5,
+             "cookie_for_experiment":{
+                                      "name":"cookie1",
+                                      "value":"11111",
+                                      "domain":".google.de",
+                                      "path":"/",
+                                      "secure":false
+                                      },
+             "cookies" : [{
+                                      "name":"cookie2",
+                                      "value":"22222",
+                                      "domain":".google.de",
+                                      "path":"/",
+                                      "secure":false
+                          },
+                          {
+                                      "name":"cookie3",
+                                      "value":"33333",
+                                      "domain":".google.de",
+                                      "path":"/",
+                                      "secure":false
+                          }],
+             "localStorage":[{"myKey1":"myValue1"},{"myKey2":"myValue2"}]
              }'
     save_json(json, file)
 
@@ -30,7 +77,11 @@ describe '#screeshot_recorder' do
     # Then
     expect(
         lineup.load_json_config(file)
-    ).to eq([['page1', 'page2'], [13,42], 'some/path', true, 'some/difference/image/path', 5, []])
+    ).to eq([['page1', 'page2'], [13,42], 'screenshots/path', true, 'screenshots/path/difference', 5,
+             [{:name=>"cookie2", :value=>"22222", :domain=>".google.de", :path=>"/", :secure=>false},
+              {:name=>"cookie3", :value=>"33333", :domain=>".google.de", :path=>"/", :secure=>false},
+              {:name=>"cookie1", :value=>"11111", :domain=>".google.de", :path=>"/", :secure=>false}],
+             [{"myKey1"=>"myValue1"}, {"myKey2"=>"myValue2"}]])
 
     # cleanup:
     FileUtils.rm file if (File.exists? file)
@@ -58,7 +109,30 @@ describe '#screeshot_recorder' do
 
   end
 
-  it 'takes a screenshot a desired resolution' do
+  it 'opens a URL and takes mobile/tablet/desktop screenshots using firefox' do
+    # Given
+    lineup = Lineup::Screenshot.new(BASE_URL)
+    lineup.use_phantomjs false
+
+    # When
+    lineup.record_screenshot('base')
+
+    # Then
+    expect(
+        File.exist? ("#{Dir.pwd}/screenshots/frontpage_640_base.png")
+    ).to be(true)
+    # And
+    expect(
+        File.exist? ("#{Dir.pwd}/screenshots/frontpage_800_base.png")
+    ).to be(true)
+    # And
+    expect(
+        File.exist? ("#{Dir.pwd}/screenshots/frontpage_1180_base.png")
+    ).to be(true)
+
+  end
+
+  it 'takes a screenshot at desired resolution' do
     # Given
     width = '700'
     lineup = Lineup::Screenshot.new(BASE_URL)
@@ -99,7 +173,7 @@ describe '#screeshot_recorder' do
 
   end
 
-  it 'raises and exception if, parameters are changed after the base screenshot' do
+  it 'raises and exception if parameters are changed after the base screenshot' do
     # Given
     lineup = Lineup::Screenshot.new(BASE_URL)
     lineup.urls('/')
@@ -144,11 +218,12 @@ describe '#screeshot_recorder' do
     # Given
     file = "#{Dir.pwd}/test_configuration.json"
     FileUtils.rm file if (File.exists? file)
+
     json = '{"urls":"page1, page2",
              "resolutions":"13,42",
-             "filepath_for_images":"some/path",
+             "filepath_for_images":"screenshots/path",
              "use_phantomjs":true,
-             "difference_path":"some/difference/image/path",
+             "difference_path":"screenshots/path/difference",
              "wait_for_asynchron_pages":5,
              "cookie_for_experiment":{
                                       "name":"CONSENT",
@@ -188,6 +263,64 @@ describe '#screeshot_recorder' do
 
     # cleanup:
     FileUtils.rm file if (File.exists? file)
+
+  end
+
+  it 'takes a screenshot when loading a json config and setting local storage key value pair containing single quotes' do
+    # Given
+    file = "#{Dir.pwd}/test_configuration.json"
+    FileUtils.rm file if (File.exists? file)
+
+    json = '{"urls":"page1",
+             "resolutions":"200",
+             "filepath_for_images":"screenshots/path",
+             "use_phantomjs":true,
+             "difference_path":"screenshots/path/difference",
+             "wait_for_asynchron_pages":5,
+             "localStorage":[{"{\'mySpecialKey\'}":"{\'customerServiceWidgetNotificationHidden\':{\'value\':true,\'timestamp\':1467723066092}}"}]
+             }'
+    save_json(json, file)
+    lineup = Lineup::Screenshot.new(BASE_URL)
+    lineup.load_json_config(file)
+
+    lineup.record_screenshot('base')
+    # expect: no exception
+
+    # cleanup:
+    FileUtils.rm file if (File.exists? file)
+
+  end
+
+  it 'compares a base and a new screenshot when loading a json config and setting local storage key value pairs' do
+    # Given
+    file = "#{Dir.pwd}/test_configuration.json"
+    FileUtils.rm file if (File.exists? file)
+
+    json = '{"urls":"page1",
+             "resolutions":"200",
+             "filepath_for_images":"screenshots/path",
+             "use_phantomjs":true,
+             "difference_path":"screenshots/path/difference",
+             "wait_for_asynchron_pages":5,
+             "localStorage":[{"us_customerServiceWidget":"{\'customerServiceWidgetNotificationHidden\':{\'value\':true,\'timestamp\':1467723066092}}"},{"myKey2":"myValue2"}]
+             }'
+    save_json(json, file)
+    lineup = Lineup::Screenshot.new(BASE_URL)
+    lineup.load_json_config(file)
+
+    lineup.record_screenshot('base')
+    lineup.record_screenshot('new')
+
+    expect(
+        # When
+        lineup.compare('base', 'new')
+
+        # Then
+    ).to eq([])
+
+    # cleanup:
+    FileUtils.rm file if (File.exists? file)
+
   end
 
   it 'compares a base and a new screenshot and returns the difference if the images are NOT the same as json log' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lineup
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.6.0
 platform: ruby
 authors:
 - Finn
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-06-21 00:00:00.000000000 Z
+date: 2016-07-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -180,7 +180,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.8
+rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: lineup will help you in your automated design regression testing