mirror of
https://github.com/fleetdm/fleet
synced 2026-05-22 08:28:52 +00:00
update logic and copy in case of vpp verification failure (#30964)
> Closes #30938 # Checklist for submitter If some of the following don't apply, delete the relevant line. <!-- Note that API documentation changes are now addressed by the product design team. --> - [x] Added/updated automated tests - [x] Manual QA for all new/changed functionality - [x] For unreleased bug fixes in a release candidate, confirmed that the fix is not expected to adversely impact load test results or alerted the release DRI if additional load testing is needed.
This commit is contained in:
parent
f79fed9712
commit
da697eb0c4
2 changed files with 29 additions and 1 deletions
|
|
@ -57,6 +57,23 @@ describe("getStatusMessage helper function", () => {
|
|||
).toBeInTheDocument();
|
||||
});
|
||||
|
||||
it("shows failed verification message", () => {
|
||||
render(
|
||||
getStatusMessage({
|
||||
displayStatus: "failed_install",
|
||||
isStatusNotNow: false,
|
||||
isStatusAcknowledged: true,
|
||||
software_title: "Logic Pro",
|
||||
host_display_name: "Marko's MacBook Pro",
|
||||
})
|
||||
);
|
||||
expect(
|
||||
screen.getByText(
|
||||
/but the installation has not been verified. Please re-attempt this installation/i
|
||||
)
|
||||
).toBeInTheDocument();
|
||||
});
|
||||
|
||||
it("shows default message for installed status", () => {
|
||||
render(
|
||||
getStatusMessage({
|
||||
|
|
|
|||
|
|
@ -67,7 +67,18 @@ export const getStatusMessage = ({
|
|||
);
|
||||
}
|
||||
|
||||
// VPP failed state
|
||||
// Verification failed (timeout)
|
||||
if (displayStatus === "failed_install" && isStatusAcknowledged) {
|
||||
return (
|
||||
<>
|
||||
The MDM command (request) to install <b>{software_title}</b> on{" "}
|
||||
{formattedHost} was acknowledged but the installation has not been
|
||||
verified. Please re-attempt this installation.
|
||||
</>
|
||||
);
|
||||
}
|
||||
|
||||
// Install command failed
|
||||
if (displayStatus === "failed_install") {
|
||||
return (
|
||||
<>
|
||||
|
|
|
|||
Loading…
Reference in a new issue