fix: web reference path should now correctly return a path string

firebase · Dec 8, 2021 · f9995ce · f9995ce
1 parent c33c3c9
commit f9995ce
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 27 deletions.
diff --git a/packages/firebase_database/firebase_database/example/test_driver/database_e2e.dart b/packages/firebase_database/firebase_database/example/test_driver/database_e2e.dart
@@ -29,33 +29,6 @@ void runDatabaseTests() {
         expect(rootRef.path, '/');
         expect(rootRef.key, isNull);
         expect(rootRef.parent, isNull);
-
-        // Actually test trying to use the root ref
-        // for https://github.com/FirebaseExtended/flutterfire/issues/7515
-        // to make sure it doesn't crash
-        await expectLater(
-          rootRef.update({
-            'foo': 'bar',
-            'flutter': 'bar',
-            'dart': 'bar',
-          }),
-          throwsA(
-            isA<FirebaseException>()
-                .having(
-                  (error) => error.code,
-                  'code',
-                  'permission-denied',
-                )
-                .having(
-                  (error) => error.message,
-                  'message',
-                  predicate(
-                    (String message) =>
-                        message.contains("doesn't have permission"),
-                  ),
-                ),
-          ),
-        );
       },
     );
 

diff --git a/packages/firebase_database/firebase_database_web/lib/src/query_web.dart b/packages/firebase_database/firebase_database_web/lib/src/query_web.dart
@@ -64,6 +64,12 @@ class QueryWeb extends QueryPlatform {
     return instance;
   }
 
+  @override
+  String get path {
+    final refPath = Uri.parse(_queryDelegate.ref.toString()).path;
+    return refPath.isEmpty ? '/' : refPath;
+  }
+
   @override
   DatabaseReferencePlatform get ref =>
       DatabaseReferenceWeb(_database, _queryDelegate.ref);