[2.2.0rc2, UI, Test hardware] Change the wording of the error message for hardware testing a deployed machine
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| MAAS |
Fix Released
|
Low
|
Steve Rydz | ||
Bug Description
[2.2.0rc2, UI, Test hardware] Change the wording of the error message for hardware testing a deployed machine.
Rephrasing the message to be more consistent with the language use in MAAS.
Bug description:
* Current result:
Given I have selected a destructive test for hardware testing a deployed machine
When I click on the button Test [x] machines
Then I see an error message with the copy "Node failed to be tested because of the following error: Unable to run destructive test while deployed!"
* Expected/ correct result:
Given I have selected a destructive test for hardware testing of a deployed machine in either the machine page or the machine listing
When I click on the button Test machine/ Test [x] machines
Then for the machine I should see an error message with the copy "The selected hardware tests contain one or more destructive tests. Destructive tests cannot run on deployed machines."
And for multiple machine selection I should see an error message with the copy: "[x] machines cannot run hardware testing. The selected hardware tests contain one or more destructive tests. Destructive tests cannot run on deployed machines."
[OS X Sierra]
Chrome, 56.0.2924.87 (64-bit)
Related branches
- Caleb Ellis (community): Approve
-
Diff: 189 lines (+123/-4)6 files modifiedsrc/maasserver/static/js/angular/controllers/node_details.js (+11/-0)
src/maasserver/static/js/angular/controllers/nodes_list.js (+28/-0)
src/maasserver/static/js/angular/controllers/tests/test_node_details.js (+30/-0)
src/maasserver/static/js/angular/controllers/tests/test_nodes_list.js (+52/-0)
src/maasserver/static/partials/node-details.html (+1/-1)
src/maasserver/static/partials/nodes-list.html (+1/-3)
| Maria Vrachni (m-vrachnis) wrote | #1 |
| Changed in maas: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| Changed in maas: | |
| importance: | Medium → Wishlist |
| milestone: | none → next |
| importance: | Wishlist → Low |
| milestone: | next → 2.2.0rc2 |
| Changed in maas: | |
| milestone: | 2.2.0rc2 → 2.2.0rc3 |
| Changed in maas: | |
| milestone: | 2.2.0rc3 → 2.2.1 |
| Changed in maas: | |
| milestone: | 2.2.1 → 2.2.x |
| tags: | added: papercut |
| Changed in maas: | |
| milestone: | 2.2.x → 2.3.x |
| Karl Williams (deadlight) wrote | #2 |
| Changed in maas: | |
| assignee: | nobody → Karl Williams (deadlight) |
| Changed in maas: | |
| assignee: | Karl Williams (deadlight) → Steve Rydz (steverydz) |
| status: | Triaged → In Progress |
| Changed in maas: | |
| status: | In Progress → Triaged |
| Changed in maas: | |
| status: | Triaged → In Progress |
| Changed in maas: | |
| status: | In Progress → Fix Committed |
| Changed in maas: | |
| status: | Fix Committed → Fix Released |
The string that needs to change is in src/maasserver/