Skip to content

fix(firestore): Change asserts to throw argumentError #17302

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
MichaelVerdon merged 9 commits into from
Apr 24, 2025
Merged

fix(firestore): Change asserts to throw argumentError #17302

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
9bb372a
fix(firestore): Change asserts to throw argumentError
MichaelVerdon Apr 23, 2025
3292979
fix: add comma
MichaelVerdon Apr 23, 2025
34dae1d
fix: formatting
MichaelVerdon Apr 24, 2025
24a8dc8
fix: ArgumentError in tests
MichaelVerdon Apr 24, 2025
ac44dfd
fix: more assertion error to argument error in tests
MichaelVerdon Apr 24, 2025
a1fdd7d
fix: more assert to argumentError
MichaelVerdon Apr 24, 2025
737fca5
fix: More asserts to argumentError
MichaelVerdon Apr 24, 2025
679912b
fix: format
MichaelVerdon Apr 24, 2025
03a9dab
fix: format consistency
MichaelVerdon Apr 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: More asserts to argumentError
  • Loading branch information
@MichaelVerdon
MichaelVerdon committed Apr 24, 2025
commit 737fca594fa944a5c781c7eee81c6abaecb141e9
10 changes: 7 additions & 3 deletions packages/cloud_firestore/cloud_firestore/lib/src/collection_reference.dart
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,9 +113,13 @@ class _JsonCollectionReference extends _JsonQuery
@override
DocumentReference<Map<String, dynamic>> doc([String? path]) {
if (path != null) {
assert(path.isNotEmpty, 'a document path must be a non-empty string');
assert(!path.contains('//'), 'a document path must not contain "//"');
assert(path != '/', 'a document path must point to a valid document');
if (path.isEmpty) {
throw ArgumentError('A document path must be a non-empty string');
} else if (path.contains('//')) {
throw ArgumentError('A document path must not contain "//"');
} else if (path == '/') {
throw ArgumentError('A document path must point to a valid document');
}
}

return _JsonDocumentReference(firestore, _delegate.doc(path));
Expand Down
Loading