Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Remove MorePackager.

  • Loading branch information...
commit 594a6d0a6d5eab6d5865d060ed8560ca745e5b52 1 parent 3eb743c
Gabe Berke-Williams gabebw authored
4 README.md
View
@@ -11,8 +11,8 @@ rebase against master a couple times before your pull request can be merged.
## What does Kumade do?
Before deploying, Kumade ensures the git repo is clean.
After that, it packages assets using
-[Jammit](http://documentcloud.github.com/jammit/) and/or
-[More](https://github.com/cloudhead/more), commits them, and pushes to origin.
+[Jammit](http://documentcloud.github.com/jammit/) (if it's installed), commits
+them, and pushes to origin.
Then it force pushes to the correct Heroku remote, runs `rake db:migrate` on the
Heroku app, and then restarts the app.
1  features/kumade_executable.feature
View
@@ -31,7 +31,6 @@ Feature: Kumade executable
==> Restarted pretend-staging
==> Deployed to: pretend-staging
"""
- But the output should not contain "==> Packaged with Kumade::MorePackager"
Scenario: Default environment is staging
When I run kumade with "-p"
1  lib/kumade.rb
View
@@ -8,7 +8,6 @@ module Kumade
autoload :Configuration, "kumade/configuration"
autoload :Heroku, "kumade/heroku"
autoload :Packager, "kumade/packager"
- autoload :MorePackager, "kumade/packagers/more_packager"
autoload :JammitPackager, "kumade/packagers/jammit_packager"
autoload :NoopPackager, "kumade/packagers/noop_packager"
autoload :PackagerList, "kumade/packager_list"
2  lib/kumade/packager_list.rb
View
@@ -2,7 +2,7 @@ module Kumade
class PackagerList
include Enumerable
- PACKAGERS = [MorePackager, JammitPackager]
+ PACKAGERS = [JammitPackager]
def initialize
@packagers = build_packagers_list
20 lib/kumade/packagers/more_packager.rb
View
@@ -1,20 +0,0 @@
-begin
- require "less/more"
-rescue LoadError
-end
-
-module Kumade
- class MorePackager
- def self.assets_path
- File.join("public", ::Less::More.destination_path)
- end
-
- def self.installed?
- !!defined?(Less::More)
- end
-
- def self.package
- ::Less::More.generate_all
- end
- end
-end
16 spec/kumade/packager_list_spec.rb
View
@@ -3,28 +3,12 @@
describe Kumade::PackagerList, "detecting packages", :with_mock_outputter do
it "returns an array containing the Jammit packager if Jammit is installed" do
Kumade::JammitPackager.stubs(:installed? => true)
- Kumade::MorePackager.stubs(:installed? => false)
Kumade::PackagerList.new.to_a.should == [Kumade::JammitPackager]
end
- it "returns an array containing the More packager if More is installed" do
- Kumade::JammitPackager.stubs(:installed? => false)
- Kumade::MorePackager.stubs(:installed? => true)
-
- Kumade::PackagerList.new.to_a.should == [Kumade::MorePackager]
- end
-
- it "returns multiple packagers if they are installed" do
- Kumade::JammitPackager.stubs(:installed? => true)
- Kumade::MorePackager.stubs(:installed? => true)
-
- Kumade::PackagerList.new.to_a.should =~ [Kumade::JammitPackager, Kumade::MorePackager]
- end
-
it "returns an array containing the no-op packager if no other packagers are found" do
Kumade::JammitPackager.stubs(:installed? => false)
- Kumade::MorePackager.stubs(:installed? => false)
Kumade::PackagerList.new.to_a.should == [Kumade::NoopPackager]
end
37 spec/kumade/packagers/more_packager_spec.rb
View
@@ -1,37 +0,0 @@
-require "spec_helper"
-
-require "less"
-
-describe Kumade::MorePackager, :with_mock_outputter do
- subject { Kumade::MorePackager }
-
- before do
- define_constant "Less::More" do
- def self.destination_path
- "awesome_destination"
- end
- end
- end
-
- it_should_behave_like "packager"
-
- its(:assets_path) { should == File.join('public', ::Less::More.destination_path) }
-
- it "knows how to package itself" do
- Less::More.stubs(:generate_all)
-
- subject.package
-
- Less::More.should have_received(:generate_all).once
- end
-
- context "when More is defined" do
- before { Less::More }
- it { should be_installed }
- end
-
- context "when Less::More is not defined" do
- before { Less.send(:remove_const, :More) }
- it { should_not be_installed }
- end
-end
Please sign in to comment.
Something went wrong with that request. Please try again.