You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by uğur <yt...@gmail.com> on 2019/09/17 19:53:55 UTC

About Testing Stream Applications Documentation

Hi,

I am not sure if it is the right email address to write about this topic,
please correct me if I am wrong.

As I read documentation of Testing Stream Application, I believe I noticed
wrong naming of method called shouldNotUpdateStoreForLargerValue. As I
understand it, either the name should change
to shouldUpdateStoreForLargerValue or assertion should be different.

Regards,
Ugur Yeter

Re: About Testing Stream Applications Documentation

Posted by uğur <yt...@gmail.com>.
Hi Bruno,

Let me see if it is possible to edit it for me.

Regards,
Ugur Yeter

On Thu, 19 Sep 2019, 00:48 Bruno Cadonna, <br...@confluent.io> wrote:

> Hi Ugur,
>
> Your finding looks correct to me. Do you mind fixing this issue?
>
> Best,
> Bruno
>
> On Tue, Sep 17, 2019 at 12:54 PM uğur <yt...@gmail.com> wrote:
> >
> > Hi,
> >
> > I am not sure if it is the right email address to write about this topic,
> > please correct me if I am wrong.
> >
> > As I read documentation of Testing Stream Application, I believe I
> noticed
> > wrong naming of method called shouldNotUpdateStoreForLargerValue. As I
> > understand it, either the name should change
> > to shouldUpdateStoreForLargerValue or assertion should be different.
> >
> > Regards,
> > Ugur Yeter
>

Re: About Testing Stream Applications Documentation

Posted by Bruno Cadonna <br...@confluent.io>.
Hi Ugur,

Your finding looks correct to me. Do you mind fixing this issue?

Best,
Bruno

On Tue, Sep 17, 2019 at 12:54 PM uğur <yt...@gmail.com> wrote:
>
> Hi,
>
> I am not sure if it is the right email address to write about this topic,
> please correct me if I am wrong.
>
> As I read documentation of Testing Stream Application, I believe I noticed
> wrong naming of method called shouldNotUpdateStoreForLargerValue. As I
> understand it, either the name should change
> to shouldUpdateStoreForLargerValue or assertion should be different.
>
> Regards,
> Ugur Yeter