Skip to content
This repository was archived by the owner on Sep 19, 2019. It is now read-only.

Comments

Add test for customer return items quantity#9

Open
aleph1ow wants to merge 1 commit intomasterfrom
test-items-array
Open

Add test for customer return items quantity#9
aleph1ow wants to merge 1 commit intomasterfrom
test-items-array

Conversation

@aleph1ow
Copy link
Member

@aleph1ow aleph1ow commented Jul 6, 2017

Tests that a shipwire return can contain multiple items

@aleph1ow aleph1ow requested review from kennyadsl and mtylty July 6, 2017 13:56
Copy link
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you also add a small description? I know that there's not a lot to say but you could link other customer return issues/PR so they are somehow connected.

@@ -1,3 +1,5 @@
require 'byebug'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove this require

expect(customer_return.shipwire_id).not_to be nil
end

it 'place a single item in the return' do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

places ?

expect(customer_return.shipwire_id).not_to be nil
end

it 'place multiple items in the return' do
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

places?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants