Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

As a user I should *not* be able to remove more items from inventory than are currently in stock. #1

Open
serra opened this issue Jul 25, 2012 · 0 comments
Open

As a user I should *not* be able to remove more items from inventory than are currently in stock. #1

serra opened this issue Jul 25, 2012 · 0 comments
Assignees
@serra

Comments

@serra
Copy link
Member

serra commented Jul 25, 2012

No description provided.

serra added a commit that referenced this issue Jul 25, 2012
@serra
#1 - Do not allow removing more items than are currently in stock
76a93d3 
Introduces `_count` member to keep track of current count.
Do not implement business logic in `Apply(event)` methods: those should never fail.
Added simple guard to `void Remove()`.
@ghost ghost assigned serra Jul 25, 2012
@serra serra closed this as completed Jul 26, 2012
@serra serra reopened this Jul 27, 2012
serra added a commit that referenced this issue Jul 27, 2012
@serra
#1 - Add test for removing more items than are in stock
a94b3d2
serra added a commit that referenced this issue Jul 27, 2012
@serra
#1 - Add test for removing more items than are in stock
3791a03
serra added a commit that referenced this issue Jul 27, 2012
@serra
#1 - Add test for removing more items than are in stock
8b9063c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@serra serra

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@serra