aperture-data / aperture-data/aperturedb-python

Fix failing test test_issue_181_example in test_ResponseHandler.py

Open
#733 1 comment 0 reactions 1 assignee View on GitHub

@ad-claw000 is already working on this.

Since May 21, 2026.

Dominant language
Python
Stars
13
Forks
4
No merged PRs in 30d

Description

There is a bug in the mocked response of test_issue_181_example.

As pointed out by Copilot in this comment:

In mock_query, the loop breaks after 2 commands (one FindImage+UpdateImage pair), so only one image can be appended to changed_ids. The test currently asserts len(changed_ids) == 2, which doesn't match the mocked behavior and will fail; either adjust the mock to return responses for 2 pairs (e.g., break after 4 commands) or change the assertion to expect 1.

We need to fix this either by adjusting the mock to return 4 commands or by expecting a length of 1 in the assertion.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.