Re: [Tails-dev] Adding "look at test suite results" to our …

Delete this message

Reply to this message
Autore: anonym
Data:  
To: The Tails public development discussion list
Oggetto: Re: [Tails-dev] Adding "look at test suite results" to our reviewing guidelines
intrigeri:
> Hi!
>
> Recently a branch was merged, that introduced a regression in our test
> suite, which Jenkins was able to spot. But for some reason, we noticed
> that regression only *after* the branch was merged into stable & devel.
>
> I think that one of the key features of our test suite, by design [1],
> is to allow us to notice such issues *before* we merge topic branches.
>
> [1] https://tails.boum.org/blueprint/automated_builds_and_tests/automated_tests_specs/
>
> So, until we have Jenkins results integrate with Redmine somehow,
> IMO we should add to our reviewing guidelines [2] something like
> "look at automated tests results on Jenkins for the proposed branch,
> and make sure it introduces no regression; but our Jenkins setup does
> not run 100% of the test suite, so bonus points if you feel like doing
> a complete test suite run locally".


+1. IMHO those of us with access to jenkins should also make sure to
check the tests before asking for a review. Clearly I failed at that
this time (but I have done it at other times, so it was just a slip!),
so the blame is on me, really! :S

Cheers!