Skip to content

Add bitspack tests and pytest dev dependency - #210

Merged
hzhangxyz merged 3 commits into
mainfrom
copilot/add-tests-bitspack
Mar 5, 2026
Merged

Add bitspack tests and pytest dev dependency#210
hzhangxyz merged 3 commits into
mainfrom
copilot/add-tests-bitspack

Conversation

Copilot AI commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Restores tests/bitspack_test.py (removed in b47b47f) with code style updated to match the existing test conventions, and adds pytest to the dev optional dependencies.

Changes

  • tests/bitspack_test.py: Re-added bitspack round-trip tests, updated to current conventions:
    • Import from qmp.utility.bitspack (was qmb.bitspack)
    • Tests grouped into classes (TestPackUnpackIntSize1/2/4/8) with per-method docstrings, matching hamiltonian_test.py style
  • pyproject.toml: Added pytest>=8.4.1 to [project.optional-dependencies] dev
Original prompt

This section details on the original issue you should resolve

<issue_title>添加测试</issue_title>
<issue_description>将 b47b47f^ 的 tests/bitspack_test.py 添加回来, 但需要更新代码风格, 保持和现有的一致. 另外, 把pytest加dev optional dep里
</issue_description>

Comments on the Issue (you are @copilot in this section)


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

Co-authored-by: hzhangxyz <11623447+hzhangxyz@users.noreply.github.com>
Copilot AI changed the title [WIP] Add tests for bitspack with updated code style Add bitspack tests and pytest dev dependency Mar 5, 2026
@hzhangxyz
hzhangxyz marked this pull request as ready for review March 5, 2026 22:39
Copilot AI review requested due to automatic review settings March 5, 2026 22:39
@hzhangxyz
hzhangxyz merged commit 353b7f6 into main Mar 5, 2026
Copilot AI removed the request for review from Copilot March 23, 2026 23:31
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