spooky-oysters / Bangazon-Command-Line-Application

0 stars 1 forks source link

User should be able to update a product #20

Closed kimberly-bird closed 6 years ago

kimberly-bird commented 6 years ago

Link to Ticket

8 User should be able to update a product

Description of Proposed Changes

Steps to Test

Outline the steps to test

git fetch --all
git checkout kb_updateProduct
..
cd bangazon-cli.Tests/
dotnet restore
dotnet test

Impacted Areas in Application

List general components of the application that this PR will affect:

Mentions @username

Tag users that need to review this code

Definition of Done

  1. The project must be fully documented. This includes the following:
    1. Complete README that documents the steps to install the code, how to install any dependencies, or system configuration needed.
    2. Every class must be documented with purpose, author, and methods.
    3. Every method must be documented with purpose and argument list - which itself must contain a short purpose for each argument.
  2. The project must be able to run fully, and without errors, on each teammate's system.
  3. Fulfills every requirement. Every line of code has been peer reviewed.
  4. For projects that require unit testing, core functionality must be identified and have at least one test for each.
krysmathis commented 6 years ago

In ProductManager_Should tests, remove the code which returns the full list in the tests that update the product.

kimberly-bird commented 6 years ago

@krysmathis - code that returned full list of product have been removed from tests

krysmathis commented 6 years ago

👍

Chewieez commented 6 years ago

👍