Move method outside of the SearchBackend.

Author: isoos

app/lib/search/backend.dart

@@ -427,38 +427,6 @@ class SearchBackend {
     }
   }
 
-  /// Downloads the remote SDK content from [uri] and creates a cached file in the
-  /// `.dart_tool/pub-search-data/` directory.
-  ///
-  /// When a local file in `app/.dart_tool/pub-search-data/<reduced-uri>` exists,
-  /// its content will be loaded instead. URL reduction replaces slashes and other
-  /// non-characters with a single dash `-`, like:
-  /// - `https-api.dart.dev-stable-latest-index.json`
-  Future<String> loadOrFetchSdkIndexJsonAsString(
-    Uri uri, {
-    @visibleForTesting Duration? ttl,
-  }) async {
-    final fileName = uri.toString().replaceAll(RegExp(r'[^a-z0-9\.]+'), '-');
-    final file = File(p.join('.dart_tool', 'pub-search-data', fileName));
-    if (await file.exists()) {
-      var canUseCached = true;
-      if (ttl != null) {
-        final age = clock.now().difference(await file.lastModified());
-        if (age > ttl) {
-          canUseCached = false;
-        }
-      }
-      if (canUseCached) {
-        return await file.readAsString();
-      }
-    }
-
-    final content = await httpGetWithRetry(uri, responseFn: (rs) => rs.body);
-    await file.parent.create(recursive: true);
-    await file.writeAsString(content);
-    return content;
-  }
-
   Future<List<PackageDocument>?> fetchSnapshotDocuments() async {
     try {
       final map = await _snapshotStorage.getContentAsJsonMap();
@@ -609,6 +577,38 @@ List<ApiDocPage> apiDocPagesFromPubData(PubDartdocData pubData) {
   return results;
 }
 
+/// Downloads the remote SDK content from [uri] and creates a cached file in the
+/// `.dart_tool/pub-search-data/` directory.
+///
+/// When a local file in `app/.dart_tool/pub-search-data/<reduced-uri>` exists,
+/// its content will be loaded instead. URL reduction replaces slashes and other
+/// non-characters with a single dash `-`, like:
+/// - `https-api.dart.dev-stable-latest-index.json`
+Future<String> loadOrFetchSdkIndexJsonAsString(
+  Uri uri, {
+  @visibleForTesting Duration? ttl,
+}) async {
+  final fileName = uri.toString().replaceAll(RegExp(r'[^a-z0-9\.]+'), '-');
+  final file = File(p.join('.dart_tool', 'pub-search-data', fileName));
+  if (await file.exists()) {
+    var canUseCached = true;
+    if (ttl != null) {
+      final age = clock.now().difference(await file.lastModified());
+      if (age > ttl) {
+        canUseCached = false;
+      }
+    }
+    if (canUseCached) {
+      return await file.readAsString();
+    }
+  }
+
+  final content = await httpGetWithRetry(uri, responseFn: (rs) => rs.body);
+  await file.parent.create(recursive: true);
+  await file.writeAsString(content);
+  return content;
+}
+
 class _CombinedSearchIndex implements SearchIndex {
   const _CombinedSearchIndex();
 

app/lib/search/dart_sdk_mem_index.dart

@@ -38,8 +38,7 @@ SdkMemIndex? get dartSdkMemIndex =>
 Future<SdkMemIndex?> createDartSdkMemIndex() async {
   try {
     final index = await SdkMemIndex.dart();
-    final content =
-        await searchBackend.loadOrFetchSdkIndexJsonAsString(index.indexJsonUri);
+    final content = await loadOrFetchSdkIndexJsonAsString(index.indexJsonUri);
     await index.addDartdocIndex(DartdocIndex.parseJsonText(content));
     index.updateWeights(
       libraryWeights: dartSdkLibraryWeights,

app/lib/search/flutter_sdk_mem_index.dart

@@ -55,8 +55,7 @@ SdkMemIndex? get flutterSdkMemIndex =>
 Future<SdkMemIndex?> createFlutterSdkMemIndex() async {
   try {
     final index = SdkMemIndex.flutter();
-    final content =
-        await searchBackend.loadOrFetchSdkIndexJsonAsString(index.indexJsonUri);
+    final content = await loadOrFetchSdkIndexJsonAsString(index.indexJsonUri);
     await index.addDartdocIndex(DartdocIndex.parseJsonText(content),
         allowedLibraries: _allowedLibraries);
     index.updateWeights(

app/test/search/backend_test.dart

@@ -14,8 +14,7 @@ void main() {
   group('search backend', () {
     testWithProfile('fetch SDK library description', fn: () async {
       final index = await SdkMemIndex.dart();
-      final content = await searchBackend
-          .loadOrFetchSdkIndexJsonAsString(index.indexJsonUri);
+      final content = await loadOrFetchSdkIndexJsonAsString(index.indexJsonUri);
       await index.addDartdocIndex(DartdocIndex.parseJsonText(content));
       expect(
         index.getLibraryDescription('dart:async'),

app/test/search/dartdoc_index_parsing_test.dart

@@ -13,7 +13,7 @@ import '../shared/test_services.dart';
 void main() {
   group('dartdoc index.json parsing', () {
     testWithProfile('parse Dart SDK index.json', fn: () async {
-      final textContent = await searchBackend.loadOrFetchSdkIndexJsonAsString(
+      final textContent = await loadOrFetchSdkIndexJsonAsString(
         Uri.parse('https://api.dart.dev/stable/latest/index.json'),
         ttl: Duration(days: 1),
       );
@@ -35,7 +35,7 @@ void main() {
     });
 
     testWithProfile('parse Flutter SDK index.json', fn: () async {
-      final textContent = await searchBackend.loadOrFetchSdkIndexJsonAsString(
+      final textContent = await loadOrFetchSdkIndexJsonAsString(
         Uri.parse('https://api.flutter.dev/flutter/index.json'),
         ttl: Duration(days: 1),
       );