Refactoring verification module by codedbyishika · Pull Request #60 · Imageomics/cautious-robot

codedbyishika · 2026-06-05T20:44:26Z

Refactor download verification module: move remaining post-download verification functions from __main__.py to buddy_check.py
Change imports
Edit AGENTS.md
Include unit tests

Copilot

Pull request overview

This PR refactors the post-download verification workflow by moving checksum generation and download verification helpers out of __main__.py into buddy_check.py, updating references/documentation accordingly, and adding unit tests to cover the moved functionality.

Changes:

Moved process_checksums() and verify_downloads() from __main__.py to buddy_check.py.
Updated tests to patch get_checksums / BuddyCheck in the new module location and added a new test suite for verification helpers.
Updated AGENTS.md to reflect the new module responsibilities and documented the new helper functions.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
`tests/test_verify_downloads.py`	Adds unit tests for `process_checksums()` and `verify_downloads()` after refactor.
`tests/test_download_images.py`	Updates patch targets to reflect moved checksum/verification code.
`src/cautiousrobot/buddy_check.py`	Adds `process_checksums()` and `verify_downloads()` and imports `get_checksums`.
`src/cautiousrobot/__main__.py`	Removes inlined checksum/verification helpers; imports them from `buddy_check.py`.
`AGENTS.md`	Updates architecture docs to reflect refactor and new helper functions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

This reverts commit 1a75a3a.

egrace479

Some notes on the test case setups.

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

- change "md5" to "hash" - specify error printouts - remove unused test

egrace479

We should probably add a test that calls verify_downloads with filename_col != "filename" to ensure something like this is caught in the future.

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

- Update docstrings from main branch - Update tests according to comments

egrace479

Please revise function descriptions as in the suggestions below.

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Add more descriptive docstrings Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

egrace479

Copilot autofix was a bit over-eager here

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

codedbyishika added 3 commits June 5, 2026 16:40

Refactor verify_downloads.py

ff9ce09

Update test calls

fb2e654

Remove unused variables

f74d66c

codedbyishika self-assigned this Jun 5, 2026

Include variable name

1186e25

codedbyishika requested a review from egrace479 June 5, 2026 20:51

Move download verification functions into buddy_check.py

08c9f03

codedbyishika linked an issue Jun 8, 2026 that may be closed by this pull request

Move download verification functions into buddy_check.py #62

Open

egrace479 requested a review from Copilot June 9, 2026 17:13

Copilot started reviewing on behalf of egrace479 June 9, 2026 17:13 View session

Copilot AI reviewed Jun 9, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

Comment thread src/cautiousrobot/buddy_check.py Outdated

Comment thread tests/test_verify_downloads.py Outdated

egrace479 reviewed Jun 9, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

egrace479 reviewed Jun 9, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

Revise error handling descriptions

9f43470

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

egrace479 reviewed Jun 9, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

egrace479 and others added 2 commits June 9, 2026 14:33

Clarify return on error

9c9a222

Potential fix for pull request finding

1a75a3a

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

egrace479 reviewed Jun 9, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py

Revert "Potential fix for pull request finding"

5ceba99

This reverts commit 1a75a3a.

egrace479 reviewed Jun 10, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py Outdated

Comment thread tests/test_verify_downloads.py

Comment thread tests/test_verify_downloads.py Outdated

Comment thread tests/test_verify_downloads.py

Comment thread tests/test_verify_downloads.py Outdated

codedbyishika and others added 3 commits June 10, 2026 17:18

Update tests/test_verify_downloads.py

7b46454

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Update tests/test_verify_downloads.py

8510b5a

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Updates for tests

0f488a0

- change "md5" to "hash" - specify error printouts - remove unused test

egrace479 requested changes Jun 10, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

codedbyishika and others added 3 commits June 11, 2026 13:11

Update src/cautiousrobot/buddy_check.py

16715be

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

Add new test to check filename

fcb7e4f

Update tests according to the change in buddy_id

b49d493

codedbyishika requested a review from egrace479 June 11, 2026 17:22

egrace479 requested a review from Copilot June 11, 2026 19:27

Copilot started reviewing on behalf of egrace479 June 11, 2026 19:28 View session

Copilot AI reviewed Jun 11, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py

Comment thread tests/test_verify_downloads.py Outdated

codedbyishika and others added 2 commits June 11, 2026 17:19

Potential fix for pull request finding

1551f1e

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Potential fix for pull request finding

23b7bf0

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

egrace479 requested a review from thompsonmj June 16, 2026 15:11

thompsonmj requested changes Jul 2, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py Outdated

codedbyishika added 2 commits July 3, 2026 13:15

Remove redundant functions

cac6e3d

Include docstrings and comments

e5b076d

thompsonmj requested changes Jul 7, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py Outdated

thompsonmj mentioned this pull request Jul 7, 2026

Handling and testing other algorithms #68

Open

egrace479 reviewed Jul 7, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py Outdated

Update tests and docstrings

7696ffa

- Update docstrings from main branch - Update tests according to comments

codedbyishika requested review from egrace479 and thompsonmj July 9, 2026 17:23

egrace479 requested changes Jul 9, 2026

View reviewed changes

codedbyishika and others added 2 commits July 10, 2026 13:02

Apply suggestions from code review

1d583f8

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

fix formatting issue

05debb6

egrace479 reviewed Jul 10, 2026

View reviewed changes

Comment thread tests/test_verify_downloads.py Outdated

Update tests/test_verify_downloads.py

c272e3a

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

egrace479 requested a review from Copilot July 10, 2026 18:23

Copilot started reviewing on behalf of egrace479 July 10, 2026 18:23 View session

Copilot AI reviewed Jul 10, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

Comment thread src/cautiousrobot/buddy_check.py

Apply suggestions from code review

1e5c0dc

Add more descriptive docstrings Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

codedbyishika requested a review from egrace479 July 10, 2026 19:11

egrace479 requested changes Jul 10, 2026

View reviewed changes

Comment thread src/cautiousrobot/buddy_check.py Outdated

Comment thread src/cautiousrobot/buddy_check.py Outdated

Comment thread src/cautiousrobot/buddy_check.py Outdated

Apply suggestions from code review

9375ec2

Co-authored-by: Elizabeth Campolongo <38985481+egrace479@users.noreply.github.com>

codedbyishika requested a review from egrace479 July 10, 2026 22:51

Uh oh!

Conversation

codedbyishika commented Jun 5, 2026 • edited by egrace479 Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

egrace479 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

codedbyishika commented Jun 5, 2026 •

edited by egrace479

Loading