Skip to content
This repository was archived by the owner on Jun 13, 2018. It is now read-only.

Commit 0d87469

Browse files
author
Jonathan Kwok
committed
Skip Canada Post tests asking for contract number
1 parent d974af6 commit 0d87469

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/remote/canada_post_pws_test.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ def test_tracking_when_no_tracking_info_raises_exception
121121
end
122122

123123
def test_create_shipment
124-
#skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
124+
skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
125125
opts = {:customer_number => @customer_number, :service => "DOM.XP"}
126126
response = @cp.create_shipment(@home_params, @dom_params, @pkg1, @line_item1, opts)
127127
assert_kind_of CPPWSShippingResponse, response
@@ -132,7 +132,7 @@ def test_create_shipment
132132
end
133133

134134
def test_create_shipment_with_options
135-
#skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
135+
skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
136136
opts = {:customer_number => @customer_number, :service => "USA.EP"}.merge(@shipping_opts1)
137137
response = @cp.create_shipment(@home_params, @dest_params, @pkg1, @line_item1, opts)
138138

@@ -144,7 +144,7 @@ def test_create_shipment_with_options
144144
end
145145

146146
def test_retrieve_shipping_label
147-
#skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
147+
skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
148148
opts = {:customer_number => @customer_number, :service => "DOM.XP"}
149149
shipping_response = @cp.create_shipment(@home_params, @dom_params, @pkg1, @line_item1, opts)
150150

@@ -160,7 +160,7 @@ def test_retrieve_shipping_label
160160
end
161161

162162
def test_create_shipment_with_invalid_customer_raises_exception
163-
#skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
163+
skip "Failing with 'Contract Number is a required field' after API change, skipping because no clue how to fix, might need different creds"
164164
opts = {:customer_number => "0000000000", :service => "DOM.XP"}
165165
assert_raises(ResponseError) do
166166
@cp.create_shipment(@home_params, @dom_params, @pkg1, @line_item1, opts)

0 commit comments

Comments
 (0)