Skip to content

Test(refactor): clean user created after each scenario#109

Merged
saw-jan merged 1 commit intomainfrom
test/cleanup-user-details
Sep 16, 2025
Merged

Test(refactor): clean user created after each scenario#109
saw-jan merged 1 commit intomainfrom
test/cleanup-user-details

Conversation

@anon-pradip
Copy link
Contributor

@anon-pradip anon-pradip commented Aug 27, 2025

Description

The test suite was encountering 404 status code when running multiple test scenarios due to the userDetails array not being reset after cleanup. This caused subsequent test runs to attempt cleaning up already-deleted resources.

Before: When running multiple test scenarios, the cleanup process would try to delete the same resources multiple times, causing "Resource Not Found" errors

After: Each test scenario now starts with a clean state, preventing duplicate cleanup attempts and making the tests more reliable

Related Issue

Motivation and Context

Run multiple tests in future smoothly

How Has This Been Tested?

  • test environment:
  • locally and CI

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Tests only (no source changes)
  • Documentation only (no source changes)
  • Maintenance (e.g. dependency updates or tooling)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Documentation updated

@anon-pradip anon-pradip self-assigned this Aug 27, 2025
@anon-pradip anon-pradip added the QA label Aug 27, 2025
@anon-pradip anon-pradip force-pushed the test/cleanup-user-details branch from 34cfc69 to f8e6d6f Compare September 15, 2025 07:10
@anon-pradip anon-pradip marked this pull request as ready for review September 15, 2025 07:14
@anon-pradip anon-pradip force-pushed the test/cleanup-user-details branch from f8e6d6f to b097642 Compare September 16, 2025 04:12
@anon-pradip anon-pradip requested a review from saw-jan September 16, 2025 04:13
@saw-jan saw-jan merged commit 69f7920 into main Sep 16, 2025
2 checks passed
@saw-jan saw-jan deleted the test/cleanup-user-details branch September 16, 2025 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants