Re: [PR] UI: Add comprehensive domain deletion confirmation dialog (Feature Request #11497) [cloudstack]

[via GitHub]

Imvedansh commented on PR #12380:
URL: https://github.com/apache/cloudstack/pull/12380#issuecomment-3816579705

   > ## Testing Summary
   > ### Overall Assessment **LGTM** (with minor observations)
   > Great work on this PR, @Imvedansh! The domain deletion confirmation dialog 
is a significant UX improvement. The implementation is solid and handles the 
key scenarios well.
   > 
   > ### Test Results
   > Test Case  Description     Result
   > TC1        Delete empty domain     **PASS**
   > TC2        Delete domain with account (no VMs)     **PASS**
   > TC3        Delete domain with mixed VM states      **PASS**
   > TC4        Cascading deletion (parent with children)       **PASS**
   > TC5        Mass delete (rapid succession)  **PASS**
   > TC6        Whitespace handling + navigation        **PARTIAL**
   > ### What Works Well
   > 1. **Modal displays accurate information** - Account/VM breakdown is 
correct and helpful
   > 2. **Input validation** - Exact domain name match required; child domain 
names correctly rejected
   > 3. **Whitespace handling** - `.trim()` correctly handles leading/trailing 
spaces
   > 4. **Async job handling** - `$pollJob` implementation works correctly
   > 5. **Cascading deletion** - `cleanup=true` successfully removes child 
domains, accounts, VMs, and networks
   > 6. **Mass deletion** - Multiple rapid deletions queue and complete without 
conflicts
   > 7. **Dynamic UI updates** - Domain tree refreshes correctly after each 
deletion
   > 
   > ### Observations & Suggestions
   > #### 1. Navigation Issue (TC6)
   > When deleting a domain **while viewing it**, the UI does not navigate 
away. The detail panel shows stale data for the deleted domain.
   > 
   > * **Current behavior:** Domain tree updates, but detail panel shows 
deleted domain
   > * **Expected behavior:** Should navigate to `/domain` after deletion
   > * **Workaround:** Hard browser refresh (F5) fixes it
   > 
   > The code exists but doesn't seem to execute:
   > 
   > ```js
   > if (this.$route.params.id === domain.id) {
   >   this.$router.push({ path: '/domain' })
   > }
   > ```
   > 
   > #### 2. Warning Message Verbiage (Minor)
   > Current text: _"All of these **must be removed** before the domain can be 
deleted."_
   > 
   > Since `cleanup=true` is always passed, the resources **are removed 
automatically**. Consider updating to something like:
   > 
   > > _"All associated accounts, users, VMs, and sub-domains **will be 
permanently deleted**. This action cannot be undone."_
   > 
   > This better reflects the actual behavior and addresses the concern raised 
by @shwstppr about accidental deletion.
   > 
   > ## Detailed Test Execution Report
   > ### TC1: Delete Empty Domain
   > **Objective:** Verify that an empty domain (no accounts, no VMs, no 
sub-domains) can be deleted using the new confirmation dialog.
   > 
   > **Test Steps:**
   > 
   > 1. Login to CloudStack UI as admin
   > 2. Navigate to Domains section
   > 3. Expand ROOT → TestDeleteParent → Click on TestChild3
   > 4. Click Delete Domain action button (trash icon) & Observe the 
confirmation modal displays
   > 5. Verify the Account/VM table shows "No Data"
   > 6. Verify "Delete domain" button is disabled
   > 7. Type `TestChild3` in the confirmation input field (try with 
whitespaces, etc)
   > 8. Verify "Delete domain" button becomes enabled only on exact match (or 
exact match with whitespaces at the beginnin/end)
   > 9. Click "Delete domain" button
   > 10. Verify domain is removed from the tree
   > 
   > **Expected Result:**
   > 
   > * Modal displays with warning message and empty table
   > * Delete button disabled until correct domain name is entered
   > * Domain deletes successfully and is removed from UI
   > 
   > **Actual Result:**
   > 
   > * Modal displayed correctly with warning: "This domain may contain 
accounts, users, or sub-domains. All of these must be removed before the domain 
can be deleted. This action cannot be undone."
   > * Table showed "No Data" (empty domain)
   > * Delete button was disabled until domain name was typed
   > * Domain deleted successfully and removed from tree dynamically
   > * **Note:** No success notification displayed after deletion
   > 
   > **Test Evidence:**
   > 
   > * Pre-test: TestChild3 existed (ID: 8a64ede9-b598-4be3-b9b9-e6c01b548443)
   > * Post-deletion verification:
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 6,
   >   "domain": [
   >     { "name": "ROOT", ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... },
   >     { "name": "TestDeleteParent", ... },
   >     { "name": "TestChild1", "vmtotal": 2, ... },
   >     { "name": "TestChild2", "vmtotal": 0, ... }
   >   ]
   > }
   > ```
   > 
   > * TestChild3 no longer present - deletion confirmed
   > 
   > **Status:** PASS
   > 
   > * domain deletion modal
   > 
   > <img alt="Screenshot from 2026-01-27 16-44-19" width="2000" height="1673" 
src="https://private-user-images.githubusercontent.com/201906357/541102936-935c5ddc-456a-4aba-9ae4-ed7ae6371a5f.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Njk2ODAyODksIm5iZiI6MTc2OTY3OTk4OSwicGF0aCI6Ii8yMDE5MDYzNTcvNTQxMTAyOTM2LTkzNWM1ZGRjLTQ1NmEtNGFiYS05YWU0LWVkN2FlNjM3MWE1Zi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwMTI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDEyOVQwOTQ2MjlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0xMzVmZDZhZDJlMGRkZDMyNjY1YzU5ZTkzZTdlNDEwOTY5NWFhY2EzMTZhYzY3NDdhMmQzNjg0ZmRjZjFiZTgwJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.eEmJWo4_M679W4L3G06RRyTTfmnfDA9ZP6yArCIpoTQ";>
   > * domain is deleted and dynamically removed from domains list (left side)
   >   [Screencast from 2026-01-27 
16-44-44.webm](https://github.com/user-attachments/assets/f08e37f1-b08b-455b-8187-053110c2750f)
   > 
   > ### TC2: Delete Domain with Account, No VMs + Input Validation
   > **Objective:** Verify that a domain containing an account (but no VMs) 
displays correctly in the confirmation modal, validates input correctly, and 
deletes successfully with cleanup=true.
   > 
   > **Test Steps:**
   > 
   > 1. Navigate to Domains section
   > 2. Click on TestChild2 (has testuser2 account, 0 VMs)
   > 3. Click Delete Domain action button
   > 4. Observe modal displays account in table with VM counts
   > 5. Verify "Delete domain" button is disabled
   > 6. Type incorrect name → verify button stays disabled
   > 7. Type `TestChild2` → verify button becomes enabled
   > 8. Click "Delete domain" button
   > 9. Verify domain and account are deleted via CLI
   > 
   > **Expected Result:**
   > 
   > * Modal shows testuser2 account with 0/0/0 VM counts
   > * Input validation prevents deletion until correct name entered
   > * Domain and associated account deleted successfully (cleanup=true)
   > 
   > **Actual Result:**
   > 
   > * Modal displayed testuser2 account correctly: Total VMs=0, Running=0, 
Stopped=0
   > * Delete button was disabled until correct domain name entered
   > * Domain deleted successfully
   > * Account testuser2 also deleted (cleanup=true worked)
   > * Domain count reduced from 6 to 5
   > 
   > <img alt="Screenshot from 2026-01-27 16-56-50" width="2000" height="1673" 
src="https://private-user-images.githubusercontent.com/201906357/541107531-88bd3f66-abdf-402b-9bf1-225199ba6804.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Njk2ODAyODksIm5iZiI6MTc2OTY3OTk4OSwicGF0aCI6Ii8yMDE5MDYzNTcvNTQxMTA3NTMxLTg4YmQzZjY2LWFiZGYtNDAyYi05YmYxLTIyNTE5OWJhNjgwNC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwMTI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDEyOVQwOTQ2MjlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1hOWJjZTg3YWJhMDBhZjJlYmZhOGU5ZmJjODJkMzc5YzMyYWYwN2I1YTdiMWVjM2VmMjY0ZTY3NDQ0MGM3OTFiJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.VWrojmO6J_VZb0cZ-PBFm3BhrxX1s5JodAVstA6gDqk";>
   >  Screencast.from.2026-01-27.16-58-53.webm 
   > * Post-deletion verification:
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 5,
   >   "domain": [
   >     { "name": "ROOT", ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... },
   >     { "name": "TestDeleteParent", "vmtotal": 2, ... },
   >     { "name": "TestChild1", "vmtotal": 2, ... }
   >   ]
   > }
   > ```
   > 
   > * TestChild2 no longer present
   > 
   > ```
   > (admin) 🐱 > list accounts domainid=426023be-6fcd-4b20-ad27-b2b64ff79b30 
listall=true
   > 🙈 Error: (HTTP 431, error code 4350) Domain id=8 doesn't exist
   > # Confirms domain and account were deleted
   > ```
   > 
   > **Status:** PASS
   > 
   > ## TC3: Delete Domain with Mixed VM States (cleanup=true)
   > **Objective:** Verify that a domain containing an account with running and 
stopped VMs displays correct counts in the modal and deletes successfully with 
cleanup=true (cascading deletion of VMs, networks, and account).
   > 
   > **Test Steps:**
   > 
   > 1. Navigate to Domains section
   > 2. Click on TestChild1 (has testuser1 account with 2 VMs)
   > 3. Click Delete Domain action button
   > 4. Observe modal displays account with VM breakdown
   > 5. Verify Running/Stopped VM counts are accurate
   > 6. Type `TestChild1` in confirmation input
   > 7. Click "Delete domain" button
   > 8. Observe success notification
   > 9. Verify domain, account, and VMs deleted via CLI
   > 
   > **Expected Result:**
   > 
   > * Modal shows testuser1 with Total=2, Running=1, Stopped=1
   > * Domain, account, and all VMs deleted via cleanup=true
   > * Success notification displayed
   > 
   > **Actual Result:**
   > 
   > * Modal correctly displayed: testuser1 | 2 | 1 | 1
   > * Success notification appeared upon deletion
   > * Domain TestChild1 deleted
   > * Account testuser1 deleted
   > * Both VMs (testvm1, testvm2) deleted
   > * Network resources cleaned up
   > * Domain count reduced from 5 to 4
   > * VM count reduced from 3 to 1 (only admin's iso-test-vm remains)
   > * domain with mixed VM states - displays correctly
   > 
   > <img alt="image" width="2000" height="1673" 
src="https://private-user-images.githubusercontent.com/201906357/541108956-e1371a37-0195-4449-adeb-bf3646865729.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Njk2ODAyODksIm5iZiI6MTc2OTY3OTk4OSwicGF0aCI6Ii8yMDE5MDYzNTcvNTQxMTA4OTU2LWUxMzcxYTM3LTAxOTUtNDQ0OS1hZGViLWJmMzY0Njg2NTcyOS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwMTI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDEyOVQwOTQ2MjlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT00YTg1YmI4MDZkZmI1OTA1NDU0YjJhM2JmMjM0Mjk1YWViYTYxOWQ1Mjk0OTVmZWRiMzM5MzQzODkzNGEwZTM4JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.59loHNfoc8TZ7GeVmwo6z8CcahrD9yVuCQ4saE3eva0";>
   > * deletion
   > 
   >  Screencast.from.2026-01-27.16-58-53.webm 
   > * Post-deletion - Domains:
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 4,
   >   "domain": [
   >     { "name": "ROOT", "vmtotal": 1, ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... },
   >     { "name": "TestDeleteParent", "haschild": false, "vmtotal": 0, ... }
   >   ]
   > }
   > ```
   > 
   > * TestChild1 deleted, TestDeleteParent now has no children
   > * Post-deletion - VMs:
   > 
   > ```
   > (admin) 🐱 > list virtualmachines listall=true
   > {
   >   "count": 1,
   >   "virtualmachine": [
   >     { "name": "iso-test-vm", "account": "admin", "domain": "ROOT", ... }
   >   ]
   > }
   > # testvm1 and testvm2 deleted - only admin's VM remains
   > ```
   > 
   > **Status:** PASS
   > 
   > ## TC4: Delete Parent Domain with Children (Cascading Deletion)
   > **Objective:** Verify that deleting a parent domain with child domains 
triggers cascading deletion of all sub-domains and their associated resources 
(accounts, users) using cleanup=true.
   > 
   > **Test Steps:**
   > 
   > 1. Create test hierarchy: TestDeleteParent → CascadeChild1 (with account), 
CascadeChild2 (empty)
   > 2. Navigate to Domains section in UI
   > 3. Click on TestDeleteParent
   > 4. Click Delete Domain action button
   > 5. Observe modal shows accounts from child domains
   > 6. Try typing child domain name (e.g., "CascadeChild1") → verify button 
stays disabled
   > 7. Type `TestDeleteParent` → verify button becomes enabled
   > 8. Click "Delete domain" button
   > 9. Verify all domains and accounts deleted via CLI
   > 
   > **Expected Result:**
   > 
   > * Modal displays accounts from entire hierarchy
   > * Only exact parent domain name enables delete button
   > * Parent and all children deleted with cleanup=true
   > * All associated accounts deleted
   > 
   > **Actual Result:**
   > 
   > * Modal correctly displayed cascadeuser account (from child domain 
CascadeChild1)
   > * Typing child domain name did NOT enable delete button (correct behavior)
   > * Only typing exact name "TestDeleteParent" enabled the button
   > * Cascading deletion successful:
   >   
   >   * TestDeleteParent deleted
   >   * CascadeChild1 deleted
   >   * CascadeChild2 deleted
   >   * cascadeuser account deleted
   > * Domain count reduced from 6 to 3
   > 
   > **Test Evidence:**
   > 
   > * Pre-deletion structure (6 domains):
   > 
   > <img alt="image" width="2000" height="1673" 
src="https://private-user-images.githubusercontent.com/201906357/541112882-41abeab8-be86-4937-b63b-42376607b5d9.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Njk2ODAyODksIm5iZiI6MTc2OTY3OTk4OSwicGF0aCI6Ii8yMDE5MDYzNTcvNTQxMTEyODgyLTQxYWJlYWI4LWJlODYtNDkzNy1iNjNiLTQyMzc2NjA3YjVkOS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwMTI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDEyOVQwOTQ2MjlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT01MWIwYTYyNDhiNDMyNjk2ZTQyZjVhYTU2M2JiNjQ4ZjE4YmZlNWFjODY2NTQwM2IwOTllZjMyYTNmYzg2NmM5JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.sn5-rO1fzBzmu1l6szeaspptX_ClyG52_J2YnHDP9X8";>
   > * Modal showed:
   > 
   > <img alt="Screenshot from 2026-01-27 17-13-41" width="2000" height="1673" 
src="https://private-user-images.githubusercontent.com/201906357/541115211-5d374195-2472-4704-965e-53b0c9b3cd36.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3Njk2ODAyODksIm5iZiI6MTc2OTY3OTk4OSwicGF0aCI6Ii8yMDE5MDYzNTcvNTQxMTE1MjExLTVkMzc0MTk1LTI0NzItNDcwNC05NjVlLTUzYjBjOWIzY2QzNi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwMTI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDEyOVQwOTQ2MjlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT0zNzMxOTJiNWQ1NGFkNjg5MjY1ZTRkZTc2MWE0NjQxNzEzZDc2ODllNTQ1YjExYjk5YThjNjdlOTJjZDdkNmVhJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.EZh3sCG-9PtAkoi_uL5p6c_IWjdo7G3Dj5HJG7zRLpc";>
   > * Deletion
   > 
   >  Screencast.from.2026-01-27.17-14-06.webm 
   > * Post-deletion verification:
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 3,
   >   "domain": [
   >     { "name": "ROOT", ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... }
   >   ]
   > }
   > ```
   > 
   > * TestDeleteParent, CascadeChild1, CascadeChild2 all deleted
   > 
   > ```
   > (admin) 🐱 > list accounts listall=true
   > {
   >   "count": 3,
   >   "account": [
   >     { "name": "admin", ... },
   >     { "name": "baremetal-system-account", ... },
   >     { "name": "ACSUser", ... }
   >   ]
   > }
   > ```
   > 
   > * cascadeuser account deleted - only system accounts remain
   > 
   > **Status:** PASS
   > 
   > **Additional Observation:** Input validation correctly rejects child 
domain names - only the exact parent domain name enables deletion. This 
prevents accidental deletion of wrong domain.
   > 
   > ### TC5: Mass Delete - Multiple Domains in Rapid Succession
   > **Objective:** Verify that multiple domains can be deleted in rapid 
succession without conflicts, errors, or UI issues. Tests async job handling 
when multiple deletions are triggered quickly.
   > 
   > **Test Steps:**
   > 
   > 1. Create 3 test domains: MassDelete1, MassDelete2, MassDelete3 under ROOT
   > 2. Refresh UI to display all domains
   > 3. Delete MassDelete1: Click → Delete action → type name → confirm
   > 4. Immediately delete MassDelete2: Click → Delete action → type name → 
confirm
   > 5. Immediately delete MassDelete3: Click → Delete action → type name → 
confirm
   > 6. Observe UI behavior during rapid deletions
   > 7. Verify all domains deleted via CLI
   > 
   > **Expected Result:**
   > 
   > * All 3 domains delete without errors
   > * Async jobs queue and complete properly
   > * UI updates dynamically after each deletion
   > * No conflicts between concurrent delete operations
   > 
   > **Actual Result:**
   > 
   > * All 3 domains (MassDelete1, MassDelete2, MassDelete3) deleted 
successfully
   > * No UI lag or delays observed
   > * No error messages or conflicts
   > * Domain tree updated dynamically after each deletion
   > * Async jobs handled properly without interference
   > * Domain count reduced from 6 to 3
   > * Pre-deletion (6 domains):
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 6,
   >   "domain": [
   >     { "name": "ROOT", ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... },
   >     { "name": "MassDelete1", "id": "b77d9d7e-20ef-4d52-a1e0-0729a30e8829", 
... },
   >     { "name": "MassDelete2", "id": "93b278bd-ea8a-41f4-a7a4-05a1688fa7e5", 
... },
   >     { "name": "MassDelete3", "id": "9a6a17ad-993c-4ac7-bad2-f09909c97880", 
... }
   >   ]
   > }
   > ```
   > 
   > * Post-deletion (3 domains):
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 3,
   >   "domain": [
   >     { "name": "ROOT", ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... }
   >   ]
   > }
   > ```
   > 
   > * All MassDelete domains successfully removed
   > 
   >  Screencast.from.2026-01-27.17-21-09.webm 
   > **Status:** PASS
   > 
   > ### TC6: Whitespace Handling + Delete While Viewing Domain
   > **Objective:** Verify that (1) domain name input correctly handles 
leading/trailing whitespace, and (2) UI navigates away when deleting a domain 
while viewing it.
   > 
   > **Test Steps:**
   > 
   > 1. Create WhitespaceTest domain under ROOT
   > 2. Navigate to Domains section and click on WhitespaceTest
   > 3. Click Delete Domain action button
   > 4. Type ` WhitespaceTest ` (with leading and trailing spaces)
   > 5. Observe if Delete button becomes enabled
   > 6. Click "Delete domain" button
   > 7. Observe if UI navigates away from deleted domain
   > 8. Click UI Refresh button
   > 9. Perform hard browser refresh (F5)
   > 10. Verify domain deleted via CLI
   > 
   > **Expected Result:**
   > 
   > * Whitespace trimmed - button should enable despite extra spaces
   > * After deletion, UI should navigate away to /domain (domains list)
   > * Detail panel should not show stale data
   > 
   > **Actual Result:**
   > 
   > * Whitespace handling: PASS - Button enabled with spaces (trim() working 
correctly)
   > * X Navigation after deletion: FAIL - UI did not navigate away
   > * X Detail panel showed stale data for deleted domain
   > * X UI Refresh button did not clear stale view
   > * Hard browser refresh (F5) correctly navigated to ROOT
   > * Domain tree correctly removed WhitespaceTest from list
   > * Domain deletion successful (confirmed via CLI)
   > 
   > **Test Evidence:**
   > 
   > * Whitespace input accepted - button enabled with "  WhitespaceTest  "
   > * Code correctly uses .trim() for comparison:
   > * canDelete () { return this.confirmText.trim() === 
this.domain.name.trim() }
   > 
   > -> After deletion:
   > 
   > * Domain tree: WhitespaceTest removed
   > * Detail panel: Still showing WhitespaceTest (stale)
   > * URL: Still on deleted domain's route
   > * UI Refresh: No effect
   > * Hard refresh (F5): Navigates to ROOT
   > * Post-deletion verification:
   > 
   > ```
   > (admin) 🐱 > list domains listall=true
   > {
   >   "count": 3,
   >   "domain": [
   >     { "name": "ROOT", ... },
   >     { "name": "Test_With_Subdomain", ... },
   >     { "name": "Test_Sub_1", ... }
   >   ]
   > }
   > ```
   > 
   > * WhitespaceTest successfully deleted but right panel **not refreshed even 
via Refresh button** (needs hard bewser refresh)
   > 
   >  Screencast.from.2026-01-27.17-29-27.webm 
   > **Status:** PARTIAL PASS (Whitespace: PASS, Navigation: FAIL - BUG)
   
   I appreciate your efforts !!Thanks


-- 
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: commits-unsubscr...@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org