fix: handle change on country selection (#247)

* fix: handle change on country selection

* docs: update changelog

* test: add case where onChange should be triggered when a country is selected

* fix: swapped the parameter placement in the expect function so that it makes more sense

CHANGELOG.md

@@ -1,4 +1,5 @@
 ## [9.2.5]
+- fix: handle change on country selection. onChanged callback would trigger if country field is updated 
 - added Korean localization messages
 
 ## [9.2.4]

lib/src/phone_form_field_state.dart

@@ -70,6 +70,8 @@ class PhoneFormFieldState extends FormFieldState<PhoneNumber> {
     final selected = await widget.countrySelectorNavigator.show(context);
     if (selected != null) {
       controller.changeCountry(selected);
+      didChange(controller.value);
+      widget.onChanged?.call(controller.value);
     }
     focusNode.requestFocus();
   }

test/phone_form_field_test.dart

@@ -250,6 +250,32 @@ void main() {
       );
     });
 
+    testWidgets('Should call onChange when countryCode updated', (tester) async {
+      bool changed = false;
+      PhoneNumber? phoneNumber =
+      PhoneNumber.parse('', destinationCountry: IsoCode.FR);
+      void onChanged(PhoneNumber? p) {
+        changed = true;
+        phoneNumber = p;
+      }
+
+      await tester.pumpWidget(
+        getWidget(
+          initialValue: phoneNumber,
+          onChanged: onChanged,
+        ),
+      );
+      await tester.tap(find.byType(CountryButton));
+      await tester.pumpAndSettle();
+      expect(find.byType(CountrySelectorPage), findsOneWidget);
+      await tester.tap(find.byType(ListTile).first);
+      expect(changed, equals(true));
+      expect(
+        phoneNumber,
+        equals(PhoneNumber.parse('', destinationCountry: IsoCode.AF)),
+      );
+    });
+
     group('validator', () {
       testWidgets(
           'Should display invalid message when PhoneValidator.valid is used '