librelist archives

« back to archive

Commit: features/metrics-batch: avoid creating clutter

Commit: features/metrics-batch: avoid creating clutter

From:
Lucas Kanashiro
Date:
2013-10-16 @ 21:28
Terceiro, when we runned the tests some clutter was created.

This files was created:

99-details.csv
t/samples/animals/cpp-details.csv
t/sample/animals/java-details.csv

This is in the branch master of Analizo.

Re: [analizo] Commit: features/metrics-batch: avoid creating clutter

From:
Antonio Terceiro
Date:
2013-10-18 @ 00:19
On Wed, Oct 16, 2013 at 06:28:09PM -0300, Lucas Kanashiro wrote:
> Terceiro, when we runned the tests some clutter was created.
> 
> This files was created:
> 
> 99-details.csv
> t/samples/animals/cpp-details.csv
> t/sample/animals/java-details.csv
> 
> This is in the branch master of Analizo.

Yeah, your code is doing that. ;-)

I just pushed a fix mocking the write_details method in the
unit tests. Please review commit 0790625b.

-- 
Antonio Terceiro <terceiro@softwarelivre.org>
http://softwarelivre.org/terceiro

Re: [analizo] Commit: features/metrics-batch: avoid creating clutter

From:
Lucas Kanashiro
Date:
2013-10-21 @ 12:28
We add some acceptance tests and don't add unit tests, in comparison with
the branch master of Analizo, but even creating clutter. We tested only
acceptance test and was not generated clutter, we tested only unit test and
was not generated clutter, but when we runned all the tests the clutter was
generated.


2013/10/17 Antonio Terceiro <terceiro@softwarelivre.org>

> On Wed, Oct 16, 2013 at 06:28:09PM -0300, Lucas Kanashiro wrote:
> > Terceiro, when we runned the tests some clutter was created.
> >
> > This files was created:
> >
> > 99-details.csv
> > t/samples/animals/cpp-details.csv
> > t/sample/animals/java-details.csv
> >
> > This is in the branch master of Analizo.
>
> Yeah, your code is doing that. ;-)
>
> I just pushed a fix mocking the write_details method in the
> unit tests. Please review commit 0790625b.
>
> --
> Antonio Terceiro <terceiro@softwarelivre.org>
> http://softwarelivre.org/terceiro
>
>
>