efo_nelfo 0.0.7 → 1.0.0

data/spec/csv_writer_spec.rb

@@ -1,8 +1,8 @@
 # encoding: utf-8
 require 'spec_helper'
 
-describe EfoNelfo::V40::Order do
-  let(:order) { EfoNelfo::V40::Order.new buyer_id: '123', customer_id: '456' }
+describe EfoNelfo::V40::BH do
+  let(:order) { EfoNelfo::V40::BH.new buyer_id: '123', customer_id: '456' }
   let(:csv)   { order.to_csv }
 
   describe "to_a" do
@@ -12,8 +12,8 @@ describe EfoNelfo::V40::Order do
     end
 
     it "includes the lines" do
-      order.add EfoNelfo::V40::Order::Line.new
-      order.add EfoNelfo::V40::Order::Line.new
+      order.add EfoNelfo::V40::BL.new
+      order.add EfoNelfo::V40::BL.new
       order.to_a.size.must_equal 3
     end
 
@@ -21,8 +21,8 @@ describe EfoNelfo::V40::Order do
 
   describe "to_csv" do
     before do
-      order.add EfoNelfo::V40::Order::Line.new(order_number: 'foo', item_name: 'Ware', item_count: 6, splitable: true)
-      order.add EfoNelfo::V40::Order::Text.new text: 'haha'
+      order.add EfoNelfo::V40::BL.new order_number: 'foo', item_name: 'Ware', item_count: 6, splitable: true
+      order.add EfoNelfo::V40::BT.new text: 'haha'
     end
 
     it "includes standard fields" do
@@ -30,22 +30,22 @@ describe EfoNelfo::V40::Order do
       csv.must_match /;4.0;/
       csv.must_match /;123;/
       csv.must_match /;600;/
-      csv.must_match /;J;/
+      csv.must_match /;J;/              # for fixing sublime syntax highlighting: /
     end
 
     it "can be parsed" do
       o = EfoNelfo.parse(csv)
-      o.must_be_instance_of EfoNelfo::V40::Order
-      o.lines.first.text.must_be_instance_of EfoNelfo::V40::Order::Text
+      o.must_be_instance_of EfoNelfo::V40::BH
+      o.lines.first.text.first.must_be_instance_of EfoNelfo::V40::BT
     end
 
     it "adds order lines" do
-      order.add EfoNelfo::V40::Order::Line.new(order_number: 'foo', item_name: 'Ware')
+      order.add EfoNelfo::V40::BL.new(order_number: 'foo', item_name: 'Ware')
       csv.must_match /Ware/
     end
 
     it "adds text to order line" do
-      csv.must_match /haha/
+      csv.must_match /BT;haha/
     end
 
   end

data/spec/efo_nelfo_spec.rb

@@ -10,19 +10,19 @@ describe EfoNelfo do
 
   describe "properties" do
     it "is accessible as alias" do
-      order = EfoNelfo::V40::Order.new
+      order = EfoNelfo::V40::BH.new
       order.buyer_id = "123"
       order.KjøpersId.must_equal "123"
     end
 
     it "can update the variable using the alias" do
-      order = EfoNelfo::V40::Order.new
+      order = EfoNelfo::V40::BH.new
       order.KjøpersId = "foo"
       order.buyer_id.must_equal "foo"
     end
 
     it "has custom setters" do
-      EfoNelfo::V40::Order.new(delivery_date: "20110402").delivery_date.must_be_kind_of Date
+      EfoNelfo::V40::BH.new(delivery_date: "20110402").delivery_date.must_be_kind_of Date
     end
 
   end
@@ -34,15 +34,17 @@ describe EfoNelfo do
         seller_warehouse: "Somewhere",
         label: "Some label",
         lines: [
-          { item_type: 1, order_number: "12345", item_name: "Foo", price_unit: "NOK" },
-          { item_type: 1, order_number: "12345", item_name: "Bar", price_unit: "SEK", text: "This is freetext"},
+          { post_type: 'BL', item_type: 1, order_number: "12345", item_name: "Foo", price_unit: "NOK" },
+          { post_type: 'BL', item_type: 1, order_number: "12345", item_name: "Bar", price_unit: "SEK", text: [
+            { post_type: 'BT', text: "some dummy lorem ipsum text" }
+          ] }
         ]
       }
     }
-    let(:order) { EfoNelfo::V40::Order.new json }
+    let(:order) { EfoNelfo::V40::BH.new json }
 
     it "creates an order" do
-      order.must_be_instance_of EfoNelfo::V40::Order
+      order.must_be_instance_of EfoNelfo::V40::BH
     end
 
     it "assigns attributes" do
@@ -52,15 +54,20 @@ describe EfoNelfo do
 
     it "assigns order lines" do
       order.lines.size.must_equal 2
+      order.lines.first.must_be_instance_of EfoNelfo::V40::BL
+      order.lines[0].item_name.must_equal "Foo"
+      order.lines[1].item_name.must_equal "Bar"
+    end
+
+    it "adds index to the line items" do
       order.lines[0].index.must_equal 1
       order.lines[1].index.must_equal 2
-      order.lines[1].item_name.must_equal "Bar"
     end
 
     it "adds text to order lines" do
-      order.lines.first.text.must_be_nil
-      order.lines[1].text.must_be_instance_of EfoNelfo::V40::Order::Text
-      order.lines[1].text.to_s.must_equal "This is freetext"
+      order.lines.first.text.must_be_empty
+      order.lines[1].text.first.must_be_instance_of EfoNelfo::V40::BT
+      order.lines[1].text.first.to_s.must_equal "some dummy lorem ipsum text"
     end
 
   end
@@ -84,6 +91,60 @@ describe EfoNelfo do
       end
     end
 
+    describe "passing a Product file" do
+      let(:nelfo) { EfoNelfo.load(csv('varefil_eksempel.csv')) }
+
+      it { nelfo.must_be_instance_of EfoNelfo::V40::VH }
+
+      it "loads the order" do
+        nelfo.format.must_equal "EFONELFO"
+        nelfo.version.must_equal "4.0"
+        nelfo.post_type.must_equal "VH"
+        nelfo.SelgersId.must_equal "NO1234567879MVA"
+        nelfo.KjøpersId.must_be_nil
+        nelfo.KundeNr.must_be_nil
+        nelfo.from_date.must_equal Date.new(2004, 1, 22)
+        nelfo.to_date.must_be_nil
+        nelfo.currency.must_equal "NOK"
+        nelfo.contract_id.must_be_nil
+        nelfo.seller_name.must_equal "ONNINEN A/S"
+        nelfo.seller_address1.must_equal "P.B. 70"
+        nelfo.seller_address2.must_be_nil
+        nelfo.seller_zip.must_equal "1483"
+        nelfo.seller_office.must_equal "SKYTTA"
+        nelfo.seller_country.must_equal "NO"
+      end
+
+      it "adds product lines" do
+        nelfo.lines.count.must_equal 3
+
+        line = nelfo.lines.first
+        line.product_type.must_equal 1
+        line.product_number.must_equal "1006100"
+        line.name.must_equal "MATTE 3MM 100W/M2  100W"
+        line.description.must_equal "1,0 m2 0,5 x 2,0 m"
+        line.unit.must_equal 1
+        line.price_unit.must_equal "EA"
+        line.price_unit_desc.must_equal "STYKK"
+        line.price.must_equal 55300
+        line.amount.must_equal 10000
+        line.price_date.must_equal Date.new(2011,9,1)
+        line.status.must_equal 1
+        line.block_number.must_be_nil
+        line.discount_group.must_equal "1004"
+        line.fabrication.must_be_nil
+        line.type.must_be_nil
+        line.stocked.must_equal true
+        line.stocked?.must_equal true
+        line.sales_package.must_equal 10000
+      end
+
+      it "adds additional information to the product" do
+        line = nelfo.lines.first
+        line.info.first.value.must_equal "http://www.elektroskandia.no/image/products/1221102.gif"
+      end
+    end
+
     describe "passing a Order file" do
 
       it "uses the correct version" do
@@ -91,13 +152,12 @@ describe EfoNelfo do
       end
 
       it "parses the file and returns an Order" do
-        EfoNelfo.load(csv('B650517.032.csv')).must_be_instance_of EfoNelfo::V40::Order
+        EfoNelfo.load(csv('B650517.032.csv')).must_be_instance_of EfoNelfo::V40::BH
       end
 
       it "the order contains order information" do
         order = EfoNelfo.load(csv('B650517.032.csv'))
         order.post_type.must_equal 'BH'
-        order.post_type_human.must_equal 'Bestilling Hodepost'
         order.format.must_equal 'EFONELFO'
         order.version.must_equal '4.0'
       end
@@ -106,10 +166,9 @@ describe EfoNelfo do
         order = EfoNelfo.load(csv('B650517.032.csv'))
 
         line = order.lines.first
-        line.must_be_instance_of EfoNelfo::V40::Order::Line
+        line.must_be_instance_of EfoNelfo::V40::BL
 
         line.post_type.must_equal 'BL'
-        line.post_type_human.must_equal 'Bestilling vareLinjepost'
 
         line.index.must_equal 1
         line.order_number.must_equal '1465'
@@ -129,8 +188,7 @@ describe EfoNelfo do
 
       it "adds text to orderline" do
         order = EfoNelfo.load(csv('B650517.032.csv'))
-        line  = order.lines.first
-        line.text.to_s.must_equal "Her er litt fritekst"
+        order.lines.first.text.first.to_s.must_equal "Her er litt fritekst"
       end
 
       it "stores the contents file in the Order object" do
@@ -143,7 +201,7 @@ describe EfoNelfo do
     # Loads all files in the samples directory
     %w(B028579.594.csv B028579.596.csv B650517.031.csv B028579.595.csv B028579.597.csv B650517.030.csv B650517.032.csv).each do |file|
       it "can load #{file}" do
-        EfoNelfo.load(csv(file)).must_be_instance_of EfoNelfo::V40::Order
+        EfoNelfo.load(csv(file)).must_be_instance_of EfoNelfo::V40::BH
       end
     end
 

data/spec/post_type_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe EfoNelfo::PostType do
+
+  module EfoNelfo
+    module V21
+      class Foo < EfoNelfo::PostType
+        property :post_type
+        property :version
+        property :seller_id
+        has_many :lines, :post_type => "Bar"
+      end
+
+      class Bar < EfoNelfo::PostType
+        property :post_type
+        property :order_number
+      end
+    end
+  end
+
+  describe "versioning" do
+
+    it ".version_from_class returns the version" do
+      EfoNelfo::V21::Foo.version_from_class.must_equal "21"
+    end
+
+    it ".version returns the version as a number" do
+      EfoNelfo::V21::Foo.version.must_equal "2.1"
+    end
+
+    it ".version_to_namespace converts version to module name" do
+      EfoNelfo::V21::Foo.version_to_namespace("2.1").must_equal "21"
+    end
+
+  end
+
+  describe ".from" do
+    let(:hash) {
+      {
+        post_type: "Foo",
+        version: "2.1",
+        seller_id: "123",
+        lines: [
+          { post_type: "Bar", order_number: "666-2" }
+        ]
+      }
+    }
+
+    let(:pt) { EfoNelfo::PostType.from hash }
+
+    it "converts the hash into a valid posttype" do
+      pt.must_be_instance_of EfoNelfo::V21::Foo
+      pt.seller_id.must_equal "123"
+    end
+
+    # it "adds lines" do
+    #   pt.lines.size.must_equal 1
+    #   pt.lines.first.order_number.must_equal "666-2"
+    # end
+
+  end
+
+end

data/spec/property_spec.rb

@@ -25,6 +25,15 @@ describe EfoNelfo::Property do
     }.must_raise EfoNelfo::DuplicateProperty
   end
 
+  it "raises an error if trying to assign an unsupported option" do
+    lambda {
+      class Bar
+        include EfoNelfo::Property
+        property :lame, funny: true
+      end
+    }.must_raise EfoNelfo::UnknownPropertyOption
+  end
+
   it "adds a getter and setter for foo" do
     obj.foo = 'Test'
     obj.foo.must_equal 'Test'
@@ -118,6 +127,11 @@ describe EfoNelfo::Property do
     end
   end
 
+  it "#has_property? returns true when property exists" do
+    obj.has_property?(:foo).must_equal true
+    obj.has_property?(:bullshit).must_equal false
+  end
+
   describe "#to_a" do
     it "returns array of all attributes in correct order" do
       obj.date = Date.new 2012, 5, 30