railsthemes 0.0.1 → 1.0.0

data/Gemfile

@@ -2,11 +2,14 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in railsthemes.gemspec
 gemspec
+gem 'thor'
+gem 'rest-client'
 
 # development gems
 group :test do
   gem 'rspec'
   gem 'rr'
   gem 'autotest'
-  gem "fakefs", :require => 'fakefs/safe'
+  gem 'fakefs', :require => 'fakefs/safe'
+  gem 'fakeweb'
 end

data/README.md

@@ -1,18 +1,10 @@
 # Railsthemes
 
-TODO: Write a gem description
+This is a gem that helps you install a theme for your Rails app from RailsThemes.com. First, purchase a theme from the website. Then, install this gem and run it. You will get a download code, and you can pass this to the installer to install the theme (see below usage.) The themes at RailsThemes.com work for Rails 3.1+ (any that use the asset pipeline.)
 
 ## Installation
 
-Add this line to your application's Gemfile:
-
-    gem 'railsthemes'
-
-And then execute:
-
-    $ bundle
-
-Or install it yourself as:
+Install using rubygems:
 
     $ gem install railsthemes
 
@@ -20,7 +12,13 @@ Or install it yourself as:
 
 Desired usage:
 
-    $ railsthemes install my_hash
+    $ railsthemes install download_code
+
+## Other notes
+
+This has been tested with Ruby 1.8.7, and should work with higher versions. 1.8.7 is the lowest version that Rails 3.1+ uses.
+
+Not sure if this will work for Windows users due to invoking tar on the command-line instead of using native Ruby. This is something that we can change going forward, just a matter of time.
 
 ## Contributing
 

data/bin/railsthemes

@@ -1,6 +1,26 @@
 #!/usr/bin/env ruby
+require "rubygems"
+require "thor"
+require "railsthemes"
 
-require 'railsthemes'
+class Installer < Thor
+  desc "install CODE", "Install from RailsThemes.com using your download code"
+  method_option :file, :aliases => "-f", :desc => "Use local filename instead of downloading", :type => :boolean
 
-installer = Railsthemes::Installer.new
-installer.execute ARGV
+  def install code_or_file
+    if options[:file]
+      file = code_or_file
+      abort 'Please specify a file to install from' unless file
+      puts "Installing from file: #{file}."
+      Railsthemes::Installer.new.install_from_file_system file
+    else
+      code = code_or_file
+      abort 'Please specify a code to install from' unless code
+      Railsthemes::Installer.new.download_from_code code
+    end
+  end
+
+  # AP: other options could include list, remove, help, etc.
+end
+
+Installer.start ARGV

data/lib/railsthemes/installer.rb

@@ -1,65 +1,47 @@
+require 'rubygems'
 require 'date'
 require 'fileutils'
 require 'logger'
 require 'tmpdir'
+require 'bundler'
+require 'net/http'
+require 'rest-client'
 
 module Railsthemes
   class Installer
     def initialize logger = nil
       @logger = logger
+      @server = File.exist?('railsthemes_server') ? File.read('railsthemes_server') : 'https://railsthemes.com'
       @logger ||= Logger.new(STDOUT)
+      # just print out basic information, not all of the extra logger stuff
       @logger.formatter = proc do |severity, datetime, progname, msg|
         "#{msg}\n"
       end
     end
 
-    def execute args = []
-      if args[0] == 'install' && args.length > 1
-        install *args[1..-1]
-      else
-        print_usage
-      end
-    end
-
-    def install *args
-      ensure_in_rails_root
-      if args[0] == '--file'
-        if args[1]
-          install_from_file_system args[1]
-        else
-          print_usage_and_abort "The parameter --file means we need another parameter after it to specify what file to load from."
-        end
-      elsif args[0] == '--help'
-        print_usage
-      else
-        if args[0]
-          download_from_hash args[0]
-        else
-          print_usage_and_abort "railsthemes expects the hash that you got from the website as a parameter in order to download the theme you bought."
-        end
-      end
-    end
-
     def ensure_in_rails_root
       unless File.directory?('app') && File.directory?('public')
         Safe.log_and_abort 'Must be in the Rails root directory to use this gem.'
       end
     end
 
-    def install_from_file_system filepath
-      if File.directory?(filepath)
-        files = files_under(filepath)
-        @logger.info 'Copying assets...'
-        files.each do |file|
-          copy_with_replacement filepath, file
-        end
-        @logger.info 'Done copying assets.'
+    def install_from_file_system source_filepath
+      if File.directory?(source_filepath)
+        ensure_in_rails_root
+
+        @logger.info 'Installing...'
+        FileUtils.cp_r(File.join(source_filepath, 'base', '.'), '.')
+        gemfile_contents = File.read('Gemfile.lock')
+        install_gems_from(source_filepath, gems_used(File.read('Gemfile.lock')) - ['haml', 'sass'])
+
+        @logger.info 'Done installing.'
         post_copying_changes
         print_post_installation_instructions
-      elsif archive?(filepath)
-        if File.exists?(filepath)
-          install_from_archive filepath
-          # no need for post_installation, because we haven't
+      elsif archive?(source_filepath)
+        if File.exists?(source_filepath)
+          install_from_archive source_filepath
+          # no need for post_installation, because we will do this in the
+          # install_from_archive method
         else
           Safe.log_and_abort 'Cannot find the file you specified.'
         end
@@ -68,98 +50,183 @@ module Railsthemes
       end
     end
 
-    # this happens after a successful copy so that we set up the environment correctly
-    def post_copying_changes
-      Utils.remove_file File.join('public', 'index.html')
-      @logger.info 'Analyzing existing project structure...'
-      create_welcome_controller unless routes_defined?
+    def install_gems_from source_filepath, gem_names
+      gems_that_we_can_install = Dir.entries("#{source_filepath}/gems").reject{|x| x == '.' || x == '..'}
+      (gem_names & gems_that_we_can_install).each do |gem_name|
+        FileUtils.cp_r(File.join(source_filepath, 'gems', gem_name, '.'), '.')
+      end
     end
 
-    def create_welcome_controller
-      Safe.system_call('rails g controller Welcome index')
-      lines = []
-      File.open('config/routes.rb').each do |line|
-        if line =~ /  # root :to => 'welcome#index'/
-          lines << "  root :to => 'welcome#index'"
-        else
-          lines << line
-        end
+    def install_from_archive filepath
+      @logger.info "Extracting..."
+      with_tempdir do |tempdir|
+        Safe.system_call untar_string(filepath, tempdir)
+        @logger.info "Finished extracting."
+        install_from_file_system tempdir
       end
-      File.open('config/routes.rb', 'w') do |f|
-        lines.each do |line|
-          f.puts line
+    end
+
+    def download_from_code code
+      check_vcs_status
+      if File.exists?('Gemfile.lock')
+        @logger.info "Downloading..."
+        with_tempdir do |tempdir|
+          archive = File.join(tempdir, 'archive.tar.gz')
+          send_gemfile code
+          config = get_primary_configuration(File.read('Gemfile.lock'))
+          dl_url = get_download_url "#{@server}/download?code=#{code}&config=#{config}"
+          if dl_url
+            Utils.download_file_to dl_url, archive
+            @logger.info "Finished downloading."
+            install_from_archive archive
+          else
+            Safe.log_and_abort("We didn't understand the code you gave to download the theme (#{code})")
+          end
         end
+      else
+        Safe.log_and_abort("We could not find your Gemfile.lock file.") unless File.exists?('Gemfile.lock')
       end
     end
 
-    def routes_defined?
-      Safe.system_call('rake routes').length > 0
+    def get_download_url server_request_url
+      response = nil
+      begin
+        response = Net::HTTP.get_response URI.parse(server_request_url)
+      rescue Exception => e
+        #@logger.info e.message
+        #@logger.info e.backtrace
+      end
+
+      response.body if response && response.code.to_s == '200'
     end
 
-    def install_from_archive filepath
-      tempdir = generate_tmpdir
-      Dir.mkdir tempdir
-      Safe.system_call untar_string(filepath, tempdir)
-      install_from_file_system tempdir
-      #FileUtils.rm_rf tempdir
+    def gems_used
+      gems_used File.read('Gemfile.lock')
     end
 
-    def files_under filepath, accum = []
-      files = Dir.chdir(filepath) { Dir['**/*'] }
-      files.select{|f| !File.directory?(File.join(filepath, f))}
+    def gems_used contents
+      return [] if contents.strip == ''
+      lockfile = Bundler::LockfileParser.new(contents)
+      lockfile.specs.map(&:name)
     end
 
-    def download_from_hash hash
-      @logger.info "Downloading from hash #{hash}"
+    def get_primary_configuration contents
+      gems = gems_used(contents)
+      to_return = []
+      to_return << (gems.include?('haml') ? 'haml' : 'erb')
+      to_return << (gems.include?('sass') ? 'scss' : 'css')
+      to_return * ','
     end
 
-    def copy_with_replacement filepath, entry
-      if File.exists?(entry)
-        # not sure if I should put in a cp -f here, might be better to toss error
-        # so I'm using rename instead
-        File.rename entry, "#{entry}.old"
+    def send_gemfile code
+      begin
+        response = RestClient.post("#{@server}/gemfiles/parse",
+          :code => code, :gemfile_lock => File.new('Gemfile.lock', 'rb'))
+      rescue Exception => e
+        #@logger.info e.message
+        #@logger.info e.backtrace
       end
-      Utils.copy_with_path File.join(filepath, entry), entry
     end
 
-    def generate_tmpdir
-      File.join(Dir.tmpdir, DateTime.now.strftime("railsthemes-%Y%m%d-%H%M%s"))
+    def with_tempdir &block
+      tempdir = generate_tempdir_name
+      FileUtils.mkdir_p tempdir
+      yield tempdir
+      FileUtils.rm_rf tempdir
+    end
+
+    def generate_tempdir_name
+      File.join(Dir.tmpdir, DateTime.now.strftime("railsthemes-%Y%m%d-%H%M%S-#{rand(100000000)}"))
     end
 
     def archive? filepath
-      filepath =~ /\.tar$/ || filepath =~ /\.tar\.gz$/
+      filepath =~ /\.tar\.gz$/
     end
 
     def untar_string filepath, newdirpath
-      gzipped = (filepath =~ /\.gz$/) ? 'z' : ''
-      "tar -#{gzipped}xf #{filepath} --strip 1"
+      "tar -zxf #{filepath}"
     end
 
-    def print_usage
-      @logger.info <<-EOS
-Usage:
-------
-railsthemes install HASH
-  install a theme from the railsthemes website
 
-railsthemes install --help
-  this message
+    # this happens after a successful copy so that we set up the environment correctly
+    # for people to view the theme correctly
+    def post_copying_changes
+      Utils.remove_file File.join('public', 'index.html')
+      @logger.info 'Analyzing existing project structure...'
+      create_railsthemes_controller
+    end
+
+    def create_railsthemes_controller
+      FileUtils.mkdir_p(File.join('app', 'controllers'))
+      File.open(File.join('app', 'controllers', 'railsthemes_controller.rb'), 'w') do |f|
+        f.write <<-EOS
+class RailsthemesController < ApplicationController
+  # normally every view will use your application layout
+  def inner
+    render :layout => 'application'
+  end
 
-railsthemes install --file filepath
-  install from the local filesystem
-      EOS
+  # this is a special layout
+  def landing
+    render :layout => 'landing'
+  end
+end
+        EOS
+      end
+
+      lines = []
+      if File.exists?('config/routes.rb')
+        lines = File.read('config/routes.rb').split("\n")
+        last = lines.pop
+        lines << "  match 'railsthemes/landing' => 'railsthemes#landing'"
+        lines << "  match 'railsthemes/inner' => 'railsthemes#inner'"
+        lines << last
+        File.open('config/routes.rb', 'w') do |f|
+          lines.each do |line|
+            f.puts line
+          end
+        end
+      end
+    end
+
+    def check_vcs_status
+      result = ''
+      variety = ''
+      if File.directory?('.git')
+        variety = 'Git'
+        result = Safe.system_call('git status -s')
+      elsif File.directory?('.hg')
+        variety = 'Mercurial'
+        result = Safe.system_call('hg status')
+      elsif File.directory?('.svn')
+        variety = 'Subversion'
+        result = Safe.system_call('svn status')
+      end
+      unless result.size == 0
+        Safe.log_and_abort("\n#{variety} reports that you have the following pending changes:\n#{result}Please stash or commit the changes before proceeding to ensure that you can roll back after installing if you want.")
+      end
     end
 
     def print_post_installation_instructions
       @logger.info <<-EOS
-Your theme is installed!
 
-Make sure that you restart your server if it's currently running.
+Yay! Your theme is installed!
+
+=============================
+
+What now?
+1) Ensure your new application layout file contains everything that you wanted
+   from the old one.
+2) Restart your development server if it is currently running (the asset pipeline can
+   be finnicky.)
+3) Check out the samples at:
+   http://localhost:3000/railsthemes/landing
+   http://localhost:3000/railsthemes/inner
+4) Let us know how it went: @railsthemes or team@railsthemes.com
       EOS
     end
 
     def print_usage_and_abort s
-      print_usage
       Safe.log_and_abort s
     end
   end

data/lib/railsthemes/safe.rb

@@ -4,16 +4,11 @@ require 'fileutils'
 module Railsthemes
   class Safe
     def self.system_call s
-      verify_not_testing
       `#{s}`
     end
 
     def self.log_and_abort s
       abort s
     end
-
-    def self.verify_not_testing
-      false
-    end
   end
 end

data/lib/railsthemes/utils.rb

@@ -1,17 +1,30 @@
 require 'fileutils'
+require 'open-uri'
 
 # a bunch of things that should never be called in testing due to side effects
 module Railsthemes
   class Utils
+    # remove file only if it exists
     def self.remove_file filepath
       if File.exists?(filepath)
         File.delete filepath
       end
     end
 
-    def self.copy_with_path src, dest
+    # copy a file, ensuring that the directory is present
+    def self.copy_ensuring_directory_exists src, dest
       FileUtils.mkdir_p(File.dirname(dest)) # create directory if necessary
       FileUtils.cp src, dest
     end
+
+    # would be nice to put download status in the output (speed, progress, etc.)
+    def self.download_file_to url, save_to
+      File.open(save_to, "wb") do |saved_file|
+        # the following "open" is provided by open-uri
+        open(url) do |read_file|
+          saved_file.write(read_file.read)
+        end
+      end
+    end
   end
 end

data/lib/railsthemes/version.rb

@@ -1,5 +1,5 @@
 module Railsthemes
   unless defined?(Railsthemes::VERSION)
-    VERSION = "0.0.1"
+    VERSION = "1.0.0"
   end
 end

data/railsthemes.gemspec

@@ -2,8 +2,8 @@
 require File.expand_path('../lib/railsthemes/version', __FILE__)
 
 Gem::Specification.new do |gem|
-  gem.authors       = ["railsthemes"]
-  gem.email         = ["panozzaj@gmail.com"]
+  gem.authors       = ["Railsthemes"]
+  gem.email         = ["anthony@railsthemes.com"]
   gem.description   = %q{railsthemes.com installer gem}
   gem.summary       = %q{Installs gems from railsthemes.com}
   gem.homepage      = "https://github.com/RailsThemes/railsthemes"
@@ -14,4 +14,7 @@ Gem::Specification.new do |gem|
   gem.name          = "railsthemes"
   gem.require_paths = ["lib"]
   gem.version       = Railsthemes::VERSION
+
+  gem.add_dependency "thor"
+  gem.add_dependency "rest-client"
 end

data/spec/fixtures/blank-assets/base/app/assets/stylesheets/style.css.erb

@@ -0,0 +1 @@
+style.css.erb

data/spec/lib/railsthemes/installer_spec.rb

@@ -0,0 +1,259 @@
+require 'spec_helper'
+require 'railsthemes'
+
+describe Railsthemes::Installer do
+  def using_gems *gems
+    "GEM\nremote: https://rubygems.org/\nspecs:\n" +
+      gems.map{|gem| "    #{gem}"}.join("\n") +
+      "\nGEM\n  remote: https://rubygems.org/"
+  end
+
+  before do
+    @logger = Logger.new(File.join Dir.tmpdir, 'railsthemes.log')
+    @installer = Railsthemes::Installer.new @logger
+    stub(@installer).ensure_in_rails_root
+    stub(@installer).generate_tempdir_name { '/tmp' }
+    FileUtils.touch('Gemfile.lock')
+  end
+
+  describe :install_from_file_system do
+    context 'when the filepath is a directory' do
+      it 'should copy the files from that directory into the Rails app' do
+        FileUtils.mkdir_p('filepath/base')
+        FileUtils.touch('filepath/base/a')
+        FileUtils.touch('filepath/base/b')
+        FileUtils.mkdir_p('filepath/gems')
+        mock(@installer).post_copying_changes
+
+        @installer.install_from_file_system('filepath')
+        File.exists?('a').should be_true
+        File.exists?('b').should be_true
+      end
+    end
+
+    context 'when the filepath is an archive file' do
+      it 'should extract the archive file to a temp directory if the archive exists' do
+        archive = 'tarfile.tar.gz'
+        FileUtils.touch archive
+        mock(@installer).install_from_archive archive
+        @installer.install_from_file_system archive
+      end
+
+      it 'should print an error message and exit if the archive cannot be found' do
+        mock(Railsthemes::Safe).log_and_abort(/Cannot find/)
+        @installer.install_from_file_system 'tarfile.tar.gz'
+      end
+    end
+
+    context 'otherwise' do
+      it 'should print usage and report an error reading the file' do
+        mock(@installer).print_usage_and_abort(/either/)
+        @installer.install_from_file_system("does not exist")
+      end
+    end
+  end
+
+  describe :install_gems_from do
+    it 'should install the gems that we specify that match' do
+      FakeFS::FileSystem.clone('spec/fixtures')
+      @installer.install_gems_from("spec/fixtures/blank-assets", ['formtastic', 'kaminari'])
+      File.exist?(File.join('app', 'assets', 'stylesheets', 'formtastic.css.scss')).should be_true
+      File.exist?(File.join('app', 'assets', 'stylesheets', 'kaminari.css.scss')).should be_false
+      File.exist?(File.join('app', 'assets', 'stylesheets', 'simple_form.css.scss')).should be_false
+    end
+  end
+
+  describe :install_from_archive do
+    it 'should extract the archive correctly' do
+      mock(@installer).install_from_file_system '/tmp'
+      mock(@installer).untar_string('filepath', anything) { 'untar string' }
+      mock(Railsthemes::Safe).system_call('untar string')
+      @installer.install_from_archive 'filepath'
+    end
+  end
+
+  describe :untar_string do
+    it 'should return correct value for *.tar.gz file' do
+      result = @installer.untar_string 'file.tar.gz', 'newdirpath'
+      result.should == 'tar -zxf file.tar.gz'
+    end
+  end
+
+  describe :archive? do
+    it 'should be true for tar.gz file' do
+      @installer.archive?('test/a/b/c/d.tar.gz').should be_true
+    end
+
+    it 'should be false for other extensions' do
+      @installer.archive?('test/a/b/c.tar/d.zip').should be_false
+    end
+  end
+
+  describe 'end to end operation' do
+    def verify_end_to_end_operation
+      ['app/assets/images/image1.png',
+       'app/assets/images/bg/sprite.png',
+       'app/assets/javascripts/jquery.dataTables.js',
+       'app/assets/javascripts/scripts.js.erb',
+       'app/assets/stylesheets/style.css.erb',
+       'app/views/layouts/_interior_sidebar.html.erb',
+       'app/views/layouts/application.html.erb',
+       'app/views/layouts/homepage.html.erb'].each do |filename|
+         File.exist?(filename).should be_true, "#{filename} was not present"
+      end
+      File.open('app/assets/stylesheets/style.css.erb').each do |line|
+        line.should match /style.css.erb/
+      end
+    end
+
+    before do
+      stub(@installer).post_copying_changes
+      FakeFS::FileSystem.clone('spec/fixtures')
+    end
+
+    it 'should extract correctly from directory' do
+      filename = 'spec/fixtures/blank-assets'
+      @installer.install_from_file_system filename
+      verify_end_to_end_operation
+    end
+
+    # TODO need to use a pure ruby solution to get this to mock in the file system right
+    it 'should extract correctly from archive' do
+      pending 'needs pure ruby untar solution'
+    end
+  end
+
+  describe :send_gemfile do
+    before do
+      File.open('Gemfile.lock', 'w') do |file|
+        file.puts "GEM\n  remote: https://rubygems.org/"
+      end
+    end
+
+    it 'should hit the server with the Gemfile and return the results, arrayified' do
+      FakeFS.deactivate! # has an issue with generating tmpfiles otherwise
+      params = { :code => 'panozzaj@gmail.com:code', :gemfile_lock => File.new('Gemfile.lock', 'rb') }
+      FakeWeb.register_uri :post, 'https://railsthemes.com/gemfiles/parse',
+        :body => 'haml,scss', :parameters => params
+      @installer.send_gemfile('panozzaj@gmail.com:code')
+    end
+
+    it 'should return a blank array when there are issues' do
+      FakeFS.deactivate! # has an issue with generating tmpfiles otherwise
+      FakeWeb.register_uri :post, 'https://railsthemes.com/gemfiles/parse',
+        :body => '', :parameters => :any, :status => ['401', 'Unauthorized']
+      @installer.send_gemfile('panozzaj@gmail.com:code')
+    end
+  end
+
+  describe :download_from_code do
+    context 'when a gemfile.lock is not present' do
+      it 'should fail with a good message' do
+        File.unlink('Gemfile.lock')
+        mock(Railsthemes::Safe).log_and_abort(/could not find/)
+        @installer.download_from_code 'anything'
+      end
+    end
+
+    context 'when a gemfile.lock is present' do
+      before do
+        mock(@installer).check_vcs_status
+        mock(@installer).send_gemfile('panozzaj@gmail.com:code')
+      end
+
+      it 'should download the file correctly when valid configuration' do
+        FakeWeb.register_uri :get,
+          /download\?code=panozzaj@gmail.com:code&config=haml,scss/,
+          :body => 'auth_url'
+        mock(@installer).get_primary_configuration('') { 'haml,scss' }
+        mock(Railsthemes::Utils).download_file_to('auth_url', '/tmp/archive.tar.gz')
+        mock(@installer).install_from_archive '/tmp/archive.tar.gz'
+        @installer.download_from_code 'panozzaj@gmail.com:code'
+      end
+
+      it 'should fail with an error message on any error message' do
+        FakeWeb.register_uri :get,
+          'https://railsthemes.com/download?code=panozzaj@gmail.com:code&config=',
+          :body => '', :status => ['401', 'Unauthorized']
+        mock(@installer).get_primary_configuration('') { '' }
+        mock(Railsthemes::Safe).log_and_abort(/didn't understand/)
+        @installer.download_from_code 'panozzaj@gmail.com:code'
+      end
+    end
+  end
+
+  describe '#get_primary_configuration' do
+    it 'should give haml+scss when haml and sass are in the Gemfile' do
+      gemfile = using_gems 'haml', 'sass'
+      @installer.get_primary_configuration(gemfile).should == 'haml,scss'
+    end
+
+    it 'should give haml+css when sass is not in the Gemfile' do
+      gemfile = using_gems 'haml'
+      @installer.get_primary_configuration(gemfile).should == 'haml,css'
+    end
+
+    it 'should give erb, scss when haml is not in the gemfile' do
+      gemfile = using_gems 'sass'
+      @installer.get_primary_configuration(gemfile).should == 'erb,scss'
+    end
+
+    it 'should give erb, scss when haml is not in the gemfile' do
+      gemfile = using_gems
+      @installer.get_primary_configuration(gemfile).should == 'erb,css'
+    end
+  end
+
+  describe :check_vcs_status do
+    context 'when git used' do
+      before do
+        Dir.mkdir('.git')
+      end
+
+      it 'should exit when the vcs is unclean' do
+        mock(Railsthemes::Safe).system_call('git status -s') { '# modified: installer_spec.rb' }
+        mock(Railsthemes::Safe).log_and_abort(/pending changes/)
+        @installer.check_vcs_status
+      end
+
+      it 'should do nothing significant when the vcs is clean' do
+        mock(Railsthemes::Safe).system_call('git status -s') { '' }
+        @installer.check_vcs_status
+      end
+    end
+
+    context 'when hg used' do
+      before do
+        Dir.mkdir('.hg')
+      end
+
+      it 'should exit when the vcs is unclean' do
+        mock(Railsthemes::Safe).system_call('hg status') { '? test.txt' }
+        mock(Railsthemes::Safe).log_and_abort(/pending changes/)
+        @installer.check_vcs_status
+      end
+
+      it 'should do nothing significant when the vcs is clean' do
+        mock(Railsthemes::Safe).system_call('hg status') { '' }
+        @installer.check_vcs_status
+      end
+    end
+
+    context 'when subversion used' do
+      before do
+        Dir.mkdir('.svn')
+      end
+
+      it 'should exit when the vcs is unclean' do
+        mock(Railsthemes::Safe).system_call('svn status') { 'M something.txt' }
+        mock(Railsthemes::Safe).log_and_abort(/pending changes/)
+        @installer.check_vcs_status
+      end
+
+      it 'should do nothing significant when the vcs is clean' do
+        mock(Railsthemes::Safe).system_call('svn status') { '' }
+        @installer.check_vcs_status
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,10 +1,12 @@
 require 'rubygems'
 require 'bundler/setup'
+require 'logger'
 require 'fakefs/spec_helpers'
-
-require 'railsthemes'
+require 'fakeweb'
 
 RSpec.configure do |config|
   config.mock_with :rr
   config.include FakeFS::SpecHelpers
 end
+
+FakeWeb.allow_net_connect = false

metadata

@@ -1,26 +1,53 @@
 --- !ruby/object:Gem::Specification 
 name: railsthemes
 version: !ruby/object:Gem::Version 
-  hash: 29
+  hash: 23
   prerelease: 
   segments: 
+  - 1
   - 0
   - 0
-  - 1
-  version: 0.0.1
+  version: 1.0.0
 platform: ruby
 authors: 
-- railsthemes
+- Railsthemes
 autorequire: 
 bindir: bin
 cert_chain: []
 
-date: 2012-03-10 00:00:00 Z
-dependencies: []
-
+date: 2012-04-20 00:00:00 Z
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: thor
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
+  name: rest-client
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :runtime
+  version_requirements: *id002
 description: railsthemes.com installer gem
 email: 
-- panozzaj@gmail.com
+- anthony@railsthemes.com
 executables: 
 - railsthemes
 extensions: []
@@ -42,7 +69,17 @@ files:
 - lib/railsthemes/utils.rb
 - lib/railsthemes/version.rb
 - railsthemes.gemspec
-- spec/lib/railsthemes_spec.rb
+- spec/fixtures/blank-assets/base/app/assets/images/bg/sprite.png
+- spec/fixtures/blank-assets/base/app/assets/images/image1.png
+- spec/fixtures/blank-assets/base/app/assets/javascripts/jquery.dataTables.js
+- spec/fixtures/blank-assets/base/app/assets/javascripts/scripts.js.erb
+- spec/fixtures/blank-assets/base/app/assets/stylesheets/style.css.erb
+- spec/fixtures/blank-assets/base/app/views/layouts/_interior_sidebar.html.erb
+- spec/fixtures/blank-assets/base/app/views/layouts/application.html.erb
+- spec/fixtures/blank-assets/base/app/views/layouts/homepage.html.erb
+- spec/fixtures/blank-assets/gems/formtastic/app/assets/stylesheets/formtastic.css.scss
+- spec/fixtures/blank-assets/gems/simple_form/app/assets/stylesheets/simple_form.css.scss
+- spec/lib/railsthemes/installer_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/RailsThemes/railsthemes
 licenses: []
@@ -73,10 +110,20 @@ required_rubygems_version: !ruby/object:Gem::Requirement
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.8.7
+rubygems_version: 1.8.21
 signing_key: 
 specification_version: 3
 summary: Installs gems from railsthemes.com
 test_files: 
-- spec/lib/railsthemes_spec.rb
+- spec/fixtures/blank-assets/base/app/assets/images/bg/sprite.png
+- spec/fixtures/blank-assets/base/app/assets/images/image1.png
+- spec/fixtures/blank-assets/base/app/assets/javascripts/jquery.dataTables.js
+- spec/fixtures/blank-assets/base/app/assets/javascripts/scripts.js.erb
+- spec/fixtures/blank-assets/base/app/assets/stylesheets/style.css.erb
+- spec/fixtures/blank-assets/base/app/views/layouts/_interior_sidebar.html.erb
+- spec/fixtures/blank-assets/base/app/views/layouts/application.html.erb
+- spec/fixtures/blank-assets/base/app/views/layouts/homepage.html.erb
+- spec/fixtures/blank-assets/gems/formtastic/app/assets/stylesheets/formtastic.css.scss
+- spec/fixtures/blank-assets/gems/simple_form/app/assets/stylesheets/simple_form.css.scss
+- spec/lib/railsthemes/installer_spec.rb
 - spec/spec_helper.rb

data/spec/lib/railsthemes_spec.rb

@@ -1,165 +0,0 @@
-require 'spec_helper'
-require 'logger'
-
-describe Railsthemes do
-  before do
-    @logger = Logger.new(File.join Dir.tmpdir, 'railsthemes.log')
-    @installer = Railsthemes::Installer.new @logger
-  end
-
-  describe :execute do
-    it 'should print usage if no params given' do
-      mock(@installer).print_usage
-      @installer.execute
-    end
-
-    it 'should run the installer if installer is the first parameter' do
-      mock(@installer).install 'a', 'b', 'c'
-      @installer.execute(['install', 'a', 'b', 'c'])
-    end
-  end
-
-  describe :install do
-    before do
-      stub(@installer).ensure_in_rails_root
-    end
-
-    context 'when --file is given as a parameter' do
-      it 'should read from the file argument' do
-        mock(@installer).install_from_file_system('filepath')
-        @installer.install '--file', 'filepath'
-      end
-
-      it 'should print usage and error message and exit if filepath is nil' do
-        dont_allow(@installer).install_from_file_system(anything)
-        dont_allow(@installer).download_from_hash(anything)
-        mock(@installer).print_usage_and_abort(/parameter/)
-        @installer.install '--file'
-      end
-    end
-
-    context 'hash given' do
-      it 'should download that hash' do
-        mock(@installer).download_from_hash('hash')
-        @installer.install 'hash'
-      end
-    end
-
-    context '--help given' do
-      it 'should print the usage' do
-        mock(@installer).print_usage
-        @installer.install '--help'
-      end
-    end
-
-    context 'when no parameters given' do
-      it 'should print usage and error message and exit' do
-        dont_allow(@installer).install_from_file_system(anything)
-        dont_allow(@installer).download_from_hash(anything)
-        mock(@installer).print_usage_and_abort(/parameter/)
-        @installer.install '--file'
-      end
-    end
-  end
-
-  describe :install_from_file_system do
-    context 'when the filepath is a directory' do
-      it 'should copy the files from that directory into the Rails app' do
-        FileUtils.mkdir('filepath')
-        FileUtils.touch('filepath/a')
-        FileUtils.touch('filepath/b')
-        mock(@installer).post_copying_changes
-
-        mock(@installer).copy_with_replacement('filepath', /a$/)
-        mock(@installer).copy_with_replacement('filepath', /b$/)
-        @installer.install_from_file_system('filepath')
-      end
-    end
-
-    context 'when the filepath is an archive file' do
-      it 'should extract the archive file to a temp directory if the archive exists' do
-        archive = 'tarfile.tar'
-        FileUtils.touch archive
-        mock(@installer).install_from_archive archive
-        @installer.install_from_file_system archive
-      end
-
-      it 'should print an error message and exit if the archive cannot be found' do
-        mock(Railsthemes::Safe).log_and_abort(/Cannot find/)
-        @installer.install_from_file_system 'tarfile.tar'
-      end
-    end
-
-    context 'otherwise' do
-      it 'should print usage and report an error reading the file' do
-        mock(@installer).print_usage_and_abort(/either/)
-        @installer.install_from_file_system("does not exist")
-      end
-    end
-  end
-
-  describe :copy_with_replacement do
-    before do
-      FileUtils.mkdir 'fp'
-      FileUtils.touch 'fp/file'
-    end
-
-    context 'when the destination file does not exist' do
-      it 'should copy the file to the local directory' do
-        @installer.copy_with_replacement 'fp', 'file'
-        File.exists?('file').should be_true
-      end
-    end
-
-    context 'when the destination file exists' do
-      before do
-        FileUtils.touch 'file'
-      end
-
-      it 'should make a backup of existing file if it is present' do
-        @installer.copy_with_replacement 'fp', 'file'
-        File.exists?('file').should be_true
-        File.exists?('file.old').should be_true
-      end
-    end
-  end
-
-  describe :install_from_archive do
-    it 'should extract the archive correctly' do
-      stub(@installer).generate_tmpdir { 'tmp' }
-      mock(@installer).install_from_file_system 'tmp'
-      mock(@installer).untar_string('filepath', anything) { 'untar string' }
-      mock(Railsthemes::Safe).system_call('untar string')
-      @installer.install_from_archive 'filepath'
-    end
-  end
-
-  describe :untar_string do
-    it 'should return correct value for *.tar.gz file' do
-      result = @installer.untar_string 'file.tar.gz', 'newdirpath'
-      result.should == 'tar -zxf file.tar.gz'
-    end
-
-    it 'should return correct value for *.tar file' do
-      result = @installer.untar_string 'file.tar', 'newdirpath'
-      result.should == 'tar -xf file.tar'
-    end
-  end
-
-  describe :archive? do
-    it 'should be true for tar file' do
-      @installer.archive?('test/a/b/c/d.tar').should be_true
-    end
-
-    it 'should be true for tar.gz file' do
-      @installer.archive?('test/a/b/c/d.tar.gz').should be_true
-    end
-
-    it 'should be false for other extensions' do
-      @installer.archive?('test/a/b/c.tar/d.zip').should be_false
-    end
-  end
-
-  describe :download_from_hash
-
-end