-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Map and Label Copy Tests #3695
Conversation
Removed vultr server and associated DNS entries |
bf8931d
to
9f49a74
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor suggestions, but all good!
editor.planx.uk/src/@planx/components/MapAndLabel/Public/index.test.tsx
Outdated
Show resolved
Hide resolved
editor.planx.uk/src/@planx/components/MapAndLabel/Public/index.test.tsx
Outdated
Show resolved
Hide resolved
const urgencyDiv = getByTitle("Urgency"); | ||
const urgencySelect = within(urgencyDiv).getByRole("combobox"); | ||
|
||
expect(getByLabelText("Species")).toHaveDisplayValue(mockTreeData.species); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DafyddLlyr is this the sort of thing you meant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect!
What does this PR do?
This continues work on the tests for the Map and Label component, building up tests for the Copy from feature