Merge pull request #674 from wmde/city-autocomplete-fix

Add model watch to City Autocomplete

Author: Abban

src/components/shared/form_fields/CityAutocompleteField.vue

@@ -181,6 +181,10 @@ watch( city, ( newCity: string ) => {
 	emit( 'update:modelValue', newCity );
 } );
 
+watch( () => props.modelValue, ( newValue: string ) => {
+	city.value = newValue;
+} );
+
 </script>
 
 <style lang="scss">

tests/unit/components/shared/form_fields/CityAutocompleteField.spec.ts

@@ -261,4 +261,14 @@ describe( 'CityAutocompleteField.vue', () => {
 
 		expect( wrapper.emitted( 'field-changed' ).length ).toStrictEqual( 1 );
 	} );
+
+	it( 'updates value on model change', async () => {
+		const wrapper = getWrapper();
+
+		expect( wrapper.find<HTMLInputElement>( '#city' ).element.value ).toBe( '' );
+
+		await wrapper.setProps( { modelValue: 'Berlin' } );
+
+		expect( wrapper.find<HTMLInputElement>( '#city' ).element.value ).toBe( 'Berlin' );
+	} );
 } );