Skip to content

chore: harden Dapr Redis integration fixture loading#3078

Merged
seratch merged 1 commit intoopenai:mainfrom
Aphroq:fix/dapr-redis-integration-fixture
May 2, 2026
Merged

chore: harden Dapr Redis integration fixture loading#3078
seratch merged 1 commit intoopenai:mainfrom
Aphroq:fix/dapr-redis-integration-fixture

Conversation

@Aphroq
Copy link
Copy Markdown
Contributor

@Aphroq Aphroq commented May 1, 2026

Summary

This pull request fixes the Dapr Redis integration fixture so the sidecar can read the mounted component file reliably and the test waits for the statestore component to be registered before using it.

Test plan

  • uv run pytest tests/extensions/memory/test_dapr_redis_integration.py -q

Issue number

N/A

Checks

  • I've added new tests (if relevant)
  • I've added/updated the relevant documentation
  • I've run make lint and make format
  • I've made sure tests pass

@Aphroq Aphroq changed the title test: harden Dapr Redis integration fixture loading chore: harden Dapr Redis integration fixture loading May 1, 2026
@seratch
Copy link
Copy Markdown
Member

seratch commented May 2, 2026

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@seratch seratch merged commit 42c3015 into openai:main May 2, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants