From ccafdbffd1419fdeda5fcb134ab66a1bd50c859a Mon Sep 17 00:00:00 2001 From: Enguerrand_ARMINJON_MAC_2 Date: Mon, 13 Jan 2025 16:59:24 +0100 Subject: [PATCH] fix: Wrong ios bundleID regex --- .../flutterfire_cli/lib/src/common/utils.dart | 2 +- packages/flutterfire_cli/test/unit_tests.dart | 21 ++++++++++++------- 2 files changed, 15 insertions(+), 8 deletions(-) diff --git a/packages/flutterfire_cli/lib/src/common/utils.dart b/packages/flutterfire_cli/lib/src/common/utils.dart index 5c886e78..4496a5fb 100644 --- a/packages/flutterfire_cli/lib/src/common/utils.dart +++ b/packages/flutterfire_cli/lib/src/common/utils.dart @@ -511,7 +511,7 @@ void validateAppBundleId( String platform, ) { final bundleIdRegex = RegExp( - r'^[a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?(\.[a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?)+$', + r'^([a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?|([a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?(\.[a-zA-Z0-9]([a-zA-Z0-9\-]{0,61}[a-zA-Z0-9])?)+))$', ); if (!bundleIdRegex.hasMatch(bundleId)) { diff --git a/packages/flutterfire_cli/test/unit_tests.dart b/packages/flutterfire_cli/test/unit_tests.dart index 3d38382b..b8011825 100644 --- a/packages/flutterfire_cli/test/unit_tests.dart +++ b/packages/flutterfire_cli/test/unit_tests.dart @@ -42,6 +42,20 @@ void main() { ), returnsNormally, ); + expect( + () => validateAppBundleId( + 'example', + kIos, + ), + returnsNormally, + ); + expect( + () => validateAppBundleId( + 'exampleEN', + kIos, + ), + returnsNormally, + ); }); test('Invalid bundle IDs', () { @@ -73,13 +87,6 @@ void main() { ), throwsA(isA()), ); - expect( - () => validateAppBundleId( - 'example', - kIos, - ), - throwsA(isA()), - ); }); });