Skip to content

Commit

Permalink
integration tests do not need to load fixtures by hand
Browse files Browse the repository at this point in the history
I don't know when this changed, checked back up to 3.2 and integration
tests instantiate fixtures just fine.
  • Loading branch information
fxn committed May 24, 2014
1 parent 0bd909a commit 36e3ca6
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 11 deletions.
3 changes: 0 additions & 3 deletions guides/code/getting_started/test/test_helper.rb
Expand Up @@ -6,9 +6,6 @@ class ActiveSupport::TestCase
ActiveRecord::Migration.check_pending!

# Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
#
# Note: You'll currently still have to declare fixtures explicitly in integration tests
# -- they do not yet inherit this setting
fixtures :all

# Add more helper methods to be used by all tests here...
Expand Down
5 changes: 0 additions & 5 deletions guides/source/testing.md
Expand Up @@ -699,8 +699,6 @@ A simple integration test that exercises multiple controllers:
require 'test_helper'
class UserFlowsTest < ActionDispatch::IntegrationTest
fixtures :users
test "login and browse site" do
# login via https
https!
Expand All @@ -727,10 +725,7 @@ Here's an example of multiple sessions and custom DSL in an integration test
require 'test_helper'
class UserFlowsTest < ActionDispatch::IntegrationTest
fixtures :users
test "login and browse site" do
# User david logs in
david = login(:david)
# User guest logs in
Expand Down
Expand Up @@ -5,9 +5,6 @@
class ActiveSupport::TestCase
<% unless options[:skip_active_record] -%>
# Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
#
# Note: You'll currently still have to declare fixtures explicitly in integration tests
# -- they do not yet inherit this setting
fixtures :all
<% end -%>
Expand Down

0 comments on commit 36e3ca6

Please sign in to comment.