Permalink
Browse files

Fix use of travis:after and add -t to the defaults.

Run travis:fake_config before check_manifest to ensure an ignore for .travis is set
  • Loading branch information...
1 parent a268028 commit 74ac60eba500f0da93f2ca290671b95ae0465a57 @drbrain committed Mar 11, 2012
Showing with 5 additions and 4 deletions.
  1. +2 −1 lib/hoe/travis.rb
  2. +3 −3 test/test_hoe_travis.rb
View
@@ -131,7 +131,7 @@ module Hoe::Travis
'rake travis:before -t',
],
'after_script' => [
- 'rake check_manifest', # separate step so clean and generate work
+ 'rake travis:after -t',
],
'script' => 'rake travis',
'token' => 'FIX - See: ri Hoe::Travis',
@@ -156,6 +156,7 @@ def define_travis_tasks
namespace :travis do
desc "Run by travis-ci after running the default checks"
task :after => %w[
+ travis:fake_config
check_manifest
]
@@ -33,7 +33,7 @@ def test_define_travis_tasks
assert_equal %w[test], travis.prerequisites
after = Rake::Task['travis:after']
- assert_equal %w[check_manifest], after.prerequisites
+ assert_equal %w[travis:fake_config check_manifest], after.prerequisites
before = Rake::Task['travis:before']
assert_equal %w[install_plugins check_extra_deps], before.prerequisites
@@ -62,7 +62,7 @@ def test_define_travis_tasks
def test_travis_after_script
expected = [
- 'rake check_manifest',
+ 'rake travis:after -t',
]
assert_equal expected, @hoe.travis_after_script
@@ -223,7 +223,7 @@ def @hoe.have_gem?(name) false end
expected = YAML.load <<-TRAVIS_YML
---
after_script:
-- rake check_manifest
+- rake travis:after -t
before_script:
- gem install hoe-travis --no-rdoc --no-ri
- rake travis:before -t

0 comments on commit 74ac60e

Please sign in to comment.