Skip to content

Comments

Fix E2E tests#2727

Merged
trslater merged 1 commit intodevelopfrom
2649-e2e-fixes
Jan 13, 2026
Merged

Fix E2E tests#2727
trslater merged 1 commit intodevelopfrom
2649-e2e-fixes

Conversation

@trslater
Copy link
Collaborator

No description provided.

type="button"
(click)="toggleInput()"
>
data-testid="search-button"
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't like having to do this, but there didn't seem to be a way around it.

@sonarqubecloud
Copy link

@sonarqubecloud
Copy link


async expectSubmissionOriginalPdfInTable(fileId: string) {
const cellTexts = ['SUBORIG', `${fileId}_APP_Submission.pdf`, 'Applicant - Portal', 'A, C*, G'];
const cellTexts = ['SUBORIG', `${fileId}_APP_Submission.pdf`, 'Applicant - Portal', 'A , C* , G'];
Copy link
Contributor

Choose a reason for hiding this comment

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

Could add a regex here if spacing is an issue. Not a big problem though.

Copy link
Contributor

@PaulGreywal PaulGreywal left a comment

Choose a reason for hiding this comment

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

LGTM, one small comment but does not change my approval.

@trslater trslater merged commit d037a69 into develop Jan 13, 2026
13 of 14 checks passed
@trslater trslater deleted the 2649-e2e-fixes branch January 13, 2026 23:22
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