loady 0.7.1 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 73912330c3f4b5362b36e18a110532db84d4635e
-  data.tar.gz: 0f8a2bd49e4eb00fde4aff8ffe79fad9d24553e9
+  metadata.gz: 5431b74708121dcbdbc9ec5743fbaebb92e8940e
+  data.tar.gz: bd108b5bf37cf749ccb4195fce1f6549c07728f7
 SHA512:
-  metadata.gz: 8e30064023249f1baaa1945a7f2b2a522d238f7c5742f2c8c591fcd50e28b8c96325dc6eeca86eca276485a46a73bf77c80a1d7fc9a30c001a57ffda3a4dd394
-  data.tar.gz: 0ee3605df2936c3a96deff8b6e1f8edabe9f6776d4c7e180f70c25382ddca6948813617c1d43cb65e4a510653c211465d78c66c13c7c1fca0988250639afc9c4
+  metadata.gz: e4028a6a6bb1e71963ce84726e1bf8b6c907a868f2e95ab9a056e01cb6e549b23fae0c8e3a50aa6a1696b1091a725e94a1bad926a820beef176666d7c5e125c2
+  data.tar.gz: 6990fc622994fd3a70845e5129d23bf8d5a59fae6215c01163575cb8214ad3a96559609281ad95b12f0e2789b7f4ad216d28d4ba310623c92b5f69047bf728fb

data/Gemfile

@@ -1,4 +1,7 @@
 source "https://rubygems.org"
 
-# see loady.gemspec
+group :test do
+  gem 'coveralls', require: false
+end
+
 gemspec

data/README.md

@@ -1,6 +1,7 @@
 [![Gem Version](https://badge.fury.io/rb/loady.png)](http://badge.fury.io/rb/loady)
 [![Build Status](https://api.travis-ci.org/teeparham/loady.png)](https://travis-ci.org/teeparham/loady)
 [![Code Climate](https://codeclimate.com/github/teeparham/loady.png)](https://codeclimate.com/github/teeparham/loady)
+[![Coverage Status](https://coveralls.io/repos/teeparham/loady/badge.png)](https://coveralls.io/r/teeparham/loady)
 
 # Loady - A file loader with simple logging
 

data/lib/loady/attribute_array.rb

@@ -15,15 +15,10 @@ module Loady
       attr_hash = {}
 
       names.each_with_index do |name, i|
-        if i < self.size
-          if options[:strip] && self[i]
-            attr_hash[name] = self[i].strip
-          else
-            attr_hash[name] = self[i]
+        attr_hash[name] = 
+          if i < self.size && self[i]
+            options[:strip] ? self[i].strip : self[i]
           end
-        else
-          attr_hash[name] = nil
-        end
       end
 
       attr_hash

data/lib/loady/csv_loader.rb

@@ -9,12 +9,6 @@ module Loady
       @line_number = 0
     end
 
-    class << self
-      def read(*args, &block)
-        self.new.read(*args, &block)
-      end
-    end
-
     # valid options:
     #   skip_first_row: true                       -- default = false
     #   logger: Logger.new('/somewhere/file.log')  -- default = Logger.new(STDOUT)
@@ -24,36 +18,35 @@ module Loady
       @logger = options.delete(:logger) || default_logger
 
       f = File.new(filename)
-      f.readline if options.delete(:skip_first_row)
+      f.autoclose = true
+      f.gets if options.delete(:skip_first_row)
 
-      begin
-        while (line = f.readline)
-          readline(line, options, &block)
-        end
-      rescue EOFError
-        f.close
-      rescue Exception => message
-        @warning += 1
-        @line_number += 1
-        @logger.error "#{message} - at line #@line_number\n#{line}"
-      end
+      f.each do |line|
+        readline line, options, &block
+      end        
 
       @logger.info "Finished. Loaded #@success rows. #@warning unprocessed rows."
     end
 
+    class << self
+      def read(*args, &block)
+        self.new.read(*args, &block)
+      end
+    end
+
   private
 
     def readline(line, options)
       @line_number += 1
       row = AttributeArray.new(CSV.parse(line, options)[0])
 
-      unless row.empty?
+      if row.any?
         yield row
         @success += 1
       end
-    rescue Exception => message
+    rescue Exception => ex
       @warning += 1
-      @logger.warn "#{message.to_s.gsub("line 1", "line #@line_number")}\n#{line}"
+      @logger.warn "#{ex.to_s.gsub("line 1", "line #@line_number")}\n#{line}"
     end
 
     def default_logger

data/lib/loady/version.rb

@@ -1,3 +1,3 @@
 module Loady
-  VERSION = "0.7.1"
+  VERSION = "0.7.2"
 end

data/loady.gemspec

@@ -1,6 +1,5 @@
 # -*- encoding: utf-8 -*-
-$:.push File.expand_path("../lib", __FILE__)
-require "loady/version"
+require "./lib/loady/version"
 
 Gem::Specification.new do |s|
   s.name        = "loady"

data/test/test_helper.rb

@@ -1,3 +1,6 @@
+require 'coveralls'
+Coveralls.wear!
+
 require 'test/unit'
 require 'shoulda-context'
 require 'mocha/setup'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: loady
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.2
 platform: ruby
 authors:
 - Tee Parham
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-08-06 00:00:00.000000000 Z
+date: 2013-10-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mocha
@@ -112,7 +112,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.5
+rubygems_version: 2.1.5
 signing_key: 
 specification_version: 4
 summary: CSV file loader with simple logging
@@ -124,4 +124,3 @@ test_files:
 - test/test_csv_loader.rb
 - test/test_helper.rb
 - test/test_memory_logger.rb
-has_rdoc: