jgdavey-movable_erb 0.1.3 → 0.2.0

data/spec/csv_spec.rb

@@ -1,73 +1,322 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
-require File.join(File.dirname(__FILE__), %w[spec_helper])
+TEMPLATE_FIXTURE = File.expand_path(File.dirname(__FILE__) + '/fixtures/template.erb')
+CSV_FIXTURE = File.expand_path(File.dirname(__FILE__) + '/fixtures/example.csv')
 
-describe MovableErb::Csv do
-  describe "intialization" do
+describe MovableErb do
+  it "should have a CSV instance" do
+    m = MovableErb.new
+    m.csv = MovableErb::CSV.setup do |csv|
+      csv.filename = CSV_FIXTURE
+    end.should be_a(MovableErb::CSV)
+  end
+
+  it "should have an Erb instance" do
+    m = MovableErb.new
+    m.erb = MovableErb::Erb.setup do |erb|
+      true
+    end.should be_a(MovableErb::Erb)
+  end
 
+  context "#convert" do
     before(:each) do
-      @csv = MovableErb::Csv.new({:file => 'example.csv'})
+      @m = MovableErb.new
+    end
+    it "should raise an error if it's missing a CSV and/or Erb instance" do
+      lambda { @m.convert }.should raise_error 
     end
 
-    it "should accept arguments" do
-      @csv.should_not be_nil
+    it "should raise an error if it's CSV instance can't parse!" do
+      @m.erb = MovableErb::Erb.setup { |erb| erb.template = TEMPLATE_FIXTURE }
+      @m.csv = "Random string"
+      lambda { @m.convert }.should raise_error(NoMethodError, "undefined method `parse!' for \"Random string\":String")
     end
 
-    it "should have @file attribute match passed in value" do
-      @csv.file.should eql('./example.csv')
+    it "should raise an error if it's missing an Erb instance" do
+      String.any_instance.stubs(:data=).returns('')
+      @m.csv = MovableErb::CSV.setup { |csv| csv.filename = CSV_FIXTURE }
+      @m.erb = "Random string"
+      lambda { @m.convert }.should raise_error(NoMethodError, "undefined method `build!' for \"Random string\":String")
     end
-    
-    it "should allow files in sub-folders" do
-      @csv = MovableErb::Csv.new({:file => 'subfolder/example.csv'})
-      @csv.file.should eql('./subfolder/example.csv')
+
+    it "should convert the CSV to a string with the Erb template" do
+      @m.csv = MovableErb::CSV.setup { |csv| csv.filename = CSV_FIXTURE }
+      @m.erb = MovableErb::Erb.setup { |erb| erb.template = TEMPLATE_FIXTURE }
+      @m.separator = "\n"
+      @m.convert.should == <<-ERB.gsub(/^ +/, "")
+      Name: John
+      Email: john@example.com
+
+      Name: Abigail
+      Email: abby@example.com
+
+      Name: Bernard
+
+      Name: Casius
+      ERB
     end
-    
-    it "should recognize the dot as current folder" do
-      @csv = MovableErb::Csv.new({:file => './example.csv'})
-      @csv.file.should eql('./example.csv')
+  end
+
+  context "initilization" do
+    it "should take a hash of options" do
+      m = MovableErb.new({:hash => 'of options'})
+      m.should be_a(MovableErb)
     end
 
+    it "should set the template file" do
+      m = MovableErb.new({ :template => TEMPLATE_FIXTURE })
+      m.erb.template.should == File.read(TEMPLATE_FIXTURE)
+    end
+
+    it "should set the csv file" do
+      m = MovableErb.new({ :csv => CSV_FIXTURE })
+      m.csv.filename.should == CSV_FIXTURE 
+    end
+    
+    it "should set a separator if given" do
+      m = MovableErb.new({ :separator  => ', ' })
+      m.separator.should == ', '
+    end
   end
   
-  describe "opening the file" do
+  context "full-on run-through" do
+    it "should work!" do
+      movable_erb = MovableErb.new({
+        :csv => File.expand_path(File.dirname(__FILE__) + '/fixtures/advanced.csv'),
+        :separator => ''
+      })
+      movable_erb.convert.should == <<-ERB.gsub(/^ +/, "")
+        TITLE: Hambone
+        CATEGORY: Silly
+        CATEGORY: Blah
+        -----
+        BODY:
+        
+        This is the content for hambone
+        
+        --------
+        TITLE: WillyNilly
+        -----
+        BODY:
+        
+        More body
+        
+        --------
+      ERB
+    end
+  end
+end
+
+describe MovableErb::CSV do
+  it "should be an instance of itself" do
+    MovableErb::CSV.new.should be_a(MovableErb::CSV)
+  end
+
+  describe "setup" do
     before(:each) do
-      @file_to_open = File.join(File.dirname(__FILE__), 'fixtures', 'example.csv')
-      @csv = MovableErb::Csv.new({:file => @file_to_open})
+      @csv = MovableErb::CSV.new
     end
 
-    it "should find the file" do
-      File.open(@file_to_open).should_not be_nil
+    it "should setup with a block" do
+      @csv.setup {|csv| false }.should == @csv
     end
 
-    it "should populate rows" do
-      @csv.rows.should_not be_nil
+    it "should accept a file argument" do
+      FasterCSV.stubs(:read).returns([])
+      @csv.setup do |csv|
+        csv.filename = "test.csv"
+      end.should == @csv
+    end
+
+    it "should call parse if filename given in block" do
+      @csv.expects(:parse!).returns('')
+      @csv.setup { |csv| csv.filename = "test.csv" }
+    end
+
+    it "should not call parse if filename not given in block" do
+      @csv.expects(:parse!).never
+      @csv.setup { |csv| false }
+    end
+
+    context "shortcut setup class method" do
+      before(:each) do
+        MovableErb::CSV.any_instance.stubs(:filename).returns("fake")
+        FasterCSV.stubs(:read).returns([])
+      end
+
+      it "should create a new instance" do
+        MovableErb::CSV.setup do |csv|
+          csv.should be_a(MovableErb::CSV)
+        end
+      end
+
+      it "should return the new instance" do
+        MovableErb::CSV.setup { |csv| @instance = csv }.should == @instance
+      end
+
+      it "should return the new instance (2)" do
+        MovableErb::CSV.setup do |csv|
+          @instance = csv
+          'fakey fake string'
+        end.should == @instance
+      end
+
+      it "should raise an error if no block given" do
+        lambda { MovableErb::CSV.setup }.should raise_error('no block given')
+      end
+
+      it "should call parse!" do
+        MovableErb::CSV.any_instance.expects(:parse!)
+        MovableErb::CSV.setup { |csv| true }
+      end
     end
   end
-  
-  describe "rows" do
+
+  describe "parsing" do
     before(:each) do
-      @file_to_open = File.join(File.dirname(__FILE__), 'fixtures', 'example.csv')
-      @csv = MovableErb::Csv.new({:file => @file_to_open})
+      FasterCSV.stubs(:read).returns([["row1"], ["row2"]])
+      @csv = MovableErb::CSV.new
+      @csv.filename = "test.csv"
+    end
+
+    it "#parse! should return self" do
+      @csv.parse!.should == @csv
+    end
+
+    it "#parse! should modify attribute hashes (via to_hashes)" do
+      @csv.expects(:to_hashes)
+      @csv.parse!
     end
 
-    it "should be an array of arrays" do
-      @csv.rows.should be_instance_of(Array)
-      @csv.rows.each do |row|
-        row.should be_instance_of(Array)
+    context "#to_hashes" do
+      it "should read from file" do
+        FasterCSV.expects(:read).with("test.csv").returns([[]])
+        @csv.to_hashes
+      end
+
+      it "should return an array of hashes (1 rows, 1 column)" do
+        @csv.to_hashes.should == [{'row1' => ['row2']}]
+      end
+
+      it "should downcase the header row" do
+        FasterCSV.stubs(:read).returns([["Name"],["Billy Bob"]])
+        @csv.to_hashes.should == [{'name' => ['Billy Bob']}]
+      end
+      
+      it "should convert the header row to snake_case" do
+        FasterCSV.stubs(:read).returns([["Extended Body"],["Billy Bob"]])
+        @csv.to_hashes.should == [{'extended_body' => ['Billy Bob']}] 
+      end
+
+      it "should return an array of hashes (3 rows, 3 columns)" do
+        FasterCSV.stubs(:read).returns([["Name", "Phone", "Email"], 
+        ["John", "773-123-1234", "john@example.com"], 
+        ["Abigail", nil, "abby@example.com"], 
+        ["Casius", nil, nil]])
+        @csv.to_hashes.should == [
+          {'name' => ['John'],    'phone' => ['773-123-1234'], 'email' => ['john@example.com'] },
+          {'name' => ['Abigail'], 'email' => ['abby@example.com'] },
+          {'name' => ['Casius'],}
+        ]
+      end
+
+      it "should collect values with the same key" do
+        FasterCSV.stubs(:read).returns([["Name", "Name", "Email"], ["John", "James", "john@example.com"]]) 
+        @csv.to_hashes.should == [{'name' => ['John', 'James'],'email' => ['john@example.com']}]
+      end
+
+      it "should collect values with the same key (more than 2)" do
+        FasterCSV.stubs(:read).returns([["Name", "Name", "Name"], ["John", "James", "Jill"]]) 
+        @csv.to_hashes.should == [{'name' => ['John', 'James', 'Jill']}]
       end
     end
+  end
+end
 
-    it "should have a header row" do
-      @csv.header.should eql(['Name','Phone','Email'])
+describe MovableErb::Erb do
+  context "initializing" do
+    it "should initialize with a template file" do
+      MovableErb::Erb.new.should be_a(MovableErb::Erb)
     end
+  end
 
-    it "should exclude header from data" do
-      @csv.data.first.should_not eql(['Name','Phone','Email'])
+  context "loading a template" do
+    it "template setter should load a file into @template" do
+      File.expects(:read).with("filename.erb").returns("This is a template")
+      @erb = MovableErb::Erb.new
+      @erb.template = 'filename.erb'
+      @erb.template.should == 'This is a template'
     end
-    
-    it "should should have data" do
-      @csv.data.first.should eql(['John','773-123-1234','john@example.com'])
+  end
+
+  context "data hash" do
+    it "should accept a data hash" do
+      @erb = MovableErb::Erb.new
+      @erb.data = {'these' => 'params'}
+      @erb.data.should == {'these' => 'params'}
     end
   end
-end
 
-# EOF
+  context "setting up with block" do
+    before(:each) do
+      @erb = MovableErb::Erb.new
+    end
+
+    it "should accept a block that yields itself" do
+      @erb.setup do |erb|
+        erb.should == @erb
+      end
+    end
+  end
+
+  context "shortcut setup class method" do
+    it "should create a new instance" do
+      MovableErb::Erb.setup do |erb|
+        erb.should be_a(MovableErb::Erb)
+      end
+    end
+
+    it "should return the new instance" do
+      MovableErb::Erb.setup { |erb| @instance = erb }.should == @instance
+    end
+
+    it "should return the new instance (2)" do
+      MovableErb::Erb.setup do |erb|
+        @instance = erb
+        'fakey fake string'
+      end.should == @instance
+    end
+
+    it "should raise an error if no block given" do
+      lambda { MovableErb::Erb.setup }.should raise_error('no block given')
+    end
+  end
+
+  context "#build!" do
+    before(:each) do
+      @erb = MovableErb::Erb.new
+    end
+
+    it "should create a new ERB instance with @template" do
+      @erb.stubs(:template).returns("string")
+      ERB.expects(:new)
+      @erb.build!
+    end
+
+    it "should set @parsed_string" do
+      @erb.stubs(:template).returns("string")
+      @erb.build!
+      @erb.parsed_string.should_not be_nil
+    end
+
+    it "should parse a template with data given" do
+      @erb.template = TEMPLATE_FIXTURE
+      @erb.data = {'name' => 'Johnny', 'email' => 'john@example.com'}
+      @erb.build!
+      @erb.parsed_string.should == <<-ERB.gsub(/^\s+/,'')
+      Name: Johnny
+      Email: john@example.com
+      ERB
+    end
+  end
+end

data/spec/fixtures/advanced.csv

@@ -0,0 +1,3 @@
+Title,Body,Category,Category
+Hambone,This is the content for hambone,Silly,Blah
+WillyNilly,More body,,

data/spec/fixtures/template.erb

@@ -0,0 +1,4 @@
+Name: <%= data['name'] %>
+<% if data['email'] %>
+Email: <%= data['email'] %>
+<% end %>

data/spec/spec.opts

@@ -0,0 +1,2 @@
+--color
+--diff u

data/spec/spec_helper.rb

@@ -1,18 +1,7 @@
 require 'rubygems'
-require 'fastercsv'
 
-require File.expand_path(
-    File.join(File.dirname(__FILE__), %w[.. lib movable_erb]))
+require File.expand_path(File.join(File.dirname(__FILE__), %w[.. lib movable_erb]))
 
 Spec::Runner.configure do |config|
-  # == Mock Framework
-  #
-  # RSpec uses it's own mocking framework by default. If you prefer to
-  # use mocha, flexmock or RR, uncomment the appropriate line:
-  #
-  # config.mock_with :mocha
-  # config.mock_with :flexmock
-  # config.mock_with :rr
+  config.mock_with :mocha
 end
-
-# EOF

data/tasks/rspec.rake

@@ -0,0 +1,11 @@
+require 'spec/rake/spectask'
+
+namespace :spec do
+  desc "Run all examples with RCov"
+  Spec::Rake::SpecTask.new('rcov') do |t|
+    t.spec_files = FileList['spec']
+    t.spec_opts << ["--color"]
+    t.rcov = true
+    t.rcov_opts = ['--exclude', 'spec/,tasks/,coverage/,features/,/System/,/Library/']
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: jgdavey-movable_erb
 version: !ruby/object:Gem::Version 
-  version: 0.1.3
+  version: 0.2.0
 platform: ruby
 authors: 
 - Joshua Davey
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-04-18 00:00:00 -07:00
+date: 2009-09-26 00:00:00 -07:00
 default_executable: movable_erb
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -23,47 +23,72 @@ dependencies:
         version: "0"
     version: 
 - !ruby/object:Gem::Dependency 
-  name: bones
+  name: trollop
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "0"
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: rspec
   type: :development
   version_requirement: 
   version_requirements: !ruby/object:Gem::Requirement 
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 2.5.0
+        version: "0"
     version: 
-description: "Usage: movable_erb [options]"
+description: A General-purpose CSV to ERB template formatter
 email: josh@joshuadavey.com
 executables: 
 - movable_erb
 extensions: []
 
 extra_rdoc_files: 
-- History.txt
+- CHANGELOG
+- LICENSE
+- README.md
 - bin/movable_erb
-- lib/movable_erb/templates/default.erb
+- lib/movable_erb.rb
+- lib/templates/mtimport.erb
+- tasks/rspec.rake
 files: 
-- History.txt
-- README.textile
+- CHANGELOG
+- LICENSE
+- README.md
 - Rakefile
 - bin/movable_erb
+- cucumber.yml
+- features/csv.feature
+- features/step_definitions/csv_steps.rb
+- features/step_definitions/tmp.csv
+- features/support/env.rb
 - lib/movable_erb.rb
-- lib/movable_erb/csv.rb
-- lib/movable_erb/mtimport.rb
-- lib/movable_erb/templates/default.erb
-- movable_erb.gemspec
+- lib/templates/mtimport.erb
 - spec/csv_spec.rb
+- spec/fixtures/advanced.csv
 - spec/fixtures/example.csv
-- spec/movable_erb_spec.rb
-- spec/mtimport_spec.rb
+- spec/fixtures/template.erb
+- spec/spec.opts
 - spec/spec_helper.rb
-- test/test_movable_erb.rb
-has_rdoc: true
-homepage: http://github.com/jgdavey/movable_erb/
+- tasks/rspec.rake
+- Manifest
+- movable_erb.gemspec
+has_rdoc: false
+homepage: http://github.com/jgdavey/movable_erb
+licenses: 
 post_install_message: 
 rdoc_options: 
+- --line-numbers
+- --inline-source
+- --title
+- Movable_erb
 - --main
-- README.textile
+- README.md
 require_paths: 
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement 
@@ -76,14 +101,14 @@ required_rubygems_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      version: "0"
+      version: "1.2"
   version: 
 requirements: []
 
 rubyforge_project: movable_erb
-rubygems_version: 1.2.0
+rubygems_version: 1.3.5
 signing_key: 
 specification_version: 3
-summary: A simple CSV to MTImport conversion utility
-test_files: 
-- test/test_movable_erb.rb
+summary: A General-purpose CSV to ERB template formatter
+test_files: []
+

data/History.txt

@@ -1,4 +0,0 @@
-== 0.1.0 / 2009-04-03
-
-* 1 major enhancement
-  * Birthday!

data/lib/movable_erb/csv.rb

@@ -1,30 +0,0 @@
-require 'rubygems'
-require 'fastercsv'
-
-module MovableErb
-  class Csv
-    attr_reader :file
-    def initialize(args)
-      if args[:file]
-        args[:file].gsub!(/^([^\.\/])/,'./\1')
-        @file = args[:file]
-      end
-    end
-    
-    def rows
-      FasterCSV.read(file)
-    end
-    
-    def header
-      rows.first
-    end
-    
-    def data
-      rows[1..rows.length]
-    end
-    
-    def body
-      data
-    end
-  end
-end

data/lib/movable_erb/mtimport.rb

@@ -1,65 +0,0 @@
-module MovableErb
-  class MTImport
-    require 'erb'
-    
-    COLUMNNAMES = ['title','body','extended', 'category', 'tags']
-    
-    attr_accessor :csv, :template, :columns
-    
-    def initialize(args = {})
-      if args[:csv]
-        @csv = Csv.new(args[:csv])
-        @columns = {:body => nil, :title => nil}
-      end
-        @template = args[:template] || File.join(File.dirname(__FILE__), 'templates', 'default.erb')
-    end
-    
-    def header_rows
-      if @csv
-        @csv.header
-      else
-        raise "No CSV file"
-      end
-    end
-    
-    def setup_column_nums
-      COLUMNNAMES.each do |colname|
-        @columns[colname.to_sym] = header_rows.to_enum(:each_with_index).collect do |x,i| 
-          i if x.downcase == colname
-        end.compact
-      end
-      # Useful defaults
-      @columns[:title] = [0] if @columns[:title].empty?
-      @columns[:body] = [1] if @columns[:body].empty?
-      @columns
-    end
-    
-    def column_nums_for(column)
-      @columns[column.to_sym]
-    end
-    
-    def content_for(column, row = 0)
-      content = column_nums_for(column).map do |i|
-        csv.body[row][i]
-      end
-    end
-
-    
-    def render_with_template(template = @template)
-      rendered = []
-      csv.body.each_with_index do |row, i|
-        title = content_for('title', i).join(" ")
-        body = content_for('body', i).join("\n")
-        extended = content_for('extended', i).join("\n")
-        category = content_for('category', i).join(" ")
-        tags = content_for('tags', i).join(", ")
-        erb = File.open(template, "rb").read
-        r = ERB.new(erb, 0, '<>') if erb
-        b = binding
-        rendered << r.result(b)
-      end
-      rendered.join("--------\n")
-    end
-    
-  end
-end

data/lib/movable_erb/templates/default.erb

@@ -1,21 +0,0 @@
-TITLE: <%= title %>
-<% if category && !category.empty? %>
-CATEGORY: <%= category %>
-<% end %>
-<% if tags && !tags.empty? %>
-TAGS: <%= tags %>
-<% end %>
------
-BODY:
-
-<%= body %>
-
-
-<% unless extended.nil? || extended.empty? %>
------
-EXTENDED:
-
-<%= extended %>
-
-
-<% end %>

data/spec/movable_erb_spec.rb

@@ -1,10 +0,0 @@
-
-require File.join(File.dirname(__FILE__), %w[spec_helper])
-
-describe MovableErb do
-  it "should have class Csv" do
-    MovableErb::Csv.should_not be_nil
-  end
-end
-
-# EOF