Improve test coverage for tools, float, binary, geo and todo - issues…

[shiva-kumar04]

… 698

Closes issue

• contributes to Python: Improve test coverage in 5 places for production code #698

Description

Improves test coverage by adding tests for previously uncovered branches. Improves coverage from 96% to 98%

Focus areas:

• normalize_pytz
• vFloat.ical_value
• vBinary.hash
• vGeo.hash
• ToDo.duration

Checklist

• I've added a change log entry to CHANGES.rst.
• I've added or updated tests if applicable.
• I've run and ensured all tests pass locally by following Run tests.
• I've added or edited documentation, both as docstrings to be rendered in the API documentation and narrative documentation, as necessary.

Additional information

beforeCoverage.html

afterCoverage.html

[read-the-docs-community]

Documentation build overview

📚 icalendar | 🛠️ Build #31906853 | 📁 Comparing 393210b against latest (fae9f6e)

🔍 Preview build

Show files changed (1 files in total): 📝 1 modified | ➕ 0 added | ➖ 0 deleted

File	Status
404.html	📝 modified

[niccokunzmann]

Thanks, I added a few comments.
Also, please format the code: https://github.com/collective/icalendar/actions/runs/23376081732/job/68008197434?pr=1291

[niccokunzmann]

Suggested change

	def test_normalize_pytz():
	def test_normalize_pytz(pytz_only):
	import pytz

Hi, icalendar also works without pytz. This could be a solution..

[niccokunzmann]

Suggested change

	@pytest.mark.parametrize("Component", [Todo])
	@pytest.mark.parametrize("Component", [Todo, Event])

Nice! Does this also work with Event?