Skip to content

test(appeals): appellant statemants e2e tests (A2-6922)#3559

Open
abitec wants to merge 1 commit intomainfrom
test/a2-6922-appeallant-statement
Open

test(appeals): appellant statemants e2e tests (A2-6922)#3559
abitec wants to merge 1 commit intomainfrom
test/a2-6922-appeallant-statement

Conversation

@abitec
Copy link
Copy Markdown
Collaborator

@abitec abitec commented Mar 27, 2026

Describe your changes

This PR implements e2e tests for appellant statements:

  • Update allocation level and specialisms
  • Accept statement
  • Redact and accept statement
  • Mark statement as incomplete
  • Add statement for overdue submission
  • Issue ticket number and link

Ticket

caseDetailsPage.validateBannerMessage('Important', 'Appellant statement awaiting review');
};

const reviewAppellantStatement = (decision, stepsCallback) => {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe followUpSteps e.g. might be better as name for that parameter, just to bit clearer as to what it is?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, will do.
Thanks.

cy.deleteAppeals(appeal);
});

it('should update allocation level and specialisms', () => {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this test only goes as far as navigating to the allocation page and checking header? is it not possible to update the allocation level yet?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, I copied this one from LPA statement.
I'll update it.
Thanks.

Copy link
Copy Markdown
Collaborator

@marcpodgorney81 marcpodgorney81 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
Nice job @abitec! 👍

@abitec abitec force-pushed the test/a2-6922-appeallant-statement branch from 7368fee to 8cde2ea Compare March 27, 2026 13:51
@abitec abitec enabled auto-merge March 27, 2026 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants