docs: explain model fixture and snapshot workflow - #1031
Conversation
Signed-off-by: Shurong Cao <170531907+CAOShurong@users.noreply.github.com>
Up to standards ✅🟢 Issues
|
jkowalleck
left a comment
There was a problem hiding this comment.
Thanks for the effort.
more docs -> more good!
Describe the exact string comparison used by SnapshotMixin without implying a raw byte comparison. Assisted-by: OpenAI Codex <noreply@openai.com> Signed-off-by: Shurong Cao <170531907+CAOShurong@users.noreply.github.com>
|
@jkowalleck, the byte-for-byte wording was corrected in |
|
looks like there are 2 PRs opened by you, @CAOShurong , for the same scope/topic:
please consolidate the content in just one PR, and close the other. |
Signed-off-by: Shurong Cao <170531907+CAOShurong@users.noreply.github.com>
Documentation build overview
|
|
thank you for taking care of the docs 🥇 |
Description
Document the shared BOM model fixture workflow in
tests/_data/snapshots/README.md:get_bom_*factories are discovered and named;The existing README only documented the recreation command. This fills the documentation gap identified by the maintainer in #914 without changing test behavior.
Resolves or fixes issue: #1030
Validation
python -m unittest tests.test_output_json tests.test_output_xml tests.test_deserialize_json tests.test_deserialize_xml— 750 tests passed.python -m unittest discover -t . -s tests— 6,962 tests passed.git diff --check— passed.AI Tool Disclosure
OpenAI CodexGPT-5Audit maintainer-requested CycloneDX contribution paths, verify repository rules and duplicates, document the existing tests/_data/models.py fixture and snapshot workflow, and run focused and complete regression tests.Affirmation