test: increase pydantic\aliases.py coverage to 100%

[AlessandroMiola]

Change Summary

Add a tiny case to increase coverage of pydantic\aliases.py to 100%.
Tests for forbidden string indexing of AliasPath argument via the search_dict_for_path utility.

note: I thought that the newly added assertions could fit into the existing test_validation_alias_parse_data, therefore I didn't add a new test function. Happy to change if you think otherwise (or get guidance in any case)!

Related issue number

Contributes to #7656.

Checklist

• The pull request title is a good summary of the changes - it will be used in the changelog
• Unit tests for the changes exist
• Tests pass on CI
• Documentation reflects the changes where applicable
• My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

[codspeed-hq]

CodSpeed Performance Report

Merging #9740 will not alter performance

_{Comparing AlessandroMiola:aliases-increase-test-cov (c4c6fec) with main (941c021)}

Summary

✅ 13 untouched benchmarks

[sydney-runkle]

Hi @AlessandroMiola,

Thanks for your contribution here!

Could we extract this logic into a separate test, maybe called test_search_dict_for_path or something similar? I think that'll help to keep things organized.

Awesome work on increasing coverage here!

[sydney-runkle]

As a side note, I wonder if we can add a more clear coverage integration to PRs that adds a comment like the codspeed integration to celebrate increases in test coverage + warn against regressions :). I suppose the CI job is fine for now, but if anyone is interested, this could be a cool CI focused task.

[AlessandroMiola]

As a side note, I wonder if we can add a more clear coverage integration to PRs that adds a comment like the codspeed integration to celebrate increases in test coverage + warn against regressions :). I suppose the CI job is fine for now, but if anyone is interested, this could be a cool CI focused task.

It would be nice, indeed. I'll try to dig into it :)