hippo 0.4.1 → 0.5.0

data/CHANGELOG

@@ -1,3 +1,10 @@
+0.5.0 - 2012/12/13
+  * Allow parsing of HIPAA_837::L2000B to properly handle the 'floating' L2300
+    loop.
+  * Make segment/loop inclusion conditional (when :parent_context_conditions
+                                             is used)
+  * Refactor TransactionSets::Base.grouped_components.
+  * Properly handle the removal of empty fields.
 0.4.1 - 2012/09/14
   * Fix major issue (stupid typo) with Date parsing.
 0.4.0 - 2012/09/13

data/Gemfile

@@ -3,6 +3,8 @@ source "http://rubygems.org"
 # Specify your gem's dependencies in hippo.gemspec
 gemspec
 
-group :development do
+group :development, :test do
   gem 'pry'
+  gem 'guard-minitest'
+  gem 'rb-fsevent', '~> 0.9.1'
 end

data/hippo.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |s|
   s.rubyforge_project = "hippo"
 
   s.add_development_dependency('minitest')
-  s.add_development_dependency('rake', '~>0.9.2')
+  s.add_development_dependency('rake', '~>10.0.2')
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")

data/lib/hippo/segments/base.rb

@@ -146,7 +146,7 @@ module Hippo::Segments
       end
 
       unless self.class.fixed_width
-        output = output.gsub(empty_field_regexp,'\1\2')
+        output = remove_empty_fields(output)
         output = output.gsub(repeating_composite_separator_regexp, @field_separator)
       end
 

data/lib/hippo/separator.rb

@@ -56,15 +56,26 @@ module Hippo
     end
 
     def empty_field_regexp
-      %r{(?<separators>
-          [
+      %r{([
             #{Regexp.escape(@composite_separator)}
             #{Regexp.escape(@field_separator)}
             #{Regexp.escape(@segment_separator)}
           ])
           \s+
-          (\g<separators>)
+          ([
+            #{Regexp.escape(@composite_separator)}
+            #{Regexp.escape(@field_separator)}
+            #{Regexp.escape(@segment_separator)}
+          ])
         }x
     end
+
+    def remove_empty_fields(input)
+      while input =~ empty_field_regexp
+        input = input.gsub(empty_field_regexp, '\1\2')
+      end
+
+      input
+    end
   end
 end

data/lib/hippo/transaction_sets/HIPAA_837/L2000B.rb

@@ -55,17 +55,6 @@ module Hippo::TransactionSets
                   'NM1.NM108' => ["PI", "XV"]
                 }
 
-      #Patient Hierarchical Level
-      loop    Hippo::TransactionSets::HIPAA_837::L2000C,
-                :name           => 'Patient Hierarchical Level',
-                :minimum        => 0,
-                :maximum        => 99999,
-                :position       => 10,
-                :identified_by => {
-                  'HL.HL03' => '23',
-                  'HL.HL04' => '0'
-                }
-
       #Claim Information
       loop    Hippo::TransactionSets::HIPAA_837::L2300,
                 :name           => 'Claim Information',
@@ -77,6 +66,20 @@ module Hippo::TransactionSets
                   'CLM.CLM07' => ["A", "B", "C"],
                   'CLM.CLM08' => ["N", "W", "Y"],
                   'CLM.CLM09' => ["I", "Y"]
+                },
+                :parent_context_conditions => {
+                  'HL.HL04' => '0'
+                }
+
+      #Patient Hierarchical Level
+      loop    Hippo::TransactionSets::HIPAA_837::L2000C,
+                :name           => 'Patient Hierarchical Level',
+                :minimum        => 0,
+                :maximum        => 99999,
+                :position       => 10,
+                :identified_by => {
+                  'HL.HL03' => '23',
+                  'HL.HL04' => '0'
                 }
 
     end

data/lib/hippo/transaction_sets/base.rb

@@ -24,29 +24,27 @@ module Hippo::TransactionSets
 
 
       def grouped_components
-        return @grouped_components if @grouped_components
-
-        initial_components = components.dup
-        @grouped_components = []
-        last_entry         = nil
+        @grouped_components ||= build_grouped_components
+      end
 
-        while initial_components.length > 0
-          component = initial_components.first
+      def build_grouped_components
+        output     = []
+        last_entry = nil
 
+        components.each do |component|
           if component.segment?
-            @grouped_components << [] if last_entry != :segment
-            @grouped_components.last << component
+            output << [] if last_entry != :segment
+            output.last << component
             last_entry = :segment
           else
-            @grouped_components << component
+            output << component
             last_entry = :transaction_set
           end
-
-          initial_components.delete(component)
         end
 
-        @grouped_components
+        output
       end
+      private :build_grouped_components
     end
 
     attr_accessor :values, :parent, :sequences, :ISA, :GS, :GE, :IEA
@@ -68,7 +66,6 @@ module Hippo::TransactionSets
 
       grouped_components.each_with_index do |component, component_index|
         if component.class == Array
-          #binding.pry
           # segments
           starting_segment_count = segments.count
           ending_segment_count   = 0
@@ -81,6 +78,7 @@ module Hippo::TransactionSets
 
               break unless segment
               next unless individual_component.valid?(segment)
+              next unless individual_component.conditions_match?(self, segment)
 
               segment.parent = self
 
@@ -103,6 +101,7 @@ module Hippo::TransactionSets
 
             break unless initial_segment
             break unless component.valid?(initial_segment)
+            break unless component.conditions_match?(self, initial_segment)
 
             component_matches_found  = []
             starting_index = component.repeating? ? component_index : component_index + 1

data/lib/hippo/transaction_sets/component.rb

@@ -1,10 +1,11 @@
 module Hippo::TransactionSets
   class Component
-    attr_reader :options, :klass, :sequence, :maximum, :identified_by
+    attr_reader :options, :klass, :sequence, :maximum, :identified_by, :conditions
 
     def initialize(options)
-      @identified_by  = options.delete(:identified_by)  || {}
-      @maximum        = options.delete(:maximum)        || 1
+      @identified_by  = options.delete(:identified_by)              || {}
+      @conditions     = options.delete(:parent_context_conditions)  || {}
+      @maximum        = options.delete(:maximum)                    || 1
       @klass          = options.delete(:klass)
       @sequence       = options.delete(:sequence)
 
@@ -81,5 +82,15 @@ module Hippo::TransactionSets
         segment.identifier == segment_id && Array(value).include?(segment.send(field_name))
       end
     end
+
+    def conditions_match?(instance, segment)
+      if conditions.empty?
+        true
+      else
+        conditions.all? do |method, expected|
+          Array(expected).include?(instance.instance_eval("self." + method))
+        end
+      end
+    end
   end
 end

data/lib/hippo/version.rb

@@ -1,3 +1,3 @@
 module Hippo
-  VERSION = "0.4.1"
+  VERSION = "0.5.0"
 end

data/test/test_helper.rb

@@ -77,6 +77,28 @@ module Hippo::TransactionSets
                 }
     end
 
+    class L0004 < Hippo::TransactionSets::Base
+      loop_name 'L0004'
+
+      segment Hippo::Segments::TSS,
+                :name => 'Indicator of Child Location',
+                :identified_by => {'TSS01' => ['HAS CHILD LOOP','HAS NO CHILD LOOP']}
+
+      loop Hippo::TransactionSets::Test::L0001,
+                :name           => 'Test Sub-Loop L0001',
+                :identified_by  => {
+                  'TSS.TSS01'   => 'Multiple Parents',
+                },
+                :parent_context_conditions => {
+                  'TSS.TSS01' => 'HAS NO CHILD LOOP'
+                }
+
+      segment Hippo::Segments::TSS,
+                :name => 'Indicator of Child Location',
+                :identified_by => {'TSS01' => 'Multiple Parents'}
+
+    end
+
     class Base < Hippo::TransactionSets::Base
 
       segment Hippo::Segments::ST,

data/test/test_parser.rb

@@ -99,7 +99,31 @@ class TestParser < MiniTest::Unit::TestCase
   end
 
   def test_parse_l2000a
-    l2000a = Hippo::TransactionSets::HIPAA_837::L2000A.new.parse(File.read('samples/837_L2000A_01.edi'))
-    l2000a = Hippo::TransactionSets::HIPAA_837::L2000A.new.parse(File.read('samples/837_L2000A_02.edi'))
+    files = ['samples/837_L2000A_01.edi', 'samples/837_L2000A_02.edi']
+    files.each do |f|
+      l2000a = Hippo::TransactionSets::HIPAA_837::L2000A.new.parse(File.read(f))
+
+      # when L2000B HL04 is '0' we must have a L2000C child
+      if l2000a.L2000B.HL.HL04 == '0'
+        assert_nil l2000a.L2000B.L2000C.HL.HL02
+      else
+        assert_equal l2000a.L2000B.HL.HL01, l2000a.L2000B.L2000C.HL.HL02
+      end
+    end
+  end
+
+  def test_same_child_exists_in_multiple_levels
+    no_child_loop_string = "TSS*HAS NO CHILD LOOP~TSS*Multiple Parents~"
+    ts                   = Hippo::TransactionSets::Test::L0004.new
+
+    ts.parse(no_child_loop_string)
+    assert_equal 'TSS*Multiple Parents~', ts.values[1].TSS.to_s
+
+
+    child_loop_string = "TSS*HAS CHILD LOOP~TSS*Multiple Parents~"
+    ts                   = Hippo::TransactionSets::Test::L0004.new
+
+    ts.parse(child_loop_string)
+    assert_equal 'TSS*Multiple Parents~', ts.values[2].to_s
   end
 end

data/test/test_segments_base.rb

@@ -181,4 +181,10 @@ class TestSegmentsBase < MiniTest::Unit::TestCase
 
     assert_equal nil, seg.TCS01_01
   end
+
+  def test_segment_parse_for_composite_fields
+    input_string = "STC*A7:755:87*20121127*U*2200******A3:448**REJECTED AT CLEARINGHOUSE PAY-TO PROVIDER PRIMARY ID# IS NOT USED (1235196510) (59141)"
+    seg = Hippo::Segments::STC.new.parse(input_string)
+    assert_equal input_string + '~', seg.to_s
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hippo
 version: !ruby/object:Gem::Version
-  version: 0.4.1
+  version: 0.5.0
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-09-14 00:00:00.000000000 Z
+date: 2012-12-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
@@ -35,7 +35,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.9.2
+        version: 10.0.2
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -43,7 +43,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: 0.9.2
+        version: 10.0.2
 description: HIPAA Transaction Set Generator/Parser
 email:
 - robertj@promedicalinc.com
@@ -341,7 +341,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 734824617931556958
+      hash: -2604370158452349682
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -350,7 +350,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 734824617931556958
+      hash: -2604370158452349682
 requirements: []
 rubyforge_project: hippo
 rubygems_version: 1.8.24