Skip to content

Fix tests based on changes in bluesky-queueserver - #98

Merged
dmgav merged 1 commit into
mainfrom
fix-tests
Oct 16, 2026
Merged

dmgav merged 1 commit into
mainfrom
fix-tests

Conversation

@thopkins32

Copy link
Copy Markdown
Contributor

Assisted-by: oh-my-pi:claude-opus-5.5

Description

  • test_system_info_socket.py:91 and test_auth_for_websockets.py:118: the manager now starts with --zmq-publish-info ON instead of --zmq-publish-console ON. The console flag was only there because it used to switch on status publishing as well. These tests only use /status/ws and /info/ws, so console output isn't needed.
  • test_core_api_main.py::test_http_server_re_metadata: the test now sets up its own metadata the way queueserver's test_zmq_api_re_metadata_07 does. It copies the default startup profile, appends RE.md['metadata_key'] = 'metadata_value', and starts the manager with --startup-dir <copy> --permitted-re-metadata-keys /metadata_key /versions. The assertions are unchanged.

Motivation and Context

Upstream changes made in bluesky-queueserver/main were causing the tests to fail.

Summary of Changes for Release Notes

Fixed

Added

Changed

Removed

How Has This Been Tested?

Assisted-by: oh-my-pi:claude-opus-5.5
@thopkins32 thopkins32 mentioned this pull request Oct 16, 2026
@thopkins32
thopkins32 requested a review from dmgav October 16, 2026 20:20
@dmgav
dmgav merged commit 5bffdd9 into main Oct 16, 2026
12 checks passed
@dmgav
dmgav deleted the fix-tests branch October 16, 2026 20:23
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.

2 participants