added some basic tests, refactorings, version 0.5

Author: ncri

.gitignore

@@ -4,6 +4,7 @@
 .config
 .yardoc
 .idea
+.sass-cache
 Gemfile.lock
 InstalledFiles
 _yardoc
@@ -16,3 +17,6 @@ spec/reports
 test/tmp
 test/version_tmp
 tmp
+spec/ajaxify_test/log
+spec/ajaxify_test/tmp
+.DS_Store

.rspec

@@ -0,0 +1,2 @@
+--color
+--format progress

.travis.yml

@@ -0,0 +1,5 @@
+rvm:
+  - 1.9.3
+before_script:
+  - "export DISPLAY=:99.0"
+  - "sh -e /etc/init.d/xvfb start"

Rakefile

@@ -1,2 +1,15 @@
 #!/usr/bin/env rake
 require "bundler/gem_tasks"
+# require "rake/testtask"
+
+# Rake::TestTask.new(:test) do |t|
+#   t.libs << 'test'
+#   t.test_files = FileList['test/**/*_test.rb']
+#   t.verbose = true
+# end
+
+
+require 'rspec/core/rake_task'
+
+RSpec::Core::RakeTask.new(:spec)
+task default: :spec

ajaxify_rails.gemspec

@@ -20,4 +20,15 @@ Gem::Specification.new do |gem|
   gem.version       = AjaxifyRails::VERSION
 
   gem.add_dependency 'rails', '>= 3.1.0'
+
+  gem.add_development_dependency 'jquery-rails'
+  gem.add_development_dependency 'rspec-rails', '2.9'
+  gem.add_development_dependency 'rspec-steps'
+  gem.add_development_dependency 'capybara'
+  gem.add_development_dependency 'selenium-webdriver', '>= 2.30.0'
+  gem.add_development_dependency 'sqlite3'
+  gem.add_development_dependency 'haml', '>= 3.1.5'
+  gem.add_development_dependency 'sass-rails', '~> 3.2.3'
+  gem.add_development_dependency 'coffee-rails', '~> 3.2.1'
+  #gem.add_development_dependency 'poltergeist'
 end

lib/ajaxify_rails/version.rb

@@ -1,3 +1,3 @@
 module AjaxifyRails
-  VERSION = "0.2.1"
+  VERSION = "0.5"
 end

spec/ajaxify_test

@@ -0,0 +1 @@
+Subproject commit 66dddc2c992a8ab6a46cddc7471c239cb5aa711b

spec/integration/ajaxify_rails_spec.rb

@@ -0,0 +1,166 @@
+require 'spec_helper'
+
+
+steps 'Test basic behavior', :js => true do
+
+	before(:each) do   # only executed once before all "it" blocks (see https://github.com/LRDesign/rspec-steps)
+	  visit root_path(ajaxify_on: true)
+	  @layout_id = find('body')['data-id']
+	end
+
+  it "should display loader" do 
+    click_on 'Page 1'
+    page.should have_css(".ajaxify_loader")
+  end
+
+  it "should load page's content" do    
+    should_have_content 'Page 1 Content'
+    push_state_path_should_be '/home/page1'
+    ensure_layout_not_reloaded
+  end
+
+  it "should redirect" do
+  	click_on 'Page 3 (redirects to page 1)'
+  	should_have_content 'Page 1 Content'
+  	push_state_path_should_be '/home/page1'
+  	ensure_layout_not_reloaded
+  end
+
+  it "should display flash" do
+  	should_have_notice 'Redirected to Page 1'
+  end
+
+  it "should submit a form" do
+  	click_on 'Page 2'
+  	fill_in 'input1', with: 'test'
+  	check 'check_me'
+  	push_state_path_should_be '/home/page2'
+  	click_button 'Submit me'
+  	should_have_notice 'Form submitted (test, , 1)'
+  	ensure_layout_not_reloaded
+  end
+
+  it "should load previous content when clicking the browser back button" do
+  	click_on 'Home'
+  	wait_for_ajaxify_loaded
+ 		click_on 'Page 1'
+  	wait_for_ajaxify_loaded
+  	click_browser_back_button
+  	should_have_content 'Home Content'
+  	push_state_path_should_be '/'
+  	ensure_layout_not_reloaded
+	end
+
+	it "should load next content when clicking the browser forward button" do
+  	click_browser_forward_button
+  	should_have_content 'Page 1 Content'
+  	push_state_path_should_be '/home/page1'
+  	ensure_layout_not_reloaded
+	end
+
+	it "should not repost form clicking the browser back button" do
+		click_on 'Page 2'
+  	click_button 'Submit me'
+  	wait_for_ajaxify_loaded
+  	click_on 'Page 1'
+  	wait_for_ajaxify_loaded
+  	click_browser_back_button
+  	wait_for_ajaxify_loaded
+  	page.should_not have_css("#notice", visible: true)
+  	should_have_content /A Form/
+  	push_state_path_should_be '/home/page2'
+	end
+
+end
+
+
+steps 'Test hash based url scheme for browsers without pushState', :js => true do
+
+	before(:each) do   # only executed once before all "it" blocks (see https://github.com/LRDesign/rspec-steps)
+	  visit root_path(ajaxify_on: true, push_state_enabled: false)
+	  @layout_id = find('body')['data-id']
+	  #sleep 2 # poltergeist needs this
+	  hash_path_should_be '/?ajaxify_on=true&push_state_enabled=false'
+	end
+
+  it "should display loader" do 
+    click_on 'Page 1'
+    page.should have_css(".ajaxify_loader")
+  end
+
+  it "should load page's content" do    
+    should_have_content 'Page 1 Content'
+    hash_path_should_be '/home/page1'
+    ensure_layout_not_reloaded
+  end
+
+  it "should redirect" do
+  	click_on 'Page 3 (redirects to page 1)'
+  	should_have_content 'Page 1 Content'
+  	hash_path_should_be '/home/page1'
+  	ensure_layout_not_reloaded
+  end
+
+  it "should display flash" do
+  	should_have_notice 'Redirected to Page 1'
+  end
+
+  it "should submit a form" do
+  	click_on 'Page 2'
+  	fill_in 'input1', with: 'test'
+  	check 'check_me'
+  	hash_path_should_be '/home/page2'
+  	click_button 'Submit me'
+  	should_have_notice 'Form submitted (test, , 1)'
+  	ensure_layout_not_reloaded
+  end
+
+  it "should load previous content when clicking the browser back button" do
+  	click_on 'Home'
+  	wait_for_ajaxify_loaded
+ 		click_on 'Page 1'
+  	wait_for_ajaxify_loaded
+  	click_browser_back_button
+  	should_have_content 'Home Content'
+  	hash_path_should_be '/'
+  	ensure_layout_not_reloaded
+	end
+
+	it "should load next content when clicking the browser forward button" do
+  	click_browser_forward_button
+  	should_have_content 'Page 1 Content'
+  	hash_path_should_be '/home/page1'
+  	ensure_layout_not_reloaded
+	end
+
+	it "should not repost form clicking the browser back button" do
+		click_on 'Page 2'
+  	click_button 'Submit me'
+  	wait_for_ajaxify_loaded
+  	click_on 'Page 1'
+  	wait_for_ajaxify_loaded
+  	click_browser_back_button
+  	wait_for_ajaxify_loaded
+  	page.should_not have_css("#notice", visible: true)
+  	should_have_content /A Form/
+  	hash_path_should_be '/home/page2'
+	end
+
+end
+
+
+describe 'Test convert urls', :js => true do
+
+	it 'should convert a hash url to a proper one in a browser supporting pushState' do
+		visit root_path(ajaxify_on: true, anchor: '/home/page1')
+		should_have_content 'Page 1 Content'
+		push_state_path_should_be '/home/page1'
+	end
+
+	it 'should convert a proper url to a hash url in a browser not supporting pushState' do
+		visit '/home/page1?ajaxify_on=true&push_state_enabled=false'
+		should_have_content 'Page 1 Content'
+		hash_path_should_be '/home/page1?ajaxify_on=true&push_state_enabled=false'
+	end
+
+end

spec/spec_helper.rb

@@ -0,0 +1,87 @@
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# Require this file using `require "spec_helper.rb"` to ensure that it is only
+# loaded once.
+#
+# See http://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+
+ENV["RAILS_ENV"] = 'test'
+require_relative "ajaxify_test/config/environment"
+
+require 'rspec/rails'
+require 'rspec-steps'
+require 'sqlite3'
+
+require 'haml'
+
+# Add this to load Capybara integration:
+require 'capybara/rspec'
+require 'capybara/rails'
+
+# require 'capybara/poltergeist'
+
+# Capybara.register_driver :poltergeist do |app|
+#   # Specifiying the phantomjs path only for RubyMine which fails to pick it up otherwise
+#   Capybara::Poltergeist::Driver.new(app, phantomjs: '/usr/local/bin/phantomjs') #, inspector: true
+# end
+
+# Capybara.javascript_driver = :poltergeist
+# Capybara.current_driver    = :poltergeist
+
+Capybara.default_selector  = :css
+Capybara.default_wait_time = 3
+
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  #config.run_all_when_everything_filtered = true
+  config.use_transactional_fixtures = false
+  config.include Capybara::DSL
+end
+
+
+def wait_until_gone css
+	page.should have_no_css(css)
+end
+
+def wait_for_ajaxify_loaded
+  wait_until_gone ".ajaxify_loader"
+end
+
+def click_browser_back_button
+	page.execute_script("window.history.back()")
+end
+
+def click_browser_forward_button
+	page.execute_script("window.history.forward()")
+end
+
+def ensure_layout_not_reloaded
+	find('body')['data-id'].to_s.should eq(@layout_id), "Layout has been reloaded but shouldn't"
+end
+
+def should_have_content content
+	page.should have_css('#main', text: content)
+end
+
+def should_have_notice notice
+	page.should have_css('#notice', text: notice)
+end
+
+def disable_push_state
+	page.execute_script("window.history.pushState = null;")
+end
+
+def current_hash
+	URI.parse(current_url).fragment.to_s
+end
+
+def hash_path_should_be path
+	current_hash.should == path
+	current_path.should == '/'
+end
+
+def push_state_path_should_be path
+	current_hash.should == ''
+	current_path.should == path
+end
+