wordmove 0.1.0.rc → 1.0.0

data/.rspec

@@ -0,0 +1 @@
+--color

data/.rvmrc

@@ -0,0 +1 @@
+rvm use 1.9.2@wordmove

data/Gemfile.lock

@@ -1,58 +1,63 @@
 PATH
   remote: .
   specs:
-    wordmove (0.0.6)
-      activesupport (~> 3.0.0)
+    wordmove (0.1.0.rc)
+      activesupport
       colored
-      escape
-      hashie
       i18n
-      net-scp
-      net-ssh
+      photocopier
       rake
       thor
 
 GEM
   remote: http://rubygems.org/
   specs:
-    activesupport (3.0.11)
-    aruba (0.4.9)
-      childprocess (>= 0.2.3)
+    activesupport (3.2.9)
+      i18n (~> 0.6)
+      multi_json (~> 1.0)
+    aruba (0.5.1)
+      childprocess (~> 0.3.6)
       cucumber (>= 1.1.1)
-      ffi (= 1.0.11)
-      rspec (>= 2.7.0)
-    builder (3.0.0)
-    childprocess (0.2.3)
-      ffi (~> 1.0.6)
+      rspec-expectations (>= 2.7.0)
+    builder (3.1.4)
+    childprocess (0.3.6)
+      ffi (~> 1.0, >= 1.0.6)
     colored (1.2)
-    cucumber (1.1.4)
+    cucumber (1.2.1)
       builder (>= 2.1.2)
-      diff-lcs (>= 1.1.2)
-      gherkin (~> 2.7.1)
+      diff-lcs (>= 1.1.3)
+      gherkin (~> 2.11.0)
       json (>= 1.4.6)
-      term-ansicolor (>= 1.0.6)
     diff-lcs (1.1.3)
     escape (0.0.4)
-    ffi (1.0.11)
-    gherkin (2.7.1)
+    ffi (1.2.0)
+    gherkin (2.11.5)
       json (>= 1.4.6)
-    hashie (1.2.0)
-    i18n (0.6.0)
-    json (1.6.3)
+    i18n (0.6.1)
+    json (1.7.5)
+    multi_json (1.4.0)
     net-scp (1.0.4)
       net-ssh (>= 1.99.1)
-    net-ssh (2.2.1)
-    rake (0.9.2.2)
-    rspec (2.7.0)
-      rspec-core (~> 2.7.0)
-      rspec-expectations (~> 2.7.0)
-      rspec-mocks (~> 2.7.0)
-    rspec-core (2.7.1)
-    rspec-expectations (2.7.0)
-      diff-lcs (~> 1.1.2)
-    rspec-mocks (2.7.0)
-    term-ansicolor (1.0.7)
-    thor (0.14.6)
+    net-ssh (2.6.2)
+    net-ssh-gateway (1.1.0)
+      net-ssh (>= 1.99.1)
+    photocopier (0.0.4)
+      activesupport
+      escape
+      i18n
+      net-scp
+      net-ssh
+      net-ssh-gateway
+    rake (10.0.2)
+    rspec (2.12.0)
+      rspec-core (~> 2.12.0)
+      rspec-expectations (~> 2.12.0)
+      rspec-mocks (~> 2.12.0)
+    rspec-core (2.12.1)
+    rspec-expectations (2.12.0)
+      diff-lcs (~> 1.1.3)
+    rspec-mocks (2.12.0)
+    thor (0.16.0)
 
 PLATFORMS
   ruby

data/README.mdown

@@ -11,9 +11,20 @@ Think of it like Capistrano for Wordpress, complete with push/pull capabilities.
 That's easy:
 
 ```
-gem install wordmove --pre
+gem install wordmove
 ```
 
+## Upgrading?
+
+Beware!
+
+From version 1.0 we have decided to change wordmove flags' behaviour: they used to tell
+wordmove which options to **skip**, now they tell instead which options to **include**.
+In the Movefile, we have also changed in the ssh subsection the "username" field to be
+just "user".
+
+So please, be very careful when upgrading :heart:
+
 ## Usage
 
 ```
@@ -64,7 +75,7 @@ Just not use the `remote.ssh.password` field on your `Movefile`. Easy peasy.
 Please take a look at the various gotchas of the underlying [`photocopier` gem](https://github.com/stefanoverna/photocopier#password-gotchas).
 
 ### How the heck you are able to sync the DB via FTP?
-Thanks for asking. We basically upload via FTP a PHP script that performs the various
+We're glad you asked! We basically upload via FTP a PHP script that performs the various
 import/export operations. This script then gets executed via HTTP. Don't worry
 too much about security though: the script is deleted just after the usage,
 and can only be executed by `wordmove`, as each time it requires a pre-shared
@@ -77,7 +88,7 @@ one-time-password to be run.
 
 (The MIT License)
 
-Copyright © 2011 weLaika
+Copyright © 2012 weLaika
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the ‘Software’), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
 

data/lib/wordmove/deployer.rb

@@ -0,0 +1,166 @@
+require 'active_support/core_ext'
+require 'hashie'
+require 'wordmove/hosts/local_host'
+require 'wordmove/hosts/remote_host'
+require 'wordmove/logger'
+require 'wordmove/sql_mover'
+
+module Wordmove
+
+  class Deployer
+
+    attr_reader :options
+    attr_reader :logger
+
+    def initialize(options = {})
+      @options = Hashie::Mash.new(options)
+      @logger = Logger.new
+      @logger.level = options.verbose ? Logger::VERBOSE : Logger::INFO
+    end
+
+    def push
+      informative_errors do
+        unless options.skip_db
+          logger.info "Pushing the DB..."
+          push_db
+        end
+
+        remotely do |host|
+          %w(uploads themes plugins).each do |step|
+            unless options.send("skip_#{step}")
+              logger.info "Pushing wp-content/#{step}..."
+              host.download_dir local_wpcontent_path(step), remote_wpcontent_path(step)
+            end
+          end
+        end
+      end
+    end
+
+    def pull
+      informative_errors do
+        unless options.skip_db
+          logger.info "Pulling the DB..."
+          pull_db
+        end
+
+        remotely do |host|
+          %w(uploads themes plugins).each do |step|
+            unless options.send("skip_#{step}")
+              logger.info "Pulling wp-content/#{step}..."
+              host.upload_dir remote_wpcontent_path(step), local_wpcontent_path(step)
+            end
+          end
+        end
+      end
+    end
+
+    private
+
+    def push_db
+      local_mysql_dump_path = local_wpcontent_path("database_dump.sql")
+      remote_mysql_dump_path = remote_wpcontent_path("database_dump.sql")
+
+      locally do |host|
+        host.run "mysqldump", "--host=#{config.local.database.host}", "--user=#{config.local.database.username}", "--password=#{config.local.database.password}", config.local.database.name, :stdout => local_mysql_dump_path
+        if options.adapt_sql
+          Wordmove::SqlMover.new(local_mysql_dump_path, config.local, config.remote).move!
+        else
+          File.open(local_mysql_dump_path, 'a') do |file|
+            file.write "UPDATE #{options_table} SET option_value=\"#{config.remote.vhost}\" WHERE option_name=\"siteurl\" OR option_name=\"home\";\n"
+          end
+        end
+      end
+
+      remotely do |host|
+        host.download_file local_mysql_dump_path, remote_mysql_dump_path
+        host.run "mysql", "--user=#{config.remote.database.username}", "--password=#{config.remote.database.password}", "--host=#{config.remote.database.host}", "--database=#{config.remote.database.name}", :stdin => remote_mysql_dump_path
+        host.run "rm", remote_mysql_dump_path
+      end
+
+      locally do |host|
+        host.run "rm", local_mysql_dump_path
+      end
+    end
+
+
+    def pull_db
+      local_mysql_dump_path = local_wpcontent_path("database_dump.sql")
+      remote_mysql_dump_path = remote_wpcontent_path("database_dump.sql")
+
+      remotely do |host|
+        host.run "mysqldump", "--host=#{config.remote.database.host}", "--user=#{config.remote.database.username}", "--password=#{config.remote.database.password}", config.remote.database.name, :stdout => remote_mysql_dump_path
+        host.upload_file remote_mysql_dump_path, local_mysql_dump_path
+      end
+
+      locally do |host|
+        if options.adapt_sql
+          Wordmove::SqlMover.new(local_mysql_dump_path, config.remote, config.local).move!
+        else
+          File.open(local_mysql_dump_path, 'a') do |file|
+            file.write "UPDATE #{options_table} SET option_value=\"#{config.local.vhost}\" WHERE option_name=\"siteurl\" OR option_name=\"home\";\n"
+          end
+        end
+        host.run "mysql", "--user=#{config.local.database.username}", "--password=#{config.local.database.password}", "--host=#{config.local.database.host}", "--database=#{config.local.database.name}", :stdin => local_mysql_dump_path
+        host.run "rm", local_mysql_dump_path
+      end
+
+      remotely do |host|
+        host.run "rm", remote_mysql_dump_path
+      end
+
+    end
+
+    def config
+      if @config.blank?
+        config_path = @options[:config] || "Movefile"
+        unless File.exists? config_path
+          raise Thor::Error, "Could not find a valid Movefile"
+        end
+        @config = Hashie::Mash.new(YAML::load(File.open(config_path)))
+      end
+      @config
+    end
+
+    def table_prefix
+      config.table_prefix || "wp_"
+    end
+
+    def options_table
+      table_prefix + "options"
+    end
+
+    def local_wpcontent_path(*args)
+      File.join(config.local.wordpress_path, "wp-content", *args)
+    end
+
+    def remote_wpcontent_path(*args)
+      File.join(config.remote.wordpress_path, "wp-content", *args)
+    end
+
+    def locally
+      host = LocalHost.new(config.local.merge(:logger => @logger))
+      yield host
+      host.close
+    end
+
+    def remotely
+      host = RemoteHost.new(config.remote.merge(:logger => @logger))
+      yield host
+      host.close
+    end
+
+    def informative_errors
+      yield
+    rescue Timeout::Error
+      logger.error "Connection timed out!"
+      puts "Timed out"
+    rescue Errno::EHOSTUNREACH
+      logger.error "Host unreachable!"
+    rescue Errno::ECONNREFUSED
+      logger.error "Connection refused!"
+    rescue Net::SSH::AuthenticationFailed
+      logger.error "SSH authentification failure, please double check the SSH credentials on your Movefile!"
+    end
+
+  end
+end

data/lib/wordmove/hosts/remote_host.rb

@@ -0,0 +1,137 @@
+require 'net/ssh'
+require 'net/scp'
+require 'net/ssh/gateway'
+
+module Wordmove
+  class RemoteHost < LocalHost
+
+    alias :locally_run :run
+
+    attr_reader :session
+
+    def initialize(options = {})
+      super
+    end
+
+    def session
+      if options.ssh.nil?
+        raise Thor::Error, "No SSH credentials provided on Movefile!"
+      end
+
+      ssh_extras = {}
+      [ :port, :password ].each do |p|
+        ssh_extras.merge!( { p => options.ssh[p] } ) if options.ssh[p]
+      end
+
+      if options.ssh.gateway.nil?
+        logger.verbose "Connecting to #{options.ssh.host}..." unless @session.present?
+        @session ||= Net::SSH.start(options.ssh.host, options.ssh.username, ssh_extras)
+      else
+        logger.verbose "Connecting to #{options.ssh.host} through the gateway..." unless @session.present?
+        @session ||= gateway.ssh(options.ssh.host, options.ssh.username, ssh_extras)
+      end
+
+      @session
+    end
+
+    def gateway
+      if options.ssh.gateway.nil?
+        raise Thor::Error, "No SSH credentials provided on Movefile!"
+      end
+
+      ssh_extras = {}
+      [ :port, :password ].each do |p|
+        ssh_extras.merge!( { p => options.ssh.gateway[p] } ) if options.ssh.gateway[p]
+      end
+
+      logger.verbose "Connecting to #{options.ssh.gateway.host}..." unless @gateway.present?
+      @gateway ||= Net::SSH::Gateway.new(options.ssh.gateway.host, options.ssh.gateway.username, ssh_extras )
+
+      @gateway
+    end
+
+    def close
+      session.close
+      if options.ssh.gateway.present?
+        gateway.close(session.transport.port)
+      end
+    end
+
+    def upload_file(source_file, destination_file)
+      logger.verbose "Copying remote #{source_file} to #{destination_file}..."
+      session.scp.download! source_file, destination_file
+    end
+
+    def download_file(source_file, destination_file)
+      logger.verbose "Copying local #{source_file} to #{destination_file}..."
+      session.scp.upload! source_file, destination_file
+    end
+
+    def download_dir(source_dir, destination_dir)
+      destination_dir = ":#{destination_dir}"
+      destination_dir = "#{options.ssh.username}@#{destination_dir}" if options.ssh.username
+      rsync "#{source_dir}/", destination_dir
+    end
+
+    def upload_dir(source_dir, destination_dir)
+      source_dir = ":#{source_dir}/"
+      rsync source_dir, destination_dir
+    end
+
+    def run(*args)
+      command = shell_command(*args)
+      logger.verbose "Executing remotely #{command}"
+      session.exec!(command)
+    end
+
+    private
+
+    def get_host_for_options(options)
+      if options.username
+        "#{options.username}@#{options.host}"
+      else
+        options.host
+      end
+    end
+
+    def rsync(source_dir, destination_dir)
+
+      exclude_file = Tempfile.new('exclude')
+      exclude_file.write(options.exclude.join("\n"))
+      exclude_file.close
+
+      arguments = [ "-azLKO" ]
+
+      if options.ssh
+        remote_shell_arguments = []
+
+        if options.ssh.gateway
+          remote_shell_arguments.push("ssh", get_host_for_options(options.ssh.gateway))
+
+          if options.ssh.gateway.port
+            remote_shell_arguments.push("-p", options.ssh.gateway.port)
+          end
+        end
+
+        remote_shell_arguments.push("ssh")
+
+        if options.ssh.port
+          remote_shell_arguments.push("-p", options.ssh.port)
+        end
+
+        if options.ssh.password
+          remote_shell_arguments.unshift("sshpass", "-p", options.ssh.password)
+        end
+
+        remote_shell_arguments.push(get_host_for_options(options.ssh))
+
+        arguments.push("-e", remote_shell_arguments.join(" "))
+      end
+
+      arguments.push("--exclude-from=#{exclude_file.path}", "--delete", source_dir, destination_dir)
+      locally_run "rsync", *arguments
+
+      exclude_file.unlink
+    end
+  end
+end

data/lib/wordmove/version.rb

@@ -1,3 +1,3 @@
 module Wordmove
-  VERSION = "0.1.0.rc"
+  VERSION = "1.0.0"
 end

data/spec/spec_helper.rb

@@ -2,11 +2,7 @@ require 'rubygems'
 require 'bundler/setup'
 
 require 'wordmove' # and any other gems you need
-require 'hashie'
-require 'wordmove/hosts/local_host'
-require 'wordmove/hosts/remote_host'
 require 'wordmove/logger'
-require 'tempfile'
 require 'active_support/core_ext'
 require 'thor'
 

data/spec/sql_mover_spec.rb

@@ -0,0 +1,107 @@
+require 'wordmove/sql_mover'
+require 'tempfile'
+
+describe Wordmove::SqlMover do
+
+  let(:sql_path) { stub }
+  let(:source_config) { stub }
+  let(:dest_config) { stub }
+  let(:sql_mover) {
+    Wordmove::SqlMover.new(
+      sql_path,
+      source_config,
+      dest_config
+    )
+  }
+
+  context ".initialize" do
+    it "should assign variables correctly on initialization" do
+      sql_mover.sql_path.should == sql_path
+      sql_mover.source_config.should == source_config
+      sql_mover.dest_config.should == dest_config
+    end
+  end
+
+  context ".sql_content" do
+    let(:sql) do
+      Tempfile.new('sql').tap { |d| d.write('DUMP'); d.close }
+    end
+    let(:sql_path) { sql.path }
+
+    it "should read the sql file content" do
+      sql_mover.sql_content.should == 'DUMP'
+    end
+  end
+
+  context ".move!" do
+    it "should replace host, path and write to sql" do
+      sql_mover.should_receive(:replace_vhost!).and_return(true)
+      sql_mover.should_receive(:replace_wordpress_path!).and_return(true)
+      sql_mover.should_receive(:write_sql!).and_return(true)
+      sql_mover.move!
+    end
+  end
+
+  context ".replace_vhost!" do
+    it "should replace source vhost with dest vhost" do
+      sql_mover.should_receive(:replace_field!).with(:vhost).and_return(true)
+      sql_mover.replace_vhost!
+    end
+  end
+
+  context ".replace_wordpress_path!" do
+    it "should replace source path with dest path" do
+      sql_mover.should_receive(:replace_field!).with(:wordpress_path).and_return(true)
+      sql_mover.replace_wordpress_path!
+    end
+  end
+
+  context ".replace_field!" do
+    let(:source_config) { stub("config") }
+    let(:dest_config) { stub("config") }
+
+    it "should replace source vhost with dest vhost" do
+      source_config.stub(:[]).with(:field).and_return("DUMP")
+      dest_config.stub(:[]).with(:field).and_return("FUNK")
+
+      sql_mover.should_receive(:serialized_replace!).ordered.with("DUMP", "FUNK").and_return(true)
+      sql_mover.should_receive(:simple_replace!).ordered.with("DUMP", "FUNK").and_return(true)
+      sql_mover.replace_field!(:field)
+    end
+  end
+
+  context ".serialized_replace!" do
+    let(:content) { 's:4:"spam";s:20:"http://dump.com/spam";s:6:"foobar";s:22:"http://dump.com/foobar";s:8:"sausages"' }
+    let(:sql) { Tempfile.new('sql').tap do |d| d.write(content); d.close end }
+    let(:sql_path) { sql.path }
+
+    it "should replace source vhost with dest vhost" do
+      sql_mover.serialized_replace!('http://dump.com', 'http://shrubbery.com')
+      sql_mover.sql_content.should == 's:4:"spam";s:25:"http://shrubbery.com/spam";s:6:"foobar";s:27:"http://shrubbery.com/foobar";s:8:"sausages"'
+    end
+  end
+
+  context ".simple_replace!" do
+    let(:content) { "THE DUMP!" }
+    let(:sql) { Tempfile.new('sql').tap do |d| d.write(content); d.close end }
+    let(:sql_path) { sql.path }
+
+    it "should replace source vhost with dest vhost" do
+      sql_mover.simple_replace!("DUMP", "FUNK")
+      sql_mover.sql_content.should == "THE FUNK!"
+    end
+  end
+
+  context ".write_sql!" do
+    let(:content) { "THE DUMP!" }
+    let(:sql) { Tempfile.new('sql').tap do |d| d.write(content); d.close end }
+    let(:sql_path) { sql.path }
+    let(:the_funk) { "THE FUNK THE FUNK THE FUNK" }
+
+    it "should write content to file" do
+      sql_mover.sql_content = the_funk
+      sql_mover.write_sql!
+      File.open(sql_path).read == the_funk
+    end
+  end
+end

data/wordmove.gemspec

@@ -25,4 +25,6 @@ Gem::Specification.new do |gem|
   gem.add_development_dependency "rspec"
   gem.add_development_dependency "cucumber"
   gem.add_development_dependency "aruba"
+
+  gem.post_install_message = "Beware! We have changed the wordmove flags' behaviour: they used to tell wordmove what to _skip_, now they tell what to _include_. Read `wordmove help` for more info."
 end

metadata

@@ -1,15 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: wordmove
 version: !ruby/object:Gem::Version
-  version: 0.1.0.rc
-  prerelease: 6
+  version: 1.0.0
+  prerelease: 
 platform: ruby
 authors:
 - Stefano Verna
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-20 00:00:00.000000000 Z
+date: 2012-12-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colored
@@ -164,6 +164,8 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .rspec
+- .rvmrc
 - Gemfile
 - Gemfile.lock
 - README.mdown
@@ -178,23 +180,27 @@ files:
 - lib/wordmove/assets/dump.php.erb
 - lib/wordmove/assets/import.php.erb
 - lib/wordmove/cli.rb
+- lib/wordmove/deployer.rb
 - lib/wordmove/deployer/base.rb
 - lib/wordmove/deployer/ftp.rb
 - lib/wordmove/deployer/ssh.rb
 - lib/wordmove/generators/Movefile
 - lib/wordmove/generators/movefile.rb
+- lib/wordmove/hosts/remote_host.rb
 - lib/wordmove/logger.rb
 - lib/wordmove/sql_mover.rb
 - lib/wordmove/version.rb
 - pkg/wordmove-0.0.1.gem
 - pkg/wordmove-0.0.2.gem
 - spec/fixtures/Movefile
-- spec/remote_host_spec.rb
 - spec/spec_helper.rb
+- spec/sql_mover_spec.rb
 - wordmove.gemspec
 homepage: ''
 licenses: []
-post_install_message: 
+post_install_message: ! 'Beware! We have changed the wordmove flags'' behaviour: they
+  used to tell wordmove what to _skip_, now they tell what to _include_. Read `wordmove
+  help` for more info.'
 rdoc_options: []
 require_paths:
 - lib
@@ -207,9 +213,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
-  - - ! '>'
+  - - ! '>='
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
@@ -223,5 +229,5 @@ test_files:
 - features/step_definitions/aruba_ext_steps.rb
 - features/support/setup.rb
 - spec/fixtures/Movefile
-- spec/remote_host_spec.rb
 - spec/spec_helper.rb
+- spec/sql_mover_spec.rb

data/spec/remote_host_spec.rb

@@ -1,62 +0,0 @@
-require 'spec_helper'
-
-describe Wordmove::RemoteHost do
-
-  let!(:config) {
-    Hashie::Mash.new(YAML::load(File.open(File.join( File.dirname(__FILE__), "fixtures", "Movefile"))))
-  }
-
-  let(:host) {
-    logger = Wordmove::Logger.new
-    logger.level = Logger::INFO
-    config.remote[:logger] = @logger
-    host = Wordmove::RemoteHost.new( config.remote )
-    host.stub!(:locally_run).and_return( 1 )
-    host
-  }
-
-  context ".upload_dir" do
-
-    it "should use config ports, username and password properly" do
-      host.should_receive(:locally_run).with("rsync", "-azLK", "-e",  "sshpass -p password ssh -p 30000", anything(), "--delete", "username@host:foobar/", "barfoo" )
-      host.upload_dir( "foobar", "barfoo" )
-    end
-
-    it "should skip port if missing" do
-      config.remote!.ssh!.port = nil
-      host.should_receive(:locally_run).with("rsync", "-azLK", "-e",  "sshpass -p password ssh", anything(), "--delete", "username@host:foobar/", "barfoo" )
-      host.upload_dir( "foobar", "barfoo" )
-    end
-
-    it "should skip username if missing" do
-      config.remote!.ssh!.username = nil
-      host.should_receive(:locally_run).with("rsync", "-azLK", "-e",  "sshpass -p password ssh -p 30000", anything(), "--delete", "host:foobar/", "barfoo" )
-      host.upload_dir( "foobar", "barfoo" )
-    end
-
-    context "when password is missing" do
-
-      before { config.remote!.ssh!.password = nil }
-
-      it "should use config ports and username properly" do
-        host.should_receive(:locally_run).with("rsync", "-azLK", "-e",  "ssh -p 30000", anything(), "--delete", "username@host:foobar/", "barfoo" )
-        host.upload_dir( "foobar", "barfoo" )
-      end
-
-      it "should skip port if missing" do
-        config.remote!.ssh!.port = nil
-        host.should_receive(:locally_run).with("rsync", "-azLK", anything(), "--delete", "username@host:foobar/", "barfoo" )
-        host.upload_dir( "foobar", "barfoo" )
-      end
-
-      it "should skip username if missing" do
-        config.remote!.ssh!.username = nil
-        host.should_receive(:locally_run).with("rsync", "-azLK", "-e",  "ssh -p 30000", anything(), "--delete", "host:foobar/", "barfoo" )
-        host.upload_dir( "foobar", "barfoo" )
-      end
-
-    end
-
-  end
-
-end