netsuite 0.8.10 → 0.8.11

data/spec/netsuite/actions/add_spec.rb

@@ -114,4 +114,40 @@ describe NetSuite::Actions::Add do
     end
   end
 
+  context 'File' do
+    let(:file) do
+      NetSuite::Records::File.new(name: 'foo.pdf', content: 'abc123')
+    end
+
+    context 'when successful' do
+      before do
+        savon.expects(:add).with(:message => {
+          'platformMsgs:record' => {
+            :content! => {
+              'fileCabinet:name' => 'foo.pdf',
+              'fileCabinet:content' => 'abc123',
+            },
+            '@xsi:type' => 'fileCabinet:File'
+          },
+        }).returns(File.read('spec/support/fixtures/add/add_file.xml'))
+      end
+
+      it 'makes a valid request to the NetSuite API' do
+        NetSuite::Actions::Add.call([file])
+      end
+
+      it 'returns a valid Response object' do
+        response = NetSuite::Actions::Add.call([file])
+        expect(response).to be_kind_of(NetSuite::Response)
+        expect(response).to be_success
+      end
+
+      it 'properly extracts internal ID from response' do
+        file.add
+
+        expect(file.internal_id).to eq('23556')
+      end
+    end
+  end
+
 end

data/spec/netsuite/actions/attach_file_spec.rb

@@ -0,0 +1,59 @@
+require 'spec_helper'
+
+describe NetSuite::Actions::AttachFile do
+  before(:all) { savon.mock! }
+  after(:all) { savon.unmock! }
+
+  let(:sales_order) { NetSuite::Records::SalesOrder.new(internal_id: 999) }
+  let(:file) {  double('file', internal_id: 111) }
+  let(:message) do
+    {
+      'platformCore:attachReference' => {
+        '@xsi:type' => 'platformCore:AttachBasicReference',
+        'platformCore:attachTo' => {
+          '@internalId' => sales_order.internal_id,
+          '@type' => 'salesOrder',
+          '@xsi:type' => 'platformCore:RecordRef'
+        },
+        'platformCore:attachedRecord' => {
+          '@internalId' => file.internal_id,
+          '@type' => 'file',
+          '@xsi:type' => 'platformCore:RecordRef'
+        }
+      }
+    }
+  end
+
+  before do
+    savon.expects(:attach).with(:message => message).returns(envelope)
+  end
+
+  context 'when successful' do
+    let(:envelope) { File.read('spec/support/fixtures/attach/attach_file_to_sales_order.xml') }
+
+    it 'returns a valid Response object' do
+      response = NetSuite::Actions::AttachFile.call([sales_order, file])
+      expect(response).to be_kind_of(NetSuite::Response)
+      expect(response).to be_success
+    end
+  end
+
+  context 'when not successful' do
+    let(:envelope) { File.read('spec/support/fixtures/attach/attach_file_to_sales_order_error.xml') }
+
+    it 'provides an error method on the object with details about the error' do
+      sales_order.attach_file(file)
+      error = sales_order.errors.first
+
+      expect(error).to be_kind_of(NetSuite::Error)
+      expect(error.type).to eq('ERROR')
+      expect(error.code).to eq('INVALID')
+      expect(error.message).to eq('Invalid request.')
+    end
+
+    it 'provides an error method on the response' do
+      response = NetSuite::Actions::AttachFile.call([sales_order, file])
+      expect(response.errors.first).to be_kind_of(NetSuite::Error)
+    end
+  end
+end

data/spec/netsuite/actions/search_spec.rb

@@ -193,6 +193,30 @@ describe NetSuite::Actions::Search do
       expect(search.results.first.location_re_order_point).to eq('2565.0')
       expect(search.results.first.location_quantity_on_order).to eq('40000.0')
     end
+
+    it "should handle an ID search with basic non-field result columns" do
+      response = File.read('spec/support/fixtures/search/saved_search_item_2.xml')
+      savon.expects(:search)
+        .with(message: {
+          "searchRecord"=>{
+            "@xsi:type"      =>"listAcct:ItemSearchAdvanced",
+            "@savedSearchId" =>42,
+            :content!        =>{"listAcct:criteria"=>{}},
+          }
+        }).returns(response)
+
+      search = NetSuite::Records::InventoryItem.search(saved: 42)
+
+      results = search.results
+      custom_fields = results.map do |record|
+        record.custom_field_list.custom_fields.map(&:internal_id)
+      end.flatten.uniq
+      [
+        :location_quantity_available,
+        :location_re_order_point,
+        :location_quantity_on_order,
+      ].each {|field| expect(custom_fields).to include(field)}
+    end
   end
 
   context "advanced search" do
@@ -201,6 +225,187 @@ describe NetSuite::Actions::Search do
   end
 
   context "basic search" do
+    it "should handle a basic search matching on RecordRef using internalId" do
+      response = File.read('spec/support/fixtures/search/basic_search_contact.xml')
+      savon.expects(:search)
+        .with(message: {
+          "searchRecord" => {
+            :content! => {
+              "listRel:basic" => {
+                "platformCommon:company" => {
+                  "@operator" => "anyOf",
+                  "@xsi:type" => "platformCore:SearchMultiSelectField",
+                  "platformCore:searchValue" => [
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@internalId" => 7497,
+                    },
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@internalId" => 7270,
+                    },
+                  ],
+                },
+              },
+            },
+            "@xsi:type" => "listRel:ContactSearch",
+          },
+        }).returns(response)
+
+      search = NetSuite::Records::Contact.search(
+        basic: [{
+          field: 'company',
+          operator: 'anyOf',
+          value: [
+            NetSuite::Records::RecordRef.new(internal_id: 7497),
+            NetSuite::Records::RecordRef.new(internal_id: 7270),
+          ],
+        }],
+      )
+
+      expect(search.results.size).to eq(1)
+    end
+
+    it "should handle a basic search matching on RecordRef using externalId" do
+      response = File.read('spec/support/fixtures/search/basic_search_contact.xml')
+      savon.expects(:search)
+        .with(message: {
+          "searchRecord" => {
+            :content! => {
+              "listRel:basic" => {
+                "platformCommon:company" => {
+                  "@operator" => "anyOf",
+                  "@xsi:type" => "platformCore:SearchMultiSelectField",
+                  "platformCore:searchValue" => [
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@externalId" => "external_abc",
+                    },
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@externalId" => "external_xyz",
+                    },
+                  ],
+                },
+              },
+            },
+            "@xsi:type" => "listRel:ContactSearch",
+          },
+        }).returns(response)
+
+      search = NetSuite::Records::Contact.search(
+        basic: [{
+          field: 'company',
+          operator: 'anyOf',
+          value: [
+            NetSuite::Records::RecordRef.new(external_id: "external_abc"),
+            NetSuite::Records::RecordRef.new(external_id: "external_xyz"),
+          ],
+        }],
+      )
+
+      expect(search.results.size).to eq(1)
+    end
+
+    it "should handle a basic search matching on RecordRef using mix of internalId and externalId" do
+      response = File.read('spec/support/fixtures/search/basic_search_contact.xml')
+      savon.expects(:search)
+        .with(message: {
+          "searchRecord" => {
+            :content! => {
+              "listRel:basic" => {
+                "platformCommon:company" => {
+                  "@operator" => "anyOf",
+                  "@xsi:type" => "platformCore:SearchMultiSelectField",
+                  "platformCore:searchValue" => [
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@internalId" => 7497,
+                    },
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@externalId" => "external_xyz",
+                    },
+                  ],
+                },
+              },
+            },
+            "@xsi:type" => "listRel:ContactSearch",
+          },
+        }).returns(response)
+
+      search = NetSuite::Records::Contact.search(
+        basic: [{
+          field: 'company',
+          operator: 'anyOf',
+          value: [
+            NetSuite::Records::RecordRef.new(internal_id: 7497),
+            NetSuite::Records::RecordRef.new(external_id: "external_xyz"),
+          ],
+        }],
+      )
+
+      expect(search.results.size).to eq(1)
+    end
+
+    it "should handle a basic search matching on RecordRef using both internalId and externalId" do
+      response = File.read('spec/support/fixtures/search/basic_search_contact.xml')
+      savon.expects(:search)
+        .with(message: {
+          "searchRecord" => {
+            :content! => {
+              "listRel:basic" => {
+                "platformCommon:company" => {
+                  "@operator" => "anyOf",
+                  "@xsi:type" => "platformCore:SearchMultiSelectField",
+                  "platformCore:searchValue" => [
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@internalId" => 7497,
+                      "@externalId" => "external_abc",
+                    },
+                    {
+                      :content! => {},
+                      "@xsi:type" => "platformCore:RecordRef",
+                      "@type" => "account",
+                      "@externalId" => 7270,
+                    },
+                  ],
+                },
+              },
+            },
+            "@xsi:type" => "listRel:ContactSearch",
+          },
+        }).returns(response)
+
+      search = NetSuite::Records::Contact.search(
+        basic: [{
+          field: 'company',
+          operator: 'anyOf',
+          value: [
+            NetSuite::Records::RecordRef.new(internal_id: 7497, external_id: "external_abc"),
+            NetSuite::Records::RecordRef.new(external_id: 7270),
+          ],
+        }],
+      )
+
+      expect(search.results.size).to eq(1)
+    end
+
     skip "should handle searching basic fields"
     skip "should handle searching with joined fields"
   end

data/spec/netsuite/records/inventory_item_spec.rb

@@ -5,27 +5,169 @@ describe NetSuite::Records::InventoryItem do
 
   it 'has all the right fields' do
     [
-      :auto_lead_time, :auto_preferred_stock_level, :auto_reorder_point, :available_to_partners, :average_cost, :copy_description,
-      :cost, :cost_estimate, :cost_estimate_type, :cost_estimate_units, :cost_units, :costing_method, :costing_method_display,
-      :country_of_manufacture, :created_date, :currency, :date_converted_to_inv, :default_return_cost, :demand_modifier,
-      :display_name, :dont_show_price, :enforce_min_qty_internally, :exclude_from_sitemap, :featured_description, :fixed_lot_size,
-      :handling_cost, :handling_cost_units, :include_children, :is_donation_item, :is_drop_ship_item, :is_gco_compliant,
-      :is_inactive, :is_online, :is_special_order_item, :is_taxable, :item_id, :last_modified_date, :last_purchase_price,
-      :lead_time, :manufacturer, :manufacturer_addr1, :manufacturer_city, :manufacturer_state, :manufacturer_tariff,
-      :manufacturer_tax_id, :manufacturer_zip, :match_bill_to_receipt, :matrix_type, :max_donation_amount, :meta_tag_html,
-      :minimum_quantity, :minimum_quantity_units, :mpn, :mult_manufacture_addr, :nex_tag_category, :no_price_message,
-      :offer_support, :on_hand_value_mli, :on_special, :original_item_subtype, :original_item_type, :out_of_stock_behavior,
-      :out_of_stock_message, :overall_quantity_pricing_type, :page_title, :preference_criterion, :preferred_stock_level,
-      :preferred_stock_level_days, :preferred_stock_level_units, :prices_include_tax, :producer, :purchase_description,
-      :quantity_available, :quantity_available_units, :quantity_back_ordered, :quantity_committed, :quantity_committed_units,
-      :quantity_on_hand, :quantity_on_hand_units, :quantity_on_order, :quantity_on_order_units, :quantity_reorder_units, :rate,
-      :related_items_description, :reorder_multiple, :reorder_point, :reorder_point_units, :safety_stock_level,
-      :safety_stock_level_days, :safety_stock_level_units, :sales_description, :schedule_b_code, :schedule_b_number,
-      :schedule_b_quantity, :search_keywords, :seasonal_demand, :ship_individually, :shipping_cost, :shipping_cost_units,
-      :shopping_dot_com_category, :shopzilla_category_id, :show_default_donation_amount, :sitemap_priority, :specials_description,
-      :stock_description, :store_description, :store_detailed_description, :store_display_name, :total_value, :track_landed_cost,
-      :transfer_price, :upc_code, :url_component, :use_bins, :use_marginal_rates, :vendor_name, :vsoe_deferral, :vsoe_delivered,
-      :vsoe_permit_discount, :vsoe_price, :weight, :weight_unit, :weight_units
+      :auto_lead_time,
+      :auto_preferred_stock_level,
+      :auto_reorder_point,
+      :available_to_partners,
+      :average_cost,
+      :backward_consumption_days,
+      :contingent_revenue_handling,
+      :conversion_rate,
+      :copy_description,
+      :cost,
+      :cost_estimate,
+      :cost_estimate_type,
+      :cost_estimate_units,
+      :costing_method,
+      :costing_method_display,
+      :cost_units,
+      :country_of_manufacture,
+      :created_date,
+      :currency,
+      :date_converted_to_inv,
+      :default_return_cost,
+      :defer_rev_rec,
+      :demand_modifier,
+      :demand_time_fence,
+      :direct_revenue_posting,
+      :display_name,
+      :dont_show_price,
+      :enable_catch_weight,
+      :enforce_min_qty_internally,
+      :exclude_from_sitemap,
+      :featured_description,
+      :fixed_lot_size,
+      :forward_consumption_days,
+      :fraud_risk,
+      :future_horizon,
+      :handling_cost,
+      :handling_cost_units,
+      :hazmat_hazard_class,
+      :hazmat_id,
+      :hazmat_item_units,
+      :hazmat_item_units_qty,
+      :hazmat_packing_group,
+      :hazmat_shipping_name,
+      :include_children,
+      :invt_classification,
+      :invt_count_interval,
+      :is_donation_item,
+      :is_drop_ship_item,
+      :is_gco_compliant,
+      :is_hazmat_item,
+      :is_inactive,
+      :is_online,
+      :is_special_order_item,
+      :is_store_pickup_allowed,
+      :is_taxable,
+      :item_carrier,
+      :item_id,
+      :last_invt_count_date,
+      :last_modified_date,
+      :last_purchase_price,
+      :lead_time,
+      :lower_warning_limit,
+      :manufacturer,
+      :manufacturer_addr1,
+      :manufacturer_city,
+      :manufacturer_state,
+      :manufacturer_tariff,
+      :manufacturer_tax_id,
+      :manufacturer_zip,
+      :match_bill_to_receipt,
+      :matrix_item_name_template,
+      :matrix_type,
+      :max_donation_amount,
+      :maximum_quantity,
+      :meta_tag_html,
+      :minimum_quantity,
+      :minimum_quantity_units,
+      :mpn,
+      :mult_manufacture_addr,
+      :nex_tag_category,
+      :next_invt_count_date,
+      :no_price_message,
+      :offer_support,
+      :on_hand_value_mli,
+      :on_special,
+      :original_item_subtype,
+      :original_item_type,
+      :out_of_stock_behavior,
+      :out_of_stock_message,
+      :overall_quantity_pricing_type,
+      :page_title,
+      :periodic_lot_size_days,
+      :periodic_lot_size_type,
+      :preference_criterion,
+      :preferred_stock_level,
+      :preferred_stock_level_days,
+      :preferred_stock_level_units,
+      :prices_include_tax,
+      :producer,
+      :purchase_description,
+      :purchase_order_amount,
+      :purchase_order_quantity,
+      :purchase_order_quantity_diff,
+      :quantity_available,
+      :quantity_available_units,
+      :quantity_back_ordered,
+      :quantity_committed,
+      :quantity_committed_units,
+      :quantity_on_hand,
+      :quantity_on_hand_units,
+      :quantity_on_order,
+      :quantity_on_order_units,
+      :quantity_reorder_units,
+      :rate,
+      :receipt_amount,
+      :receipt_quantity,
+      :receipt_quantity_diff,
+      :related_items_description,
+      :reorder_multiple,
+      :reorder_point,
+      :reorder_point_units,
+      :reschedule_in_days,
+      :reschedule_out_days,
+      :round_up_as_component,
+      :safety_stock_level,
+      :safety_stock_level_days,
+      :safety_stock_level_units,
+      :sales_description,
+      :schedule_b_code,
+      :schedule_b_number,
+      :schedule_b_quantity,
+      :search_keywords,
+      :seasonal_demand,
+      :ship_individually,
+      :shipping_cost,
+      :shipping_cost_units,
+      :shopping_dot_com_category,
+      :shopzilla_category_id,
+      :show_default_donation_amount,
+      :sitemap_priority,
+      :specials_description,
+      :stock_description,
+      :store_description,
+      :store_detailed_description,
+      :store_display_name,
+      :supply_time_fence,
+      :total_value,
+      :track_landed_cost,
+      :transfer_price,
+      :upc_code,
+      :upper_warning_limit,
+      :url_component,
+      :use_bins,
+      :use_marginal_rates,
+      :vendor_name,
+      :vsoe_deferral,
+      :vsoe_delivered,
+      :vsoe_permit_discount,
+      :vsoe_price,
+      :vsoe_sop_group,
+      :weight,
+      :weight_unit,
+      :weight_units,
     ].each do |field|
       expect(item).to have_field(field)
     end
@@ -95,9 +237,84 @@ describe NetSuite::Records::InventoryItem do
     end
   end
 
+  it 'has all the right fields with specific classes' do
+    {
+      bin_number_list: NetSuite::Records::BinNumberList,
+      custom_field_list: NetSuite::Records::CustomFieldList,
+      item_ship_method_list: NetSuite::Records::RecordRefList,
+      item_vendor_list: NetSuite::Records::ItemVendorList,
+      locations_list: NetSuite::Records::LocationsList,
+      matrix_option_list: NetSuite::Records::MatrixOptionList,
+      pricing_matrix: NetSuite::Records::PricingMatrix,
+      subsidiary_list: NetSuite::Records::RecordRefList,
+    }.each do |field, klass|
+      expect(item).to have_field(field, klass)
+    end
+  end
+
   it 'has all the right record refs' do
     [
-      :alternate_demand_source_item, :asset_account, :bill_exch_rate_variance_acct, :bill_price_variance_acct, :bill_qty_variance_acct, :billing_schedule, :cogs_account, :cost_category, :custom_form, :deferred_revenue_account, :demand_source, :department, :expense_account, :gain_loss_account, :income_account, :issue_product, :klass, :location, :parent, :preferred_location, :pricing_group, :purchase_price_variance_acct, :purchase_tax_code, :purchase_unit, :quantity_pricing_schedule, :rev_rec_schedule, :sale_unit, :sales_tax_code, :ship_package, :soft_descriptor, :stock_unit, :store_display_image, :store_display_thumbnail, :store_item_template, :supply_lot_sizing_method, :supply_replenishment_method, :supply_type, :tax_schedule, :units_type, :vendor
+      :alternate_demand_source_item,
+      :asset_account,
+      :bill_exch_rate_variance_acct,
+      :billing_schedule,
+      :bill_price_variance_acct,
+      :bill_qty_variance_acct,
+      :klass,
+      :cogs_account,
+      :consumption_unit,
+      :cost_category,
+      :create_revenue_plans_on,
+      :custom_form,
+      :default_item_ship_method,
+      :deferred_revenue_account,
+      :demand_source,
+      :department,
+      :distribution_category,
+      :distribution_network,
+      :dropship_expense_account,
+      :expense_account,
+      :gain_loss_account,
+      :income_account,
+      :interco_cogs_account,
+      :interco_def_rev_account,
+      :interco_income_account,
+      :issue_product,
+      :item_revenue_category,
+      :location,
+      :parent,
+      :planning_item_category,
+      :preferred_location,
+      :pricing_group,
+      :purchase_price_variance_acct,
+      :purchase_tax_code,
+      :purchase_unit,
+      :quantity_pricing_schedule,
+      :revenue_allocation_group,
+      :revenue_recognition_rule,
+      :rev_rec_forecast_rule,
+      :rev_reclass_f_x_account,
+      :rev_rec_schedule,
+      :sales_tax_code,
+      :sale_unit,
+      :secondary_base_unit,
+      :secondary_consumption_unit,
+      :secondary_purchase_unit,
+      :secondary_sale_unit,
+      :secondary_stock_unit,
+      :secondary_units_type,
+      :ship_package,
+      :soft_descriptor,
+      :stock_unit,
+      :store_display_image,
+      :store_display_thumbnail,
+      :store_item_template,
+      :supply_lot_sizing_method,
+      :supply_replenishment_method,
+      :supply_type,
+      :tax_schedule,
+      :units_type,
+      :vendor,
     ].each do |record_ref|
       expect(item).to have_record_ref(record_ref)
     end

data/spec/netsuite/records/invoice_spec.rb

@@ -392,6 +392,35 @@ describe NetSuite::Records::Invoice do
     end
   end
 
+  describe '#attach_file' do
+    let(:test_data) { { :email => 'test@example.com', :fax => '1234567890' } }
+    let(:file) { double('file') }
+
+    context 'when the response is successful' do
+      let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }
+
+      it 'returns true' do
+        invoice = NetSuite::Records::Invoice.new(test_data)
+        expect(NetSuite::Actions::AttachFile).to receive(:call).
+          with([invoice, file], {}).
+          and_return(response)
+        expect(invoice.attach_file(file)).to be_truthy
+      end
+    end
+
+    context 'when the response is unsuccessful' do
+      let(:response) { NetSuite::Response.new(:success => false, :body => {}) }
+
+      it 'returns false' do
+        invoice = NetSuite::Records::Invoice.new(test_data)
+        expect(NetSuite::Actions::AttachFile).to receive(:call).
+          with([invoice, file], {}).
+          and_return(response)
+        expect(invoice.attach_file(file)).to be_falsey
+      end
+    end
+  end
+
   describe '#delete' do
     context 'when the response is successful' do
       let(:response) { NetSuite::Response.new(:success => true, :body => { :internal_id => '1' }) }

data/spec/netsuite/records/item_option_custom_field_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe NetSuite::Records::ItemOptionCustomField do
+
+  describe ".get" do
+    let(:response) do
+      NetSuite::Response.new(
+        success: true,
+        body: {
+          label: "Value of Label",
+        }
+      )
+    end
+
+    it "returns a ItemOptionCustomField instance with populated fields" do
+      expect(NetSuite::Actions::Get)
+        .to receive(:call)
+        .with([NetSuite::Records::ItemOptionCustomField, internal_id: 1], {})
+        .and_return(response)
+
+      record = NetSuite::Records::ItemOptionCustomField.get(internal_id: 1)
+      
+      expect(record.label).to eql("Value of Label")
+    end
+  end
+
+end

data/spec/netsuite/records/item_vendor_list_spec.rb

@@ -10,17 +10,14 @@ describe NetSuite::Records::ItemVendorList do
   describe '#to_record' do
     before do
       list.item_vendors << NetSuite::Records::ItemVendor.new(
-        :vendor => {:name => 'Spring Water'}
+        :vendor_code => 'Spring Water',
       )
     end
 
     it 'can represent itself as a SOAP record' do
       record =  {
         "listAcct:itemVendor"=>[
-          {"listAcct:vendor"=>{
-            :name=>"Spring Water"
-            }
-          }
+          {"listAcct:vendorCode"=>"Spring Water"}
         ]
       }