Update MultiMap contract and implementations so the setting a null value removes the entry from the MultiMap. Tests have been cleaned up and extended so that all MultiMap implementations are equally tested. - closes #3267

This commit is contained in:
Julien Viet
2020-01-29 15:06:58 +01:00
parent 93645d9758
commit 67011f027b

View File

@@ -193,9 +193,9 @@ public interface MultiMap extends Iterable<Map.Entry<String, String>> {
MultiMap addAll(Map<String, String> headers);
/**
* Sets a value under the specified name.
* Sets a {@code value} under the specified {@code name}.
* <p>
* If there is an existing header with the same name, it is removed.
* If there is an existing header with the same name, it is removed. Setting a {@code null} value removes the entry.
*
* @param name The name
* @param value The value