Refactor proposal: use unnamed callback function in with*HttpPubApiClient methods.

Author: isoos

app/lib/fake/backend/fake_auth_provider.dart

@@ -419,10 +419,10 @@ Future<String> _acquireCsrfToken({
 ///
 /// The [email] is used to create an HTTP session and the related CSRF token is
 /// extracted from the session, both are sent alongside the requests.
-Future<R> withFakeAuthHttpPubApiClient<R>({
+Future<R> withFakeAuthHttpPubApiClient<R>(
+  Future<R> Function(PubApiClient client) fn, {
   required String email,
   List<String>? scopes,
-  required Future<R> Function(PubApiClient client) fn,
   String? pubHostedUrl,
   Set<String>? experimental,
 }) async {
@@ -440,8 +440,8 @@ Future<R> withFakeAuthHttpPubApiClient<R>({
     sessionId: sessionId,
     csrfToken: csrfToken,
     pubHostedUrl: pubHostedUrl,
-    fn: fn,
     experimental: experimental,
+    fn,
   );
 }
 

app/lib/tool/test_profile/importer.dart

@@ -49,7 +49,7 @@ Future<void> importProfile({
       email: firstMemberEmail,
       scopes: [webmasterScope],
       pubHostedUrl: pubHostedUrl,
-      fn: (client) async {
+      (client) async {
         try {
           await client.createPublisher(p.name);
         } on RequestException catch (e) {
@@ -98,7 +98,7 @@ Future<void> importProfile({
           bearerToken: createFakeAuthTokenForEmail(uploaderEmail,
               audience: activeConfiguration.pubClientAudience),
           pubHostedUrl: pubHostedUrl,
-          fn: (client) => client.uploadPackageBytes(bytes),
+          (client) => client.uploadPackageBytes(bytes),
         );
         published = true;
       } catch (e, st) {
@@ -123,7 +123,7 @@ Future<void> importProfile({
     await withFakeAuthHttpPubApiClient(
       email: activeEmail!,
       pubHostedUrl: pubHostedUrl,
-      fn: (client) async {
+      (client) async {
         // update publisher
         if (testPackage.publisher != null) {
           await client.setPackagePublisher(
@@ -155,7 +155,7 @@ Future<void> importProfile({
         bearerToken:
             createFakeServiceAccountToken(email: adminUserEmail ?? activeEmail),
         pubHostedUrl: pubHostedUrl,
-        fn: (client) async {
+        (client) async {
           await client.adminPostAssignedTags(
             packageName,
             PatchAssignedTags(
@@ -173,7 +173,7 @@ Future<void> importProfile({
     await withFakeAuthHttpPubApiClient(
       email: u.email,
       pubHostedUrl: pubHostedUrl,
-      fn: (client) async {
+      (client) async {
         // creates user (regardless of likes being specified)
         await client.listPackageLikes();
 
@@ -197,7 +197,7 @@ Future<void> importProfile({
         await withFakeAuthHttpPubApiClient(
           email: userEmail,
           pubHostedUrl: pubHostedUrl,
-          fn: (client) async {
+          (client) async {
             await client.likePackage(p.name);
           },
         );

app/lib/tool/utils/pub_api_client.dart

@@ -98,13 +98,13 @@ class _FakeTimeClient implements http.Client {
 ///
 /// If [bearerToken], [sessionId] or [csrfToken] is specified, the corresponding
 /// HTTP header will be sent alongside the request.
-Future<R> withHttpPubApiClient<R>({
+Future<R> withHttpPubApiClient<R>(
+  Future<R> Function(PubApiClient client) fn, {
   String? bearerToken,
   String? sessionId,
   String? csrfToken,
   String? pubHostedUrl,
   Set<String>? experimental,
-  required Future<R> Function(PubApiClient client) fn,
 }) async {
   final httpClient = httpClientWithAuthorization(
     tokenProvider: () async => bearerToken,

app/test/account/consent_backend_test.dart

@@ -28,7 +28,7 @@ void main() {
       await withFakeAuthHttpPubApiClient(
         email: adminEmail,
         pubHostedUrl: activeConfiguration.primarySiteUri.toString(),
-        fn: (client) async {
+        (client) async {
           await client.invitePackageUploader(
               'oxygen', InviteUploaderRequest(email: userAtPubDevEmail));
         },
@@ -309,15 +309,13 @@ void main() {
 
   group('Sanity check', () {
     testWithProfile('consent parameter length', fn: () async {
-      await withFakeAuthHttpPubApiClient(
-          email: adminAtPubDevEmail,
-          fn: (c) async {
-            await expectApiException(
-              c.consentInfo('abcd' * 500),
-              status: 400,
-              code: 'InvalidInput',
-            );
-          });
+      await withFakeAuthHttpPubApiClient(email: adminAtPubDevEmail, (c) async {
+        await expectApiException(
+          c.consentInfo('abcd' * 500),
+          status: 400,
+          code: 'InvalidInput',
+        );
+      });
     });
   });
 }

app/test/admin/exported_api_sync_test.dart

@@ -21,7 +21,7 @@ void main() {
     }) async {
       await withHttpPubApiClient(
         bearerToken: siteAdminToken,
-        fn: (api) async {
+        (api) async {
           await api.adminInvokeAction(
             'exported-api-sync',
             AdminInvokeActionArguments(arguments: {

app/test/admin/moderate_package_test.dart

@@ -34,7 +34,7 @@ void main() {
   group('Moderate package', () {
     Future<ModerationCase> _report(String package) async {
       await withHttpPubApiClient(
-        fn: (client) async {
+        (client) async {
           await client.postReport(ReportForm(
             email: '[email protected]',
             subject: 'package:$package',

app/test/admin/moderate_package_version_test.dart

@@ -34,7 +34,7 @@ void main() {
   group('Moderate package version', () {
     Future<ModerationCase> _report(String package, String version) async {
       await withHttpPubApiClient(
-        fn: (client) async {
+        (client) async {
           await client.postReport(ReportForm(
             email: '[email protected]',
             subject: 'package-version:$package/$version',

app/test/admin/moderate_publisher_test.dart

@@ -25,7 +25,7 @@ void main() {
   group('Moderate Publisher', () {
     Future<ModerationCase> _report(String publisherId) async {
       await withHttpPubApiClient(
-        fn: (client) async {
+        (client) async {
           await client.postReport(ReportForm(
             email: '[email protected]',
             subject: 'publisher:$publisherId',

app/test/admin/moderate_user_test.dart

@@ -29,7 +29,7 @@ void main() {
   group('Moderate User', () {
     Future<ModerationCase> _report(String package) async {
       await withHttpPubApiClient(
-        fn: (client) async {
+        (client) async {
           await client.postReport(account_api.ReportForm(
             email: '[email protected]',
             subject: 'package:$package',

app/test/admin/moderation_case_resolve_test.dart

@@ -19,7 +19,7 @@ void main() {
       required bool? apply,
     }) async {
       await withHttpPubApiClient(
-        fn: (client) async {
+        (client) async {
           await client.postReport(ReportForm(
             email: '[email protected]',
             caseId: appealCaseId,