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.

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

Test for Update Inventory #1042

Closed
wants to merge 5 commits into from
Closed

Conversation

Carlos-123321
Copy link
Collaborator

@Carlos-123321 Carlos-123321 commented Nov 11, 2024

JIRA: link to jira ticket

Context:

Whenever an inventory is successfully updated there should be a message showing that the inventory was successfully updated.

Does this PR change the .vscode folder in petclinic-frontend?:

If the PR changes the .vscode folder, explain why in detail because it should not.
Be sure to include cgerard321 as a reviewer.

Reviewers need to check for any changes to the
.vscode folder and add a comment about it to their review comments.

Changes

What are the various changes and what other modules do those changes affect.
This can be bullet point or sentence format.

Before and After UI (Required for UI-impacting PRs)

If this is a change to the UI, include before and after screenshots to show the differences.
If this is a new UI feature, include screenshots to show reviewers what it looks like.

Dev notes (Optional)

Specific technical changes that should be noted

Linked pull requests (Optional)

Pull request links

@Carlos-123321 Carlos-123321 changed the title Test Signup page (#1035) Test for Update Inventory Nov 11, 2024
Copy link
Collaborator

@Sunveerg Sunveerg left a comment

Choose a reason for hiding this comment

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

Tests look good

flyindonut and others added 4 commits November 11, 2024 17:40
## Context:
Made an e2e test for edit a product name
## Does this PR change the .vscode folder in petclinic-frontend?:
If the PR changes the .vscode folder, explain why in detail because it
should not.
it doesn't
## Context:
Made an e2e test to add vet education and delete vet education  
## Does this PR change the .vscode folder in petclinic-frontend?:
No

<span style="color:red">**Reviewers need to check for any changes to the
.vscode folder and add a comment about it to their review
comments.**</span>

## Changes
Added e2e test files for adding and deleteing vet education
@Carlos-123321 Carlos-123321 deleted the branch test/INVT/CPC/UpdateInventory November 11, 2024 23:37
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.

7 participants