Skip to content

Fix: 'rake rails:templates:copy' doesn't work #6690

Merged
merged 2 commits into from Jun 11, 2012

3 participants

@suginoy
suginoy commented Jun 9, 2012

I fixed the copy source dir of stylesheet templates.

% rake rails:templates:copy
rake aborted!
No such file or directory - /Users/suginoyasuhiro/.rbenv/versions/1.9.3-p194/lib/ruby/gems/1.9.1/gems/railties-3.2.5/lib/rails/generators/rails/stylesheets/templates
@kennyj kennyj added a commit to kennyj/rails that referenced this pull request Jun 10, 2012
@kennyj kennyj Added a testcase for #6690. 1681b2c
@suginoy
suginoy commented Jun 11, 2012

kennyj's test failed before fix.

% bundle exec ruby -Ilib:test test/application/rake_test.rb --name=test_copy_templates
Run options: --name=test_copy_templates --seed 31705

# Running tests:

rake aborted!
No such file or directory - /Users/suginoyasuhiro/work/rails-bug/rails/railties/lib/rails/generators/rails/stylesheets/templates

Tasks: TOP => rails:templates:copy
(See full trace by running task with --trace)
[[:puke, [ApplicationTests::RakeTest, "test_copy_templates", #<MiniTest::Assertion: Failed assertion, no message given.>]]]

Finished tests in 3.987838s, 0.2508 tests/s, 0.0000 assertions/s.

  1) Failure:
test_copy_templates(ApplicationTests::RakeTest) [test/application/rake_test.rb:200]:
Failed assertion, no message given.

1 tests, 0 assertions, 1 failures, 0 errors, 0 skips

kennyj's test passes after fix.

% bundle exec ruby -Ilib:test test/application/rake_test.rb --name=test_copy_templates
Run options: --name=test_copy_templates --seed 21184

# Running tests:

.

Finished tests in 4.541046s, 0.2202 tests/s, 0.0000 assertions/s.

1 tests, 0 assertions, 0 failures, 0 errors, 0 skips
@rafaelfranca
Ruby on Rails member

Ok. I'll merge if it is fine to @kennyj.

@kennyj
kennyj commented Jun 11, 2012

@rafaelfranca It seems good for me :-)

@rafaelfranca rafaelfranca merged commit 04a7426 into rails:master Jun 11, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.