-
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: E2E coverage for Draw Boundary #4035
Conversation
6251e38
to
dc9caa0
Compare
Removed vultr server and associated DNS entries |
26d70e0
to
c5a5d7c
Compare
@@ -37,3 +43,23 @@ export const mockMapGeoJson: Feature = { | |||
"organisation-entity": "13", | |||
}, | |||
}; | |||
|
|||
export const mockChangedMapGeoJson: GeoJsonChangeHandler = { |
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.
pulled this from the tests so I had a static mock to check against when running the tests
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.
This looks like a solid start! Some initial comments below and a larger one:
- I was expecting to find mocks for Ordnance Survey API calls as part of this work - when you load up the DrawBoundary here, is it making live calls to the OS Vector Tiles API ? Are OS mocks something you previously handled with FindProperty & PropertyInformation components before this ?
We definitely don't want our live OS APIs to be called repeated by E2E so let's catch up about this please to move this forward !
remove upload and label refs initial expects for area and uplaod file button
ea4c351
to
cd70fe5
Compare
address comments from review refine osMapsMocks rename OSMapMocks
d086f26
to
e2d2534
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.
Thank you for adding mocks here, looks good to go to me 👍
What does this Test do?
For the Draw Boundary testing, I have focused only on:
I have left testing the file upload to the relevant unit tests or related testing elsewhere in the code base. I thought this was the easiest given we are yet to do the
FileUploadAndLabel
e2e tests. I can come back to add this in after that.Rather than add in comments, in some parts I have just added custom text to the
expect()
. It made it easier for me to track when running the test and it was acting the same as the comments in the code.