Skip to content

Improve Test Coverage for GildedRose UpdateQuality Functionality #48

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.

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

justin-marian
Copy link

  • Add comprehensive tests to cover all branches and code paths in the updateQuality method.
  • Add tests for Normal items, Aged Brie, Sulfuras, Backstage passes, and Conjured items to ensure both quality and sellIn are correctly updated, achieving full coverage.
  • Attach picture with results:
    image

Added extra items to the inventory for testing purposes. This change ensures that all item types (Normal, Aged Brie, Sulfuras, Backstage passes, and Conjured) are present and can be validated by the test suite.
Added test cases to cover all branches of the updateQuality method. The tests now verify that the quality and sellIn values are updated correctly for each item type, ensuring 100% branch coverage.
Added a variety of items to the inventory (including normal items, Aged Brie, Backstage passes, and Conjured items) to ensure comprehensive coverage of the updateQuality functionality. This update helps verify that all branches in the code are exercised and the program behaves correctly.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant