DSingh0304 opened a new pull request, #3253:
URL: https://github.com/apache/apisix-dashboard/pull/3253
**Why submit this pull request?**
- [ ] Bugfix
- [x] New feature provided
- [ ] Improve performance
- [ ] Backport patches
**What changes will this PR take into?**
This PR adds comprehensive E2E tests for the Services resource, covering all
CRUD operations and list page functionality.
**Test Coverage (5 tests, all passing ✅):**
- **List page tests** (`services.list.spec.ts`) - 3 tests
- Navigation to services page
- Pagination using table controls
- Pagination using URL search parameters
- **CRUD with required fields** (`services.crud-required-fields.spec.ts`) -
1 test
- Uses minimal fields (name only)
- Tests create, read, update, and delete operations
- Verifies label addition during edit
- **CRUD with all fields** (`services.crud-all-fields.spec.ts`) - 1 test
- Uses all available fields (name, description, labels, upstream nodes,
hosts, WebSocket)
- Tests complete create and delete operations with complex configurations
**Files Added:**
- `e2e/utils/ui/services.ts` - UI helper functions for form filling and
verification with proper selector disambiguation
- `e2e/tests/services.list.spec.ts` - List page and pagination tests
- `e2e/tests/services.crud-required-fields.spec.ts` - Minimal fields CRUD
tests
- `e2e/tests/services.crud-all-fields.spec.ts` - All fields CRUD tests
**Implementation Details:**
- Proper selector disambiguation using `.first()` to avoid conflicts between
service and upstream fields (name, description, labels)
- Handles hidden WebSocket switch using JavaScript evaluation
- Upstream node configuration (host, port, weight) properly filled and
verified
- Follows test patterns from upstreams and routes
- Proper cleanup with `deleteAllServices` helper
- Auto-generated ID capture and verification
**Related issues**
resolve #3085
**Checklist:**
- [x] Did you explain what problem does this PR solve? Or what new features
have been added?
- [x] Have you added corresponding test cases?
- [ ] Have you modified the corresponding document? (N/A - E2E tests don't
require documentation updates)
- [x] Is this PR backward compatible? If it is not backward compatible,
please discuss on the mailing list first (Yes, only adds tests, no breaking
changes)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
