Skip to content
This repository has been archived by the owner on Dec 29, 2022. It is now read-only.

Throw in tests when getting onGeoQueryError. #116

Merged
merged 1 commit into from Dec 22, 2017
Merged

Throw in tests when getting onGeoQueryError. #116

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 1 addition & 2 deletions src/test/java/com/firebase/geofire/GeoQueryDataEventTestListener.java
View file
Open in desktop
Expand Up @@ -74,7 +74,6 @@ public void onGeoQueryReady() {

@Override
public void onGeoQueryError(DatabaseError error) {

throw error.toException();
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've noticed that this is using 2 spaces which is part of the Java Google Style Guide. I also prefer 2 spaces. Should we change it everywhere or was this a mistake?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was probably a mistake, I prefer 4 spaces but was using a new version of IntelliJ I had not set up.

}

}
1 change: 1 addition & 0 deletions src/test/java/com/firebase/geofire/GeoQueryEventTestListener.java
View file
Open in desktop
Expand Up @@ -59,5 +59,6 @@ public void onGeoQueryReady() {

@Override
public void onGeoQueryError(DatabaseError error) {
throw error.toException();
}
}