Skip to content

Commit e13a05b

Browse files
committed
fix format errors
1 parent bf8a031 commit e13a05b

File tree

8 files changed

+56
-28
lines changed

8 files changed

+56
-28
lines changed

packages/auth/amplify_auth_cognito/lib/src/flows/hosted_ui/hosted_ui_platform_flutter.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ class HostedUiPlatformImpl extends io.HostedUiPlatformImpl {
9898
options.language,
9999
options.loginHint,
100100
options.prompt?.map((obj) => obj.value).toList(),
101-
options.resource
101+
options.resource,
102102
);
103103
dispatcher
104104
.dispatch(

packages/auth/amplify_auth_cognito/lib/src/native_auth_plugin.g.dart

Lines changed: 13 additions & 14 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/auth/amplify_auth_cognito/test/hosted_ui_platform_flutter_test.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ final class ThrowingNativeBridge extends Fake implements NativeAuthBridge {
8787
String? arg_language,
8888
String? arg_loginHint,
8989
List<String>? arg_prompt,
90-
String? arg_resource
90+
String? arg_resource,
9191
) async {
9292
throw PlatformException(code: 'CANCELLED');
9393
}

packages/auth/amplify_auth_cognito_dart/lib/src/flows/hosted_ui/hosted_ui_config.dart

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,10 @@ extension HostedUiConfig on OAuthOutputs {
2727
/// References:
2828
/// - https://docs.aws.amazon.com/cognito/latest/developerguide/authorization-endpoint.html
2929
/// - https://docs.aws.amazon.com/cognito/latest/developerguide/login-endpoint.html
30-
Uri signInUri([AuthProvider? provider, CognitoSignInWithWebUIPluginOptions? options]) {
30+
Uri signInUri([
31+
AuthProvider? provider,
32+
CognitoSignInWithWebUIPluginOptions? options,
33+
]) {
3134
Uri baseUri;
3235
// ignore: invalid_use_of_internal_member
3336
if (this.signInUri != null) {

packages/auth/amplify_auth_cognito_dart/lib/src/flows/hosted_ui/hosted_ui_platform.dart

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,11 @@ abstract class HostedUiPlatform implements Closeable {
102102
@protected
103103
@visibleForTesting
104104
@nonVirtual
105-
Future<Uri> getSignInUri({Uri? redirectUri, AuthProvider? provider, CognitoSignInWithWebUIPluginOptions? options}) async {
105+
Future<Uri> getSignInUri({
106+
Uri? redirectUri,
107+
AuthProvider? provider,
108+
CognitoSignInWithWebUIPluginOptions? options,
109+
}) async {
106110
final state = generateState();
107111
final codeVerifier = createCodeVerifier();
108112

@@ -124,7 +128,7 @@ abstract class HostedUiPlatform implements Closeable {
124128
codeVerifier: codeVerifier,
125129
httpClient: httpClient,
126130
provider: provider,
127-
options: options
131+
options: options,
128132
);
129133
final uri = _authCodeGrant!.getAuthorizationUrl(
130134
redirectUri ?? signInRedirectUri,

packages/auth/amplify_auth_cognito_dart/lib/src/flows/hosted_ui/hosted_ui_platform_html.dart

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,10 @@ class HostedUiPlatformImpl extends HostedUiPlatform {
5454
required CognitoSignInWithWebUIPluginOptions options,
5555
AuthProvider? provider,
5656
}) async {
57-
final signInUrl = (await getSignInUri(provider: provider, options: options)).toString();
57+
final signInUrl = (await getSignInUri(
58+
provider: provider,
59+
options: options,
60+
)).toString();
5861
await launchUrl(signInUrl);
5962
}
6063

packages/auth/amplify_auth_cognito_dart/lib/src/model/signin/cognito_sign_in_with_web_ui_plugin_options.dart

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ class CognitoSignInWithWebUIPluginOptions extends SignInWithWebUIPluginOptions {
1919
this.language,
2020
this.loginHint,
2121
this.prompt,
22-
this.resource
22+
this.resource,
2323
});
2424

2525
/// {@macro amplify_auth_cognito.model.cognito_sign_in_with_web_ui_plugin_options}
@@ -49,7 +49,7 @@ class CognitoSignInWithWebUIPluginOptions extends SignInWithWebUIPluginOptions {
4949
/// {@template amplify_auth_cognito.model.cognito_sign_in_with_web_ui_options.nonce}
5050
/// A random value that you can add to the request. The nonce value that you provide is included in the ID token
5151
/// that Amazon Cognito issues. To guard against replay attacks, your app can inspect the nonce claim in the ID
52-
/// token and compare it to the one you generated.
52+
/// token and compare it to the one you generated.
5353
/// {@endtemplate}
5454
final String? nonce;
5555

@@ -81,7 +81,15 @@ class CognitoSignInWithWebUIPluginOptions extends SignInWithWebUIPluginOptions {
8181
final String? resource;
8282

8383
@override
84-
List<Object?> get props => [isPreferPrivateSession, browserPackageName, nonce, language, loginHint, prompt, resource];
84+
List<Object?> get props => [
85+
isPreferPrivateSession,
86+
browserPackageName,
87+
nonce,
88+
language,
89+
loginHint,
90+
prompt,
91+
resource,
92+
];
8593

8694
@override
8795
String get runtimeTypeName => 'CognitoSignInWithWebUIPluginOptions';

packages/auth/amplify_auth_cognito_test/test/state/hosted_ui_state_machine_test.dart

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -575,7 +575,7 @@ void main() {
575575
) async {
576576
final signInUrl = await platform.getSignInUri(
577577
provider: provider,
578-
options: options
578+
options: options,
579579
);
580580
_launchUrl.complete(signInUrl);
581581
},
@@ -584,8 +584,16 @@ void main() {
584584
expect(options.nonce == 'nonce', isTrue);
585585
expect(options.language == 'en', isTrue);
586586
expect(options.loginHint == 'username', isTrue);
587-
expect(options.prompt?.contains(CognitoSignInWithWebUIPrompt.login), isTrue);
588-
expect(options.prompt?.contains(CognitoSignInWithWebUIPrompt.consent), isTrue);
587+
expect(
588+
options.prompt?.contains(CognitoSignInWithWebUIPrompt.login),
589+
isTrue,
590+
);
591+
expect(
592+
options.prompt?.contains(
593+
CognitoSignInWithWebUIPrompt.consent,
594+
),
595+
isTrue,
596+
);
589597
expect(options.resource == 'myapp://', isTrue);
590598
}),
591599
),
@@ -607,8 +615,11 @@ void main() {
607615
nonce: 'nonce',
608616
language: 'en',
609617
loginHint: 'username',
610-
prompt: [CognitoSignInWithWebUIPrompt.login, CognitoSignInWithWebUIPrompt.consent],
611-
resource: 'myapp://'
618+
prompt: [
619+
CognitoSignInWithWebUIPrompt.login,
620+
CognitoSignInWithWebUIPrompt.consent,
621+
],
622+
resource: 'myapp://',
612623
),
613624
),
614625
)

0 commit comments

Comments
 (0)