[Hardik K (hardikkaurani)]

Description

Fixes #208

This PR addresses the usability improvements requested for HeadersCollection:

1. Dictionary and HeadersCollection Constructor Initialization:
   HeadersCollection now accepts an optional initial headers parameter (Optional[Union[HeadersCollection, Mapping[str, Union[str, Iterable[str]]]]]), allowing direct instantiation from a dictionary or copy-construction from another HeadersCollection.
2. add_all Dictionary Support:
   add_all now accepts either a HeadersCollection or any Mapping (dict) of header names to values (Union[str, Iterable[str]]), matching the docstring expectation and easing integration across callers.
3. Python Container / Mapping Protocol:
   Implements standard Python container magic methods:
   • __contains__: supports "header" in headers
   • __getitem__: supports headers["header"]
   • __setitem__: supports headers["header"] = "value"
   • __len__: supports len(headers)
   • __iter__: supports iterating header keys directly

Related Issue

Fixes #208

Testing

Added unit tests in packages/abstractions/tests/test_request_header.py:

• test_init_with_dict
• test_init_with_headers_collection
• test_add_all_with_dict
• test_container_methods

cc Vincent Biret (@baywet)

[Hardik K (hardikkaurani)]

Hi Vincent Biret (@baywet), submitted this PR to resolve #208 by enabling dictionary and HeadersCollection constructor initialization, mapping support in add_all, and Python container protocol magic methods (in, [], len(), iter()). Comprehensive unit tests added in test_request_header.py. Ready for your review, thank you!

[Copilot]

🟡 Changes recommended

Empty-input handling, replacement assignment, and single-value header imports have unresolved correctness issues.

3 open issues

• Reject only None so empty configurations and headers are accepted · New
• Apply single-value header handling to iterable initialization · New
• Replace existing header values on subscript assignment · New

_{Review effort: Balanced}

[github-actions]

This pull request has conflicting changes, the author must resolve the conflicts before this pull request can be merged.

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[github-actions]

Conflicts have been resolved. A maintainer will take a look shortly.

[Hardik K (hardikkaurani)]

Hi Vincent Biret (@baywet)!

I have resolved the merge conflicts with \main\ and addressed all review feedback:

1. Empty-Input Handling:

   • Updated \�dd_all\ to only reject \None\ (\if headers is None:), allowing empty \HeadersCollection\ instances and empty mappings to complete cleanly as no-ops.
   • Updated \ est_request_information.py:29\ to an explicit initialization check (\�ssert request_info.headers is not None).
   • Added unit tests in \ est_request_header.py\ verifying initialization and \�dd_all\ with empty mappings/collections.

2. Iterable Initialization & Single-Value Headers:

   • Materialized iterable values into a list when delegating to \self.add(key, list_values)\ so that single-value headers (e.g., \Content-Length) properly retain only the first value.
   • Added regression test \ est_init_iterable_single_value_header.

3. Subscript Assignment Replacement:

   • Implemented _setitem_\ to replace existing header values on assignment (case-insensitively) rather than appending, and validates the replacement before mutating stored headers. Also added _delitem_.
   • Added tests covering repeated assignments, case-insensitivity, validation, and single-value header assignment.

All local quality gates and checks (\pytest\ 168/168, \mypy\ 0 issues in 48 files, \pylint\ 10.00/10, \yapf, \isort) pass cleanly. Ready for review!

[Copilot]

🟢 Approval recommended

Previously reported regressions are addressed with regression tests, and no blocking issues remain.

No open issues 🎉

3 resolved since last review

• Replace existing header values on subscript assignment
• Apply single-value header handling to iterable initialization
• Reject only None so empty configurations and headers are accepted

_{Review effort: Balanced}

[Vincent Biret (baywet)]

Thanks for the contribution!

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[sonarqubecloud]

Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud