digipolisantwerp / location-picker_widget_angular

Antwerp UI Location Picker Widget UI (Angular)
Other
2 stars 6 forks source link

Feature: Added coordinateSearch parameter for LocationPicker. #121

Closed DanielSavin2001 closed 6 months ago

DanielSavin2001 commented 6 months ago

@TriangleJuice

Ik heb lokaal de aanpassingen gemaakt, en zou je mij meer info kunnen geven over welke testen specifiek, want ik heb in de project eens gezien en ik zag geen gelijkaardige testen dat ik kon hergebruiken & bijsturen.

Wat & hoe moet er effectief getest worden?

TriangleJuice commented 6 months ago

@TriangleJuice

Ik heb lokaal de aanpassingen gemaakt, en zou je mij meer info kunnen geven over welke testen specifiek, want ik heb in de project eens gezien en ik zag geen gelijkaardige testen dat ik kon hergebruiken & bijsturen.

Wat & hoe moet er effectief getest worden?

Hey @DanielSavin2001 je kan bv. wat zaken door de 'buildAddressQuery' function van de helper service gooien en testen of de output naargelang de input telkens juist is.

DanielSavin2001 commented 6 months ago

@TriangleJuice Ik heb lokaal de aanpassingen gemaakt, en zou je mij meer info kunnen geven over welke testen specifiek, want ik heb in de project eens gezien en ik zag geen gelijkaardige testen dat ik kon hergebruiken & bijsturen. Wat & hoe moet er effectief getest worden?

Hey @DanielSavin2001 je kan bv. wat zaken door de 'buildAddressQuery' function van de helper service gooien en testen of de output naargelang de input telkens juist is.

@TriangleJuice

Testen zijn toegevoegd, het was zeker terechte feedback! 😄

Ik wacht dan enkel nog op feedback ivm Icoontje: https://github.com/digipolisantwerp/location-picker_widget_angular/commit/7f5559fedcd0f2f721a8dfb72131867cb448ff12