Skip to content

feat(cucumberPutUser): add cucumber scenario for put user endpoint#81

Merged
MayuriXx merged 1 commit into
mainfrom
feat/cucumberPutUser
Apr 29, 2026
Merged

feat(cucumberPutUser): add cucumber scenario for put user endpoint#81
MayuriXx merged 1 commit into
mainfrom
feat/cucumberPutUser

Conversation

@MayuriXx

@MayuriXx MayuriXx commented Apr 27, 2026

Copy link
Copy Markdown
Collaborator

#54

Copilot AI review requested due to automatic review settings April 27, 2026 13:18
@MayuriXx MayuriXx linked an issue Apr 27, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds Cucumber integration coverage for the PUT /random-users/{id} endpoint, extending the existing Cucumber-based test suite under src/test/resources/features and src/test/java/feature.

Changes:

  • Adds a new Cucumber feature file covering successful update-after-create and 404-on-missing-user scenarios.
  • Extends Cucumber step definitions with update payload setup and PUT invocation steps.
  • Adds a shared executePut helper to the Spring+Cucumber integration test base.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/test/resources/features/update_user.feature New feature scenarios validating PUT update behavior (200) and not-found behavior (404).
src/test/java/feature/StepDefinition.java Adds step definitions for update payload creation and PUT calls (created user + explicit id).
src/test/java/feature/SpringIntegrationTest.java Adds executePut helper method using TestRestTemplate.exchange(...) with JSON body.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MayuriXx MayuriXx force-pushed the feat/cucumberPutUser branch from a23b2fa to 2a97fbc Compare April 29, 2026 06:53
@sonarqube-xpeho

Copy link
Copy Markdown

@Theo-lbg Theo-lbg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@MayuriXx MayuriXx requested a review from profotoce59 April 29, 2026 07:16
@MayuriXx MayuriXx merged commit f8bf26b into main Apr 29, 2026
13 checks passed
@MayuriXx MayuriXx deleted the feat/cucumberPutUser branch April 29, 2026 07:41
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.

Create a cucumber scenario to test the put user endpoint

4 participants