Skip to content

Commit

Permalink
Merge pull request #34 from n2uitive/master
Browse files Browse the repository at this point in the history
Support for *_tests.rb
  • Loading branch information
rymai committed Nov 3, 2012
2 parents 0614555 + 211230c commit 2cf77ff
Show file tree
Hide file tree
Showing 4 changed files with 21 additions and 9 deletions.
3 changes: 2 additions & 1 deletion lib/guard/test/inspector.rb
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@ def clear_test_files_list
end

def check_test_files(path)
Dir[File.join(path, '**', 'test_*.rb')] + Dir[File.join(path, '**', '*_test.rb')]
Dir[File.join(path, '**', 'test_*.rb')] +
Dir[File.join(path, '**', '*_test{s,}.rb')]
end
end

Expand Down
8 changes: 4 additions & 4 deletions spec/guard/test/inspector_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

it "should add all test files under the given dir" do
subject.clean(["test"]).should \
== ["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
== ["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
end

it "should remove non-test files" do
Expand All @@ -30,17 +30,17 @@

it "should remove duplication" do
subject.clean(["test", "test"]).should \
== ["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
== ["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
end

it "should remove test folder includes in other test folder" do
subject.clean(["test/unit", "test"]).should \
== ["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
== ["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
end

it "should remove test files includes in test folder" do
subject.clean(["test/unit/failing_test.rb", "test"]).should \
== ["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
== ["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"]
end

it "should remove test files includes in test folder (2)" do
Expand Down
8 changes: 4 additions & 4 deletions spec/guard/test_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@

describe "#run_all" do
it "runs all tests specified by the default :test_paths with a message" do
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"],:message => "Running all tests"
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"],:message => "Running all tests"
)
subject.run_all
end
Expand All @@ -68,7 +68,7 @@
runner.should_receive(:run).with(["test/unit/error/error_test.rb", "test/unit/failing_test.rb"]).and_return(false)
subject.run_on_changes(["test/unit"])

runner.should_receive(:run).with(["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"], :message => "Running all tests").and_return(true)
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"], :message => "Running all tests").and_return(true)
subject.run_all

runner.should_receive(:run).with(["test/unit/error/error_test.rb", "test/unit/failing_test.rb"]).and_return(true)
Expand All @@ -90,7 +90,7 @@
subject.reload

runner.should_receive(:run).with(["test/unit/error/error_test.rb", "test/unit/failing_test.rb"]).and_return(true)
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"], :message => "Running all tests").and_return(true)
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"], :message => "Running all tests").and_return(true)
subject.run_on_changes(["test/unit"])
end
end
Expand All @@ -116,7 +116,7 @@

it "calls #run_all by default if the changed specs pass after failing" do
runner.should_receive(:run).with(["test/succeeding_test.rb"]).and_return(false, true)
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"], :message => "Running all tests")
runner.should_receive(:run).with(["test/succeeding_test.rb", "test/succeeding_tests.rb", "test/test_old.rb", "test/unit/error/error_test.rb", "test/unit/failing_test.rb"], :message => "Running all tests")

subject.run_on_changes(["test/succeeding_test.rb"])
subject.run_on_changes(["test/succeeding_test.rb"])
Expand Down
11 changes: 11 additions & 0 deletions test/succeeding_tests.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,11 @@
require 'test_helper'

# encoding: utf-8
class SucceedingTests < Test::Unit::TestCase
def test_succeeding1
assert_equal(true, true)
end
def test_succeeding2
assert_equal(true, true)
end
end

0 comments on commit 2cf77ff

Please sign in to comment.