Skip to content

Catalist testing#1842

Merged
shaunagm merged 11 commits into
move-coop:mainfrom
bmos:catalist-testing
Apr 16, 2026
Merged

Catalist testing#1842
shaunagm merged 11 commits into
move-coop:mainfrom
bmos:catalist-testing

Conversation

@bmos
Copy link
Copy Markdown
Collaborator

@bmos bmos commented Apr 15, 2026

What is this change?

  • Refactor catalist testing to use real files on disk
  • Increase coverage of catalist connector
  • Sync ruff version between pyproj and pre-commit (it got out of sync during a recent PR, resulting in the pre-commit job and ruff-format job sometimes having different outputs.

Considerations for discussion

How to test the changes (if needed)

  • (How should a reviewer test this functionality.)

Breaking Changes

Breaking changes are changes to our public API which may require existing users to change their code. If there are no breaking changes, any existing parsons user should not need to do anything after updating their parsons version.

Does this PR introduce breaking changes?
  • label: Breaking change — This PR introduces one or more breaking changes.
  • label: Non-breaking change — This PR does not introduce one or more breaking changes.

Details (if needed)

  • (List out any changes to the API that may cause breaks for developer implementation.)

@bmos
Copy link
Copy Markdown
Collaborator Author

bmos commented Apr 15, 2026

Oops, seems like updating from main broke something.

Post required changes to log
@bmos bmos changed the base branch from ian/patch/catalist-zip-operation to main April 16, 2026 01:17
@bmos
Copy link
Copy Markdown
Collaborator Author

bmos commented Apr 16, 2026

Okay, this is ready to be merged @shaunagm.
Once it is, @IanRFerguson's PR should be updated from main and I think it'll pass all the tests.

@github-actions
Copy link
Copy Markdown

ghost commented Apr 16, 2026

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  parsons/catalist
  catalist.py
Project Total  

This report was generated by python-coverage-comment-action

@shaunagm shaunagm merged commit b8b1e5d into move-coop:main Apr 16, 2026
92 checks passed
@bmos bmos deleted the catalist-testing branch April 16, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

non-breaking-change Status - Indicates that the code in this PR does not have any breaking changes. python Pull requests that update Python code testing Work type - writing or changing code tests for core Parsons features or Parsons connectors

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants