Skip to content

Conversation

@YuriZmytrakov
Copy link
Collaborator

@YuriZmytrakov YuriZmytrakov commented Nov 22, 2025

Description:

When running tests locally, in dev environement, we are encountering redis import error, because it is not being installed:

ImportError while loading conftest '/app/stac_fastapi/tests/conftest.py'.
conftest.py:16: in <module>
    from stac_fastapi.core.core import (
../core/stac_fastapi/core/core.py:27: in <module>
    from stac_fastapi.core.redis_utils import redis_pagination_links
../core/stac_fastapi/core/redis_utils.py:11: in <module>
    from redis import asyncio as aioredis
E   ModuleNotFoundError: No module named 'redis'

PR Checklist:

  • Code is formatted and linted (run pre-commit run --all-files)
  • Tests pass (run make test)
  • Documentation has been updated to reflect changes, if applicable
  • Changes are added to the changelog

Copy link
Collaborator

@jonhealy1 jonhealy1 left a comment

Choose a reason for hiding this comment

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

Might as well add it to Elasticsearch too

@YuriZmytrakov
Copy link
Collaborator Author

Might as well add it to Elasticsearch too

I have added redis to dev list of dependencies.

@jonhealy1 jonhealy1 merged commit 7ac75ee into main Nov 25, 2025
12 checks passed
@jonhealy1 jonhealy1 deleted the CAT-1615 branch November 25, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants