ghostbuster 0.3.6 → 0.3.7

data/README.md

@@ -40,6 +40,10 @@ To set the command for starting your server use:
 
     ghost.start_command "./start.sh"
 
+To set the timeout in seconds for the start_command to complete use:
+
+    ghost.start_wait = 2   # this is the default
+
 To set the command for stopping your server use:
 
     ghost.stop_command "./stop.sh"
@@ -118,6 +122,13 @@ This location will be relative to your root if it doesn't start with "http". You
 
 This will click the nth element matching the selector. If no index is specified it uses the first one found.
 
+### clickFollow
+
+*Arguments*: selector, [index or options]
+
+This will click the nth element matching the selector and assert the location changed. If no index is specified it uses the first one found.
+If no path option is given, it will merely assert that the location changed.
+
 ### input
 
 *Arguments*: selector, text
@@ -158,6 +169,12 @@ The callback will be called for each matching DOM element for the selector. The
 
 This assertion will attempt to match the current browser location. If your location does not start with `http`, it will be considered relative to the root of your test.
 
+### refuteLocation
+
+*Arguments*: location, [options]
+
+This assertion will attempt to refute the current browser location. If your location does not start with `http`, it will be considered relative to the root of your test.
+
 ### assertCount
 
 *Arguments*: selector, [options], callback

data/Rakefile

@@ -8,7 +8,7 @@ namespace :test do
     task task_name do
       print "#{task_name} ... "
       Dir.chdir("test/working_ghost") do
-        matcher = [/10 success, 0 failure, 1 pending/]
+        matcher = [/13 success, 0 failure, 1 pending/]
         fork {
           ENV['BUNDLE_GEMFILE'] = File.expand_path("./Gemfile")
           `bundle install`
@@ -19,7 +19,7 @@ namespace :test do
           begin
             matcher.each{|m| out[m] or raise("Couldn't match for #{m.inspect}")}
             real_size = Dir['*.png'].to_a.size
-            expected_size = 14
+            expected_size = 21
             raise("There are a weird number of screenshots, expected #{expected_size}, got #{real_size}") unless expected_size == real_size
             exit
           rescue
@@ -38,7 +38,7 @@ namespace :test do
   task :non_working do
     print "non_working_ghost ... "
     Dir.chdir("test/non_working_ghost") do
-      matcher = [/0 success, 9 failure, 1 pending/, /Bad link traversal\s+Assert location failed: Excepted http:\/\/127\.0\.0\.1:4567\/not-correct, got http:\/\/127\.0\.0\.1:4567\//, /Form input not equal\s+Assert first for selector #out did not meet expectations/, /To an invalid URL\s+The request for http:\/\/127\.0\.0\.1:this-url-is-invalid failed/, /This test will explode!\s+I hate you!/, /This test has no succeed\s+This test took too long/, /This test has a custom assertion name\s+Assert first "custom assertion name" did not meet expectations/, /Bad click selector\s+Couldn't find element 0 for selector i-just-made-this-up/]
+      matcher = [/0 success, 10 failure, 1 pending/, /Bad link traversal\s+Assert location failed: Excepted http:\/\/127\.0\.0\.1:4567\/not-correct, got http:\/\/127\.0\.0\.1:4567\//, /Form input not equal\s+Assert first for selector #out did not meet expectations/, /To an invalid URL\s+The request for http:\/\/127\.0\.0\.1:this-url-is-invalid failed/, /This test will explode!\s+I hate you!/, /This test has no succeed\s+This test took too long/, /This test has a custom assertion name\s+Assert first "custom assertion name" did not meet expectations/, /Bad click selector\s+Couldn't find element 0 for selector i-just-made-this-up/, /Bad click follow\s+Assert not location failed: Excepted not http:\/\/127\.0\.0\.1:4567\/, got http:\/\/127\.0\.0\.1:4567\//]
       fork {
         ENV['BUNDLE_GEMFILE'] = File.expand_path("./Gemfile")
         `bundle install`

data/lib/ghostbuster.coffee

@@ -1,6 +1,13 @@
 class Test
   constructor: (@runner, @name, @maxDuration, @testBody) ->
     @page = new WebPage()
+    @resources = []
+    resources = @resources
+    @page.onResourceRequested = (req) ->
+      resources.push(req.url)
+    @page.onResourceReceived = (req) ->
+      while (index = resources.indexOf(req.url)) != -1
+        resources.splice(index, 1)
     if @runner.useScreenshots()
       @page.viewportSize = @runner.viewportDimensions()
     @page.onConsoleMessage = (msg) ->
@@ -21,7 +28,7 @@ class Test
     @runner.lastErrors[@name] = error
   waitForAssertions: (whenDone) ->
     @stopTestTimer()
-    if @assertions.length == 0
+    if @assertions.length == 0 and @resources.length == 0
       @startTestTimer()
       whenDone.call(this)
     else
@@ -147,6 +154,39 @@ class Body
       "
       withValue @page.evaluate(evaluator)
 
+  clickFollow: (selector, opts) ->
+    opts ||= {}
+    test = @test
+    @test.assert opts, (withValue) ->
+      idx = opts.index || 0
+      currentLocation = @page.evaluate -> return window.location.href
+      eval "
+        var evaluator = function() {
+          var targets = document.querySelectorAll('#{selector}'),
+              evt = document.createEvent('MouseEvents'),
+              idx = #{idx};
+          evt.initMouseEvent('click', true, true, window, 0, 0, 0, 0, 0, false, false, false, false, 0, null); 
+          if (idx < targets.length) {
+            targets[idx].dispatchEvent(evt);
+            return true;
+          } else {
+            alert('Couldn\\'t find element #{idx} for selector #{selector}');
+            return false;
+          }
+        };
+      "
+      val = @page.evaluate(evaluator)
+      if val?
+        if opts.path?
+          test.body.assertLocation(opts.path)
+        else
+          test.body.refuteLocation(currentLocation)
+        page = @page
+        fn = -> withValue(val)
+        setTimeout fn, 50
+      else
+        withValue @page.evaluate(evaluator)
+
   assertCount: (selector, opts, assertionCallback) ->
     unless assertionCallback?
       assertionCallback = opts
@@ -197,6 +237,27 @@ class Body
       "
       withValue @page.evaluate(fn)
 
+  refuteLocation: (path, opts) ->
+    opts ||= {}
+    test = @test
+    location = @test.runner.normalizePath(path)
+    @test.assert opts, (withValue) ->
+      assertionDescription = if opts.name then " \"#{opts.name}\"" else ""
+      alerter = if test.getLastError()? then "" else "alert('Assert not location#{assertionDescription} failed: Excepted not #{location}, got '+currentLocation);"
+      eval "
+        var fn = function() {
+          var currentLocation = window.location.href;
+          if (window.location.href !== '#{location}') {
+            console.log('succeeded with ' +window.location.href);
+            return true;
+          } else {
+            #{alerter}
+            return false;
+          }
+        }
+      "
+      withValue @page.evaluate(fn)
+
   assertFirst: (selector, opts, assertionCallback) ->
     unless assertionCallback?
       assertionCallback = opts

data/lib/ghostbuster.rb

@@ -36,7 +36,7 @@ class Ghostbuster
         else
           sh @config.start_command
         end
-        sleep 2
+        sleep @config.start_wait
       end
       begin
         _, status = Process.waitpid2 fork { 

data/lib/ghostbuster/config.rb

@@ -1,6 +1,6 @@
 class Ghostbuster
   class Config
-    attr_accessor :pattern, :screenshot_dir, :start_command, :stop_command, :phantom_bin, :temp_dir, :verbose
+    attr_accessor :pattern, :screenshot_dir, :start_command, :stop_command, :phantom_bin, :temp_dir, :verbose, :start_wait
     attr_reader :screenshot_x, :screenshot_y
     def initialize(path_to_file = nil)
       @config_file = path_to_file || './Ghostfile'
@@ -12,6 +12,7 @@ class Ghostbuster
       @verbose = false
       @temp_dir = "/tmp"
       @start_command, @stop_command = "./start.sh", "./stop.sh"
+      @start_wait = 2
       if File.exist?(@config_file)
         instance_eval File.read(@config_file), @config_file, 1
       end

data/lib/ghostbuster/version.rb

@@ -1,3 +1,3 @@
 class Ghostbuster
-  VERSION = '0.3.6'
+  VERSION = '0.3.7'
 end

data/test/non_working_ghost/test_ghost.coffee

@@ -5,7 +5,12 @@ phantom.test.add "Bad link traversal", ->
     @body.click 'a'
     @body.assertLocation('/not-correct')
     @succeed()
-    
+
+phantom.test.add "Bad click follow", ->
+  @get '/', ->
+    @body.clickFollow '#hiya'
+    @succeed()
+
 phantom.test.add "Bad click selector", ->
   @get '/', ->
     @body.click 'i-just-made-this-up'

data/test/server1/views/index.erb

@@ -11,5 +11,6 @@
     <li>List item 3</li>
   </ul>
   <a href="/form">form</a>
+  <a id="hiya" href="javascript: false;">not_form</a>
 </body>
 </html>

data/test/working_ghost/test_ghost.coffee

@@ -6,8 +6,11 @@ phantom.test.add "Simple index", ->
       p.innerHTML == 'This is my paragraph'
     @body.assertAll 'ul li', (li, idx) ->
       li.innerHTML == "List item #{idx + 1}"
-    @body.assertCountAndAll 'a', 1, (a, idx) ->
-      a.href == 'http://127.0.0.1:4567/form'
+    @body.assertCountAndAll 'a', 2, (a, idx) ->
+      if idx == 0
+        a.href == 'http://127.0.0.1:4567/form'
+      else if idx == 1
+        a.href == 'javascript:%20false;'
     @succeed()
 
 phantom.test.add "Simple slow index", ->
@@ -28,3 +31,19 @@ phantom.test.add "Link traversal", ->
     @body.assertLocation('/form')
     @succeed()
 
+phantom.test.add "Link traversal with counter-case", ->
+  @get '/', ->
+    @body.click 'a'
+    @body.refuteLocation('/')
+    @succeed()
+
+phantom.test.add "Click follow", ->
+  @get '/', ->
+    @body.clickFollow 'a'
+    @succeed()
+
+phantom.test.add "Click follow with positive case", ->
+  @get '/', ->
+    @body.clickFollow 'a', path: "/form"
+    @succeed()
+

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: ghostbuster
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: 
   segments: 
   - 0
   - 3
-  - 6
-  version: 0.3.6
+  - 7
+  version: 0.3.7
 platform: ruby
 authors: 
 - Josh Hull
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-10-15 00:00:00 Z
+date: 2011-10-20 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   version_requirements: &id001 !ruby/object:Gem::Requirement