fuelsdk 0.0.4 → 0.0.5

data/lib/fuelsdk/soap.rb

@@ -54,7 +54,7 @@ module FuelSDK
         definition = raw.body[raw.body.keys.first][:object_definition]
         _props = definition[:properties]
         _props.each do  |p|
-          @retrievable << p[:name] if p[:is_retrievable]
+          @retrievable << p[:name] if p[:is_retrievable] and (p[:name] != 'DataRetentionPeriod')
           @updatable << p[:name] if p[:is_updatable]
           @required << p[:name] if p[:is_required]
           @properties << p[:name]
@@ -125,7 +125,7 @@ module FuelSDK
     end
 
     def soap_get object_type, properties=nil, filter=nil
-      if properties.nil?
+      if properties.nil? or properties.empty?
         rsp = soap_describe object_type
         if rsp.success?
           properties = rsp.retrievable
@@ -169,15 +169,6 @@ module FuelSDK
       soap_cud :delete, object_type, properties
     end
 
-    def format_attributes attributes
-      attrs = []
-      attributes.each do |name, value|
-        attrs.push 'Name' => name, 'Value' => value
-      end
-
-      attrs
-    end
-
     private
 
       def soap_cud action, object_type, properties
@@ -191,7 +182,7 @@ module FuelSDK
           p.each do |k, v|
             if type_attrs.include? k
               p.delete k
-              attrs = format_attributes k => v
+              attrs = FuelSDK.format_name_value_pairs k => v
               formated_attrs.concat attrs
             end
           end

data/lib/fuelsdk/utils.rb

@@ -0,0 +1,11 @@
+module FuelSDK
+  module_function
+    def format_name_value_pairs attributes
+      attrs = []
+      attributes.each do |name, value|
+        attrs.push 'Name' => name, 'Value' => value
+      end
+
+      attrs
+    end
+end

data/lib/fuelsdk/version.rb

@@ -1,3 +1,3 @@
 module FuelSDK
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/samples/sample-AddSubscriberToList.rb

@@ -1,7 +1,8 @@
-require '../ET_Client.rb'
+require 'fuelsdk'
+require_relative 'sample_helper'
 
 begin
-	stubObj = ET_Client.new(false, false)
+	stubObj = ET_Client.new auth
 
 	NewListName = "RubySDKList"
 
@@ -19,6 +20,7 @@ begin
 	p 'Message: ' + postResponse.message.to_s
 	p 'Result Count: ' + postResponse.results.length.to_s
 	p 'Results: ' + postResponse.results.inspect
+  raise 'Failure creating list' unless postResponse.success?
 
 	if postResponse.status then
 
@@ -31,6 +33,7 @@ begin
 		p 'Message: ' + AddSubResponse.message.to_s
 		p 'Result Count: ' + AddSubResponse.results.length.to_s
 		p 'Results: ' + AddSubResponse.results.inspect
+    raise 'Failure adding user to list' unless AddSubResponse.success?
 
 		# Delete List
 		p '>>> Delete List'
@@ -43,6 +46,7 @@ begin
 		p 'Message: ' + deleteResponse.message.to_s
 		p 'Results Length: ' + deleteResponse.results.length.to_s
 		p 'Results: ' + deleteResponse.results.to_s
+    raise 'Failure deleting list' unless deleteResponse.success?
 	end
 
 rescue => e

data/samples/sample-CreateDataExtensions.rb

@@ -1,7 +1,8 @@
-require '../ET_Client.rb'
+require 'fuelsdk'
+require_relative 'sample_helper'
 
 begin
-	myclient = ET_Client.new(false, false)
+	myclient = ET_Client.new auth
 
 	## Example using CreateDataExtensions() method
 
@@ -20,6 +21,8 @@ begin
 	p 'Message: ' + createDEResponse.message.to_s
 	p 'Results Length: ' + createDEResponse.results.length.to_s
 	p 'Results: ' + createDEResponse.results.to_s
+  raise 'Failure creating data extensions' unless createDEResponse.success?
+  raise 'Failure creating data extensions' unless createDEResponse.results.count == 2
 
 	# Cleaning up the newly created DEs
 	# Delete deOne

data/samples/sample-dataextension.rb

@@ -1,9 +1,9 @@
-require '../ET_Client.rb'
-
+require 'fuelsdk'
+require_relative 'sample_helper'
+require 'pry'
 
 begin
-	stubObj = ET_Client.new(false, false)
-
+	stubObj = ET_Client.new auth
 
 	# Get all of the DataExtensions in an Account
 	p '>>> Get all of the DataExtensions in an Account'
@@ -14,29 +14,31 @@ begin
 	p 'Retrieve Status: ' + getResponse.status.to_s
 	p 'Code: ' + getResponse.code.to_s
 	p 'Message: ' + getResponse.message.to_s
-	p 'MoreResults: ' + getResponse.moreResults.to_s
+	p 'MoreResults: ' + getResponse.more?.to_s
 	p 'RequestID: ' + getResponse.request_id.to_s
 	p 'Results Length: ' + getResponse.results.length.to_s
 	#p 'Results: ' + getResponse.results.to_s
+  raise 'Failure retrieving data extensions' unless getResponse.success?
 
 	# Specify a name for the data extension that will be used for testing
 	# Note: Name and CustomerKey will be the same value
 	# WARNING: Data Extension will be deleted so don't use the name of a
 	# production data extension
-	NameOfDE = "ThisWillBeDeleted-Test"
-
+	NameOfDE = "ThisWillBeDeleted-Testz"
 
 	# Create  Data Extension
 	p '>>> Create Data Extension'
 	de2 = ET_DataExtension.new
 	de2.authStub = stubObj
 	de2.props = {"Name" => NameOfDE,"CustomerKey" => NameOfDE}
-	de2.columns = [{"Name" => "Name", "FieldType" => "Text", "IsPrimaryKey" => "true", "MaxLength" => "100", "IsRequired" => "true"},{"Name" => "OtherField", "FieldType" => "Text"}]
+	de2.columns = [{"Name" => "Name", "FieldType" => "Text", "IsPrimaryKey" => "true", "MaxLength" => "100", "IsRequired" => "true"},
+  {"Name" => "OtherField", "FieldType" => "Text"}]
 	postResponse = de2.post
 	p 'Post Status: ' + postResponse.status.to_s
 	p 'Code: ' + postResponse.code.to_s
 	p 'Message: ' + postResponse.message.to_s
 	p 'Results: ' + postResponse.results.inspect
+  raise 'Failure creating data extension' unless postResponse.success?
 
 	# Update DE to add new field
 	p '>>> Update DE to add new field'
@@ -49,8 +51,7 @@ begin
 	p 'Code: ' + patchResponse.code.to_s
 	p 'Message: ' + patchResponse.message.to_s
 	p 'Results: ' + patchResponse.results.inspect
-
-
+  raise 'Failure updating data extension' unless patchResponse.success?
 
 	# Retrieve all columns for data extension
 	p '>>> Retrieve all columns for data extension '
@@ -62,11 +63,12 @@ begin
 	p 'Retrieve Status: ' + getResponse.status.to_s
 	p 'Code: ' + getResponse.code.to_s
 	p 'Message: ' + getResponse.message.to_s
-	p 'MoreResults: ' + getResponse.moreResults.to_s
+	p 'MoreResults: ' + getResponse.more?.to_s
 	p 'RequestID: ' + getResponse.request_id.to_s
 	p 'Results Length: ' + getResponse.results.length.to_s
 	p 'Results: ' + getResponse.results.to_s
-
+  raise 'Failure retrieving data extension columns' unless getResponse.success?
+  raise 'Failure retrieving correct number of data extension columns' unless getResponse.results.count == 3
 
 	# Add a row to a data extension (using CustomerKey)
 	p '>>>  Add a row to a data extension'
@@ -79,6 +81,7 @@ begin
 	p 'Code: ' + postResponse.code.to_s
 	p 'Message: ' + postResponse.message.to_s
 	p 'Results: ' + postResponse.results.inspect
+  raise 'Failure creating data extension row' unless postResponse.success?
 
 	# Add a row to a data extension (Using Name)
 	p '>>> Add a row to a data extension'
@@ -91,6 +94,7 @@ begin
 	p 'Code: ' + postResponse.code.to_s
 	p 'Message: ' + postResponse.message.to_s
 	p 'Results: ' + postResponse.results.inspect
+  raise 'Failure creating data extension row' unless postResponse.success?
 
 	# Retrieve all rows
 	p '>>> Retrieve all rows'
@@ -102,10 +106,12 @@ begin
 	p 'Retrieve Status: ' + getResponse.status.to_s
 	p 'Code: ' + getResponse.code.to_s
 	p 'Message: ' + getResponse.message.to_s
-	p 'MoreResults: ' + getResponse.moreResults.to_s
+	p 'MoreResults: ' + getResponse.more?.to_s
 	p 'RequestID: ' + getResponse.request_id.to_s
 	p 'Results Length: ' + getResponse.results.length.to_s
 	p 'Results: ' + getResponse.results.to_s
+  raise 'Failure retrieving data extension rows' unless getResponse.success?
+  raise 'Failure retrieving correct number of data extension rows' unless getResponse.results.count == 2
 
 	# Update a row in  a data extension
 	p '>>> Update a row in  a data extension'
@@ -113,11 +119,12 @@ begin
 	de4.authStub = stubObj
 	de4.CustomerKey = NameOfDE
 	de4.props = {"Name" => "MAC3", "OtherField" => "UPDATED!"}
-	postResponse = de4.patch
-	p 'Patch Status: ' + postResponse.status.to_s
-	p 'Code: ' + postResponse.code.to_s
-	p 'Message: ' + postResponse.message.to_s
-	p 'Results: ' + postResponse.results.inspect
+	patchResponse = de4.patch
+	p 'Patch Status: ' + patchResponse.status.to_s
+	p 'Code: ' + patchResponse.code.to_s
+	p 'Message: ' + patchResponse.message.to_s
+	p 'Results: ' + patchResponse.results.inspect
+  raise 'Failure updating data extension row' unless patchResponse.success?
 
 	# Retrieve only updated row
 	p '>>> Retrieve only updated row'
@@ -130,10 +137,11 @@ begin
 	p 'Retrieve Status: ' + getResponse.status.to_s
 	p 'Code: ' + getResponse.code.to_s
 	p 'Message: ' + getResponse.message.to_s
-	p 'MoreResults: ' + getResponse.moreResults.to_s
+	p 'MoreResults: ' + getResponse.more?.to_s
 	p 'RequestID: ' + getResponse.request_id.to_s
 	p 'Results Length: ' + getResponse.results.length.to_s
 	p 'Results: ' + getResponse.results.to_s
+  raise 'Failure retrieving data extension rows' unless getResponse.success?
 
 	# Delete a row from a data extension
 	p '>>> Delete a row from a data extension'
@@ -146,6 +154,7 @@ begin
 	p 'Code: ' + deleteResponse.code.to_s
 	p 'Message: ' + deleteResponse.message.to_s
 	p 'Results: ' + deleteResponse.results.inspect
+  raise 'Failure deleting data extension row' unless deleteResponse.success?
 
 	# Delete a Data Extension
 	p '>>> Delete a  Data Extension'
@@ -157,10 +166,11 @@ begin
 	p 'Code: ' + delResponse.code.to_s
 	p 'Message: ' + delResponse.message.to_s
 	p 'Results: ' + delResponse.results.inspect
+  raise 'Failure deleting data extension' unless deleteResponse.success?
 
 =begin
-	# Retrieve lots of rows with moreResults
-	p '>>> Retrieve lots of rows with moreResults'
+	# Retrieve lots of rows with more?
+	p '>>> Retrieve lots of rows with more?'
 	row = ET_DataExtension::Row.new()
 	row.authStub = stubObj
 	row.Name = "zipstolong"
@@ -169,18 +179,19 @@ begin
 	p 'Retrieve Status: ' + getResponse.status.to_s
 	p 'Code: ' + getResponse.code.to_s
 	p 'Message: ' + getResponse.message.to_s
-	p 'MoreResults: ' + getResponse.moreResults.to_s
+	p 'MoreResults: ' + getResponse.more?.to_s
 	p 'RequestID: ' + getResponse.request_id.to_s
 	p 'Results Length: ' + getResponse.results.length.to_s
 	#p 'Results: ' + getResponse.results.to_s
+  raise 'Failure retrieving data extension rows' unless getResponse.success?
 
-	while getResponse.moreResults do
-		p '>>> Continue Retrieve lots of rows with moreResults'
+	while getResponse.more? do
+		p '>>> Continue Retrieve lots of rows with more?'
 		getResponse = row.getMoreResults
 		p 'Retrieve Status: ' + getResponse.status.to_s
 		p 'Code: ' + getResponse.code.to_s
 		p 'Message: ' + getResponse.message.to_s
-		p 'MoreResults: ' + getResponse.moreResults.to_s
+		p 'MoreResults: ' + getResponse.more?.to_s
 		p 'RequestID: ' + getResponse.request_id.to_s
 		p 'Results Length: ' + getResponse.results.length.to_s
 	end
@@ -189,4 +200,10 @@ begin
 rescue => e
   p "Caught exception: #{e.message}"
   p e.backtrace
+
+  de5 = ET_DataExtension.new
+  de5.authStub = stubObj
+  de5.props = {"Name" => NameOfDE,"CustomerKey" => NameOfDE}
+  de5.delete
+
 end

data/spec/objects_helper_spec.rb

@@ -1,25 +1,32 @@
 
+# Everything will be readable so test for shared from Read behavior
 shared_examples_for 'Soap Read Object' do
-  it { should respond_to(:info) }
-  it { should respond_to(:get) }
+  # begin backwards compat
+  it { should respond_to :props= }
+  it { should respond_to :authStub= }
+  # end
+  it { should respond_to :id }
+  it { should respond_to :properties }
+  it { should respond_to :client }
+  it { should respond_to :filter }
+  it { should respond_to :info }
+  it { should respond_to :get }
 end
 
 shared_examples_for 'Soap CUD Object' do
-  it { should respond_to(:post) }
-  it { should respond_to(:patch) }
-  it { should respond_to(:delete) }
+  it { should respond_to :post }
+  it { should respond_to :patch }
+  it { should respond_to :delete }
 end
 
 shared_examples_for 'Soap Object' do
-  it { should respond_to(:id) }
   it_behaves_like 'Soap Read Object'
   it_behaves_like 'Soap CUD Object'
 end
 
 shared_examples_for 'Soap Read Only Object' do
-  it { should respond_to(:id) }
   it_behaves_like 'Soap Read Object'
-  it { should_not respond_to(:post) }
-  it { should_not respond_to(:patch) }
-  it { should_not respond_to(:delete) }
+  it { should_not respond_to :post }
+  it { should_not respond_to :patch }
+  it { should_not respond_to :delete }
 end

data/spec/objects_spec.rb

@@ -1,6 +1,29 @@
 require 'spec_helper.rb'
 require 'objects_helper_spec.rb'
 
+describe FuelSDK::Objects::Base do
+
+  let(:object) { FuelSDK::Objects::Base.new }
+  subject{ object }
+
+  describe '#properties' do
+    it 'is empty by default' do
+      expect(object.properties).to be_empty
+    end
+
+    it 'returns item in array when item is not an array' do
+      object.properties = {'name' => 'value'}
+      expect(object.properties).to eq([{'name' => 'value'}])
+    end
+
+    it 'returns array when assigned array' do
+      object.properties = [{'name' => 'value'}]
+      expect(object.properties).to eq([{'name' => 'value'}])
+    end
+  end
+
+end
+
 describe FuelSDK::BounceEvent do
 
   let(:object) { FuelSDK::BounceEvent.new }
@@ -100,6 +123,354 @@ describe FuelSDK::Subscriber do
   its(:id){ should eq 'Subscriber' }
 end
 
+describe FuelSDK::DataExtension::Column do
+
+  let(:object) { FuelSDK::DataExtension::Column.new }
+  subject{ object }
+
+  it_behaves_like 'Soap Read Only Object'
+  its(:id){ should eq 'DataExtensionField' }
+end
+
+describe FuelSDK::DataExtension do
+  let(:object) { FuelSDK::DataExtension.new }
+  subject{ object }
+
+  it_behaves_like 'Soap Object'
+  its(:id){ should eq 'DataExtension' }
+  it { should respond_to :columns= }
+  it { should respond_to :fields }
+  it { should respond_to :fields= }
+
+  describe '#post' do
+    subject {
+      object.stub_chain(:client, :soap_post) do |id, properties|
+        [id, properties]
+      end
+
+      object
+    }
+
+    # maybe one day will make it smart enough to zip properties and fields if count is same?
+    it 'raises an error when it has a list of properties and fields' do
+      subject.fields = [{'Name' => 'Name'}]
+      subject.properties = [{'Name' => 'Some DE'}, {'Name' => 'Some DE'}]
+      expect{subject.post}.to raise_error(
+        'Unable to handle muliple DataExtension definitions and a field definition')
+    end
+
+    it 'fields must be empty if not nil' do
+      subject.fields = []
+      subject.properties = [{'Name' => 'Some DE', 'fields' => [{'Name' => 'A field'}]}]
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+
+    it 'DataExtension can be created using properties and fields accessors' do
+      subject.fields = [{'Name' => 'A field'}]
+      subject.properties = {'Name' => 'Some DE'}
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+
+    it 'DataExtension fields can be apart of the DataExtention properties' do
+      subject.properties = {'Name' => 'Some DE', 'Fields' => {'Field' => [{'Name' => 'A field'}]}}
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+
+    it 'List of DataExtension definitions can be passed' do
+      subject.properties = [{'Name' => 'Some DE', 'Fields' => {'Field' => [{'Name' => 'A field'}]}},
+        {'Name' => 'Another DE', 'Fields' => {'Field' => [{'Name' => 'A second field'}]}}]
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          },{
+            'Name' => 'Another DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A second field'}]
+            }
+          }]
+        ])
+    end
+
+    it 'DataExtension definitions will translate fields entry to correct format' do
+      subject.properties = {'Name' => 'Some DE', 'fields' => [{'Name' => 'A field'}]}
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+
+    it 'DataExtension definitions will translate columns entry to correct format' do
+      subject.properties = {'Name' => 'Some DE', 'columns' => [{'Name' => 'A field'}]}
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+
+    it 'supports columns attribute for a single DataExtension definition' do
+      subject.columns = [{'Name' => 'A field'}]
+      subject.properties = {'Name' => 'Some DE'}
+      expect(subject.post).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+
+    describe 'fields are defined twice' do
+      it 'when defined in properties and by fields' do
+        subject.fields = [{'Name' => 'A field'}]
+        subject.properties = {'Name' => 'Some DE', 'Fields' => {'Field' => [{'Name' => 'A field'}]}}
+        expect{subject.post}.to raise_error 'Fields are defined in too many ways. Please only define once.'
+      end
+      it 'when defined in properties explicitly and with columns key' do
+        subject.properties = {'Name' => 'Some DE',
+          'columns' => [{'Name' => 'A fields'}],
+          'Fields' => {'Field' => [{'Name' => 'A field'}]
+        }}
+        expect{subject.post}.to raise_error 'Fields are defined in too many ways. Please only define once.'
+      end
+      it 'when defined in properties explicitly and with fields key' do
+        subject.properties = {'Name' => 'Some DE',
+          'fields' => [{'Name' => 'A fields'}],
+          'Fields' => {'Field' => [{'Name' => 'A field'}]
+        }}
+        expect{subject.post}.to raise_error 'Fields are defined in too many ways. Please only define once.'
+      end
+      it 'when defined in with fields and colums key' do
+        subject.properties = {'Name' => 'Some DE',
+          'fields' => [{'Name' => 'A fields'}],
+          'columns' => [{'Name' => 'A field'}]
+        }
+        expect{subject.post}.to raise_error 'Fields are defined in too many ways. Please only define once.'
+      end
+      it 'when defined in with fields key and accessor' do
+        subject.fields = [{'Name' => 'A field'}]
+        subject.properties = {'Name' => 'Some DE',
+          'fields' => [{'Name' => 'A fields'}]
+        }
+        expect{subject.post}.to raise_error 'Fields are defined in too many ways. Please only define once.'
+      end
+    end
+  end
+
+  describe '#patch' do
+    subject {
+      object.stub_chain(:client, :soap_patch) do |id, properties|
+        [id, properties]
+      end
+
+      object
+    }
+
+    it 'DataExtension can be created using properties and fields accessors' do
+      subject.fields = [{'Name' => 'A field'}]
+      subject.properties = {'Name' => 'Some DE'}
+      expect(subject.patch).to eq(
+        [
+          'DataExtension',
+          [{
+            'Name' => 'Some DE',
+            'Fields' => {
+              'Field' => [{'Name' => 'A field'}]
+            }
+          }]
+        ])
+    end
+  end
+end
+
+describe FuelSDK::DataExtension::Row do
+  let(:object) { FuelSDK::DataExtension::Row.new }
+  subject{ object }
+
+  it_behaves_like 'Soap Object'
+  its(:id){ should eq 'DataExtensionObject' }
+  it { should respond_to :name }
+  it { should respond_to :name= }
+  it { should respond_to :customer_key }
+  it { should respond_to :customer_key= }
+
+  describe '#name' do
+    it 'raises error when missing both name and customer key' do
+      expect{ subject.name }.to raise_error('Unable to process DataExtension::Row '\
+        'request due to missing CustomerKey and Name')
+    end
+
+    it 'returns value' do
+      subject.name = 'name'
+      expect( subject.name ).to eq 'name'
+    end
+  end
+
+  describe '#customer_key' do
+    it 'raises error when missing both name and customer key' do
+      expect{ subject.customer_key }.to raise_error('Unable to process DataExtension::Row '\
+        'request due to missing CustomerKey and Name')
+    end
+
+    it 'returns value' do
+      subject.customer_key = 'key'
+      expect( subject.customer_key ).to eq 'key'
+    end
+  end
+
+  describe '#retrieve_required' do
+    it 'raises error when missing both name and customer key' do
+      expect{ subject.send(:retrieve_required)}.to raise_error('Unable to process DataExtension::Row '\
+        'request due to missing CustomerKey and Name')
+      expect{ subject.name }.to raise_error('Unable to process DataExtension::Row '\
+        'request due to missing CustomerKey and Name')
+    end
+
+    it 'updates missing' do
+      rsp = mock(FuelSDK::SoapResponse)
+      rsp.stub(:results).and_return([{:name => 'Products', :customer_key => 'ProductsKey'}])
+      rsp.stub(:success?).and_return true
+
+      subject.stub_chain(:client,:soap_get).and_return(rsp)
+      subject.name = 'Not Nil'
+
+      # this really wouldn't work this way. name shouldn't be updated since its whats being used for filter,
+      # but its a good test to show retrieve_required being fired
+      expect(subject.name).to eq 'Not Nil' # not fired
+      expect(subject.customer_key).to eq 'ProductsKey' # fired... stubbed get returns customer_key and name for update
+      expect(subject.name).to eq 'Products' # returned name
+    end
+  end
+
+  describe '#get' do
+    subject {
+      object.stub_chain(:client, :soap_get) do |id, properties, filter|
+        [id, properties, filter]
+      end
+
+      object
+    }
+
+    it 'passes id including name to super get' do
+      subject.name = 'Justin'
+      expect(subject.get).to eq(['DataExtensionObject[Justin]', [], nil])
+    end
+  end
+
+  describe '#post' do
+    subject {
+      object.stub_chain(:client, :soap_post) do |id, properties|
+        [id, properties]
+      end
+
+      object
+    }
+
+    it 'raises an error when missing both name and customer key' do
+      subject.properties = [{'Name' => 'Some DE'}, {'Name' => 'Some DE'}]
+      expect{subject.post}.to raise_error('Unable to process DataExtension::Row ' \
+        'request due to missing CustomerKey and Name')
+    end
+
+    it 'uses explicitly defined properties' do
+      subject.properties = [{'CustomerKey' => 'Subscribers',
+        'Properties' => {'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+      expect(subject.post).to eq([
+        'DataExtensionObject', [{
+          'CustomerKey' => 'Subscribers',
+          'Properties' => {'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+      ])
+    end
+
+    it 'inserts customer key into properties when set using accessor' do
+      subject.customer_key = 'Subscribers'
+      subject.properties = [{'Properties' => {
+        'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+      expect(subject.post).to eq([
+        'DataExtensionObject', [{
+          'CustomerKey' => 'Subscribers',
+          'Properties' => {'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+      ])
+    end
+
+    it 'uses name to get customer key for inseration' do
+      subject.name = 'Subscribers'
+
+      rsp = mock(FuelSDK::SoapResponse)
+      rsp.stub(:results).and_return([{:name => 'Products', :customer_key => 'ProductsKey'}])
+      rsp.stub(:success?).and_return true
+
+      subject.stub_chain(:client, :soap_get).and_return(rsp)
+      subject.properties = [{'Properties' => {
+        'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+
+      expect(subject.post).to eq([
+        'DataExtensionObject', [{
+          'CustomerKey' => 'ProductsKey',
+          'Properties' => {'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+      ])
+    end
+
+    it 'correctly formats array property' do
+      subject.customer_key = 'Subscribers'
+
+      subject.properties = [{'Name' => 'Justin'}]
+
+      expect(subject.post).to eq([
+        'DataExtensionObject', [{
+          'CustomerKey' => 'Subscribers',
+          'Properties' => {'Property' => [{'Name' => 'Name', 'Value' => 'Justin'}]}}]
+      ])
+    end
+  end
+end
+
 # verify backward compats
 describe ET_Subscriber do