Rename RSocketStrategies.Builder.metadataExtractors

This commit renames the `metadataExtractors` method to something that is
closer to the actual intent: addming new metadata extractors against the
registry given as a parameter of the `Consumer`.

The method is renamed to `metadataExtractorRegistry`.
This commit is contained in:
Brian Clozel 2019-09-19 14:54:36 +02:00
parent e06d2a9c57
commit 809009e6c4
3 changed files with 3 additions and 3 deletions

View File

@ -204,7 +204,7 @@ final class DefaultRSocketStrategies implements RSocketStrategies {
} }
@Override @Override
public Builder metadataExtractors(Consumer<MetadataExtractorRegistry> consumer) { public Builder metadataExtractorRegistry(Consumer<MetadataExtractorRegistry> consumer) {
this.metadataExtractors.add(consumer); this.metadataExtractors.add(consumer);
return this; return this;
} }

View File

@ -223,7 +223,7 @@ public interface RSocketStrategies {
* Apply the consumer to the {@link MetadataExtractorRegistry} in order * Apply the consumer to the {@link MetadataExtractorRegistry} in order
* to register extra metadata entry extractors. * to register extra metadata entry extractors.
*/ */
Builder metadataExtractors(Consumer<MetadataExtractorRegistry> extractorRegistry); Builder metadataExtractorRegistry(Consumer<MetadataExtractorRegistry> consumer);
/** /**
* Build the {@code RSocketStrategies} instance. * Build the {@code RSocketStrategies} instance.

View File

@ -110,7 +110,7 @@ class DefaultRSocketStrategiesTests {
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")
void applyMetadataExtractors() { void applyMetadataExtractors() {
Consumer<MetadataExtractorRegistry> consumer = (Consumer<MetadataExtractorRegistry>) mock(Consumer.class); Consumer<MetadataExtractorRegistry> consumer = (Consumer<MetadataExtractorRegistry>) mock(Consumer.class);
RSocketStrategies strategies = RSocketStrategies.builder().metadataExtractors(consumer).build(); RSocketStrategies strategies = RSocketStrategies.builder().metadataExtractorRegistry(consumer).build();
verify(consumer, times(1)).accept(any()); verify(consumer, times(1)).accept(any());
} }