test(core): add Box<dyn Error> test for XmlError

[anidroid1184]

Contribution Checklist

• ⭐ I have starred the repository
• 🔀 I have forked the repository and created a new branch for my changes
• 📖 I have read the CONTRIBUTING.md guide

Description

Implement XmlError → std::error::Error trait implementation (if missing)

Related Issue

Closes #25

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 💥 Breaking change (fix or feature that would cause existing functionality to change)
• 📝 Documentation update
• 🔧 Refactor (code change that neither fixes a bug nor adds a feature)
• ⚡ Performance improvement
• ✅ Test addition or update
• 🏗️ CI/CD or build system change

Checklist

• I have added or updated tests that prove my fix is effective or my feature works
• cargo clippy --all-targets -- -D warnings passes with no warnings
• cargo fmt --check passes with no formatting issues
• cargo test --all-targets passes
• cargo test --doc passes
• I have updated documentation as needed (rustdoc, README, etc.)
• I have updated the changelog (if applicable)
• My changes do not introduce any new unsafe code (or it is properly justified and documented)

Additional Notes

Summary by CodeRabbit

• Tests
  • Added coverage to verify XML errors work correctly with the standard error interface, including message formatting and error-source access.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 397a3a75-1854-4227-827f-8edbb573006a

📥 Commits

Reviewing files that changed from the base of the PR and between 3b834cb and 71179ed.

📒 Files selected for processing (1)

• crates/tinyxml2/src/error.rs

---

📝 Walkthrough

Walkthrough

Adds a std-only unit test confirming that XmlError::NoAttribute can be boxed as Box<dyn std::error::Error> and queried through to_string() and source().

Changes

XmlError validation

Layer / File(s)	Summary
Boxed Error trait-object test crates/tinyxml2/src/error.rs	Adds xml_error_as_box_dyn_error, which validates Display and Error::source() through a boxed standard error trait object.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Linked Issues check	❓ Inconclusive	The summary only shows a new test, so the required Display/Error implementations and warning-free verification from #25 can't be confirmed.	Provide the full diff or summarize the trait-impl changes and test/clippy results so #25 can be verified.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title is concise and accurately describes the main change: adding a Box test for XmlError.
Out of Scope Changes check	✅ Passed	The change appears focused on XmlError error interoperability testing and does not introduce unrelated scope.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}