est 0.2.1 → 0.3

data/.rultor.yml

@@ -12,6 +12,8 @@ release:
     rake
     rm -rf *.gem
     sed -i "s/1\.0\.snapshot/${tag}/g" lib/est/version.rb
+    git add lib/est/version.rb
+    git commit -m "version set to ${tag}"
     gem build est.gemspec
     chmod 0600 ../rubygems.yml
     gem push *.gem --config-file ../rubygems.yml

data/assets/est.xsd

@@ -33,7 +33,7 @@
     <xs:complexType>
       <xs:all>
         <xs:element name="total" type="xs:int"/>
-        <xs:element name="ests">
+        <xs:element name="ests" minOccurs="0">
           <xs:complexType>
             <xs:sequence>
               <xs:element name="est" type="est" minOccurs="1" maxOccurs="unbounded" />

data/features/empty_dir.feature

@@ -0,0 +1,21 @@
+Feature: Command Line Processing
+  As an estimator I want to be able to
+  call Est as a command line tool
+
+  Scenario: Estimates empty directory
+    Given I have a "test.txt" file with content:
+    """
+    hello
+    """
+    When I run bin/est with "--dir=. --format=text"
+    Then Exit code is zero
+    And Stdout contains "Total: 0"
+
+  Scenario: Estimates absent directory
+    Given I have a "test.txt" file with content:
+    """
+    hello
+    """
+    When I run bin/est with "--dir=./absent-dir --format=text"
+    Then Exit code is zero
+    And Stdout contains "Total: 0"

data/lib/est.rb

@@ -76,12 +76,14 @@ module Est
           xml << "<?xml-stylesheet type='text/xsl' href='#{xsl}'?>"
           xml.estimate(attrs) do
             xml.total estimates.total
-            xml.ests do
-              estimates.iterate.each do |est|
-                xml.est do
-                  xml.date est.date
-                  xml.total est.total
-                  xml.author est.author
+            unless estimates.iterate.empty?
+              xml.ests do
+                estimates.iterate.each do |est|
+                  xml.est do
+                    xml.date est.date
+                    xml.total est.total
+                    xml.author est.author
+                  end
                 end
               end
             end

data/lib/est/estimates.rb

@@ -42,24 +42,33 @@ module Est
     # Get total estimate.
     def total
       estimates = iterate
-      estimates.reduce(0) do |a, e|
-        Est.log.info "#{e.date}/#{e.author}: #{e.total}"
-        a + e.total
-      end / estimates.size
+      if estimates.empty?
+        total = 0
+      else
+        total = estimates.reduce(0) do |a, e|
+          Est.log.info "#{e.date}/#{e.author}: #{e.total}"
+          a + e.total
+        end / estimates.size
+      end
+      total
     end
 
     # Iterate them all
     def iterate
       unless @iterate
-        @iterate = Dir.entries(@dir)
-          .reject { |f| f.index('.') == 0 }
-          .select { |f| f =~ /^.*\.est$/ }
-          .map { |f| File.join(@dir, f) }
-          .each { |f| Est.log.info "#{f} found" }
-          .map { |f| Estimate.new(f) }
-          .map { |f| Estimate::Const.new(f) }
+        if File.exist?(@dir) && File.directory?(@dir)
+          @iterate = Dir.entries(@dir)
+            .reject { |f| f.index('.') == 0 }
+            .select { |f| f =~ /^.*\.est$/ }
+            .map { |f| File.join(@dir, f) }
+            .each { |f| Est.log.info "#{f} found" }
+            .map { |f| Estimate.new(f) }
+            .map { |f| Estimate::Const.new(f) }
+        else
+          Est.log.info "#{@dir} is absent or is not a directory"
+          @iterate = []
+        end
       end
-      fail "no .est files found in #{@dir}" if @iterate.empty?
       @iterate
     end
 

data/lib/est/version.rb

@@ -26,5 +26,5 @@
 # Copyright:: Copyright (c) 2014 Yegor Bugayenko
 # License:: MIT
 module Est
-  VERSION = '0.2.1'
+  VERSION = '0.3'
 end

data/test/test_est.rb

@@ -75,6 +75,36 @@ class TestEst < Minitest::Test
     end
   end
 
+  def test_empty_dir
+    Dir.mktmpdir 'test' do |dir|
+      opts = opts(['-v', '-d', dir])
+      matches(
+        Nokogiri::XML(Est::Base.new(opts).xml),
+        [
+          '/estimate/@version',
+          '/estimate/@date',
+          '/estimate[total="0"]',
+          '/estimate[not(ests)]'
+        ]
+      )
+    end
+  end
+
+  def test_empty_dir
+    Dir.mktmpdir 'test' do |dir|
+      opts = opts(['-v', '-d', File.join(dir, 'absent')])
+      matches(
+        Nokogiri::XML(Est::Base.new(opts).xml),
+        [
+          '/estimate/@version',
+          '/estimate/@date',
+          '/estimate[total="0"]',
+          '/estimate[not(ests)]'
+        ]
+      )
+    end
+  end
+
   def opts(args)
     Slop.parse args do
       on 'v', 'verbose'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: est
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: '0.3'
   prerelease: 
 platform: ruby
 authors:
@@ -214,6 +214,7 @@ files:
 - est.gemspec
 - est/first.est
 - features/cli.feature
+- features/empty_dir.feature
 - features/gem_package.feature
 - features/step_definitions/steps.rb
 - features/support/env.rb
@@ -255,6 +256,7 @@ specification_version: 2
 summary: Estimates Automated
 test_files:
 - features/cli.feature
+- features/empty_dir.feature
 - features/gem_package.feature
 - features/step_definitions/steps.rb
 - features/support/env.rb