Merge pull request #839 from jhkennedy/dont-bonk

Only check if results are cloud hosted when there are results

CHANGELOG.md

@@ -1,31 +1,40 @@
 # Changelog
 
+All notable changes to this project will be documented in this file.
+
+The format is based on [Common Changelog](https://common-changelog.org/)
+and this project uses [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
+
 ## [Unreleased]
 
 ### Changed
 
 - Use built-in `assert` statements instead of `unittest` assertions in
   integration tests ([#743](https://github.com/nsidc/earthaccess/issues/743))
-  (@chuckwondo)
+  ([@chuckwondo](https://github.com/chuckwondo))
 
 ### Added
 
 - Add support for `NETRC` environment variable to override default `.netrc` file
   location ([#480](https://github.com/nsidc/earthaccess/issues/480))
-  (@chuckwondo)
+  ([@chuckwondo](https://github.com/chuckwondo))
 - Add `nox` session for running integration tests locally
-  ([#815](https://github.com/nsidc/earthaccess/issues/815)) (@chuckwondo)
+  ([#815](https://github.com/nsidc/earthaccess/issues/815)) ([@chuckwondo](https://github.com/chuckwondo))
 - Auto-add comment to PR that requires maintainer to review and re-run
   integration tests ([#824](https://github.com/nsidc/earthaccess/issues/824))
-  (@chuckwondo)
+  ([@chuckwondo](https://github.com/chuckwondo))
 
 ### Removed
 
 ### Fixed
 
+
 - Integration tests no longer clobber existing `.netrc` file
   ([#806](https://github.com/nsidc/earthaccess/issues/806))
   (@chuckwondo)
+- Return an empty list instead of raising an `IndexError` when searches find no results.
+  ([#526](https://github.com/nsidc/earthaccess/issues/526))
+  ([@jhkennedy](https://github.com/jhkennedy))
 
 ## [0.11.0] 2024-10-01
 

earthaccess/search.py

@@ -468,8 +468,7 @@ def get(self, limit: int = 2000) -> List[DataGranule]:
             RuntimeError: The CMR query failed.
         """
         response = get_results(self.session, self, limit)
-
-        cloud = self._is_cloud_hosted(response[0])
+        cloud = len(response) > 0 and self._is_cloud_hosted(response[0])
 
         return [DataGranule(granule, cloud_hosted=cloud) for granule in response]