kafka-jira mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Guozhang Wang (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (KAFKA-5676) MockStreamsMetrics should be in o.a.k.test
Date Mon, 07 Aug 2017 20:55:00 GMT

    [ https://issues.apache.org/jira/browse/KAFKA-5676?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16117249#comment-16117249

Guozhang Wang commented on KAFKA-5676:

Could you list the test cases that do depend on the {{Metrics}} object? If they do depend
on the metrics then perhaps they should not use a mocked StreamsMetrics.

> MockStreamsMetrics should be in o.a.k.test
> ------------------------------------------
>                 Key: KAFKA-5676
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5676
>             Project: Kafka
>          Issue Type: Bug
>          Components: streams
>            Reporter: Guozhang Wang
>            Assignee: Chanchal Singh
>              Labels: newbie
> {{MockStreamsMetrics}}'s package should be `o.a.k.test` not `o.a.k.streams.processor.internals`.

> In addition, it should not require a {{Metrics}} parameter in its constructor as it is
only needed for its extended base class; the right way of mocking should be implementing {{StreamsMetrics}}
with mock behavior than extended a real implementaion of {{StreamsMetricsImpl}}.

This message was sent by Atlassian JIRA

View raw message