[wildfly-dev] Tests for incomplete bugs

Panagiotis Sotiropoulos psotirop at redhat.com
Thu Apr 19 04:10:18 EDT 2018


+1 (In cases of 1 to 1 server code and testcases)

On Thu, Apr 19, 2018 at 9:52 AM, Carlo de Wolf <cdewolf at redhat.com> wrote:

> It would prove the test code is valid. I have seen some false positives. :)
>
> So having an independent run of the test is nice, but it must end up in
> the same PR as the fix.
>
> Filing a test-only PR isn't very pretty, because it creates smoke for the
> mergers.
>
> Carlo
>
>
> On 18-04-18 14:54, Stuart Douglas wrote:
>
> In the cases I am taking about the is no point even testing on CI, as it
> is already known that the underling bug is not fixed yet.
>
> Stuart
>
> On Wed, 18 Apr. 2018, 9:05 pm Darran Lofthouse, <
> darran.lofthouse at redhat.com> wrote:
>
>> +1
>>
>> If multiple engineers are working on the same problem they should set up
>> their own topic branch (can just be in one of their personal repos)
>> somewhere and collaborate together then submit a combined PR from that
>> branch that contains both the fix and the test case.
>>
>> Sometimes I think PRs are used as an easy way to get branches into CI, we
>> should probably look again at how easy it is to get topic branches into CI
>> without relying on the PR process to get jobs running.
>>
>> Regards,
>> Darran Lofthouse.
>>
>>
>> On Wed, 18 Apr 2018 at 02:12 Stuart Douglas <stuart.w.douglas at gmail.com>
>> wrote:
>>
>>> Hi Everyone,
>>>
>>> Historically we have been ok with pull requests being submitted that add
>>> tests for things that are not yet complete.
>>>
>>> In practice this has been kinda problematic for reviewers, as these pull
>>> requests can hang around for a long time, and this ends up wasting a lot of
>>> reviewer time as we have to continually check the status of the PR. You
>>> basically end up with multiple reviewers continually polling the issue to
>>> check if it is ready, instead of a single person just submitting it when it
>>> is complete.
>>>
>>> I think that there is no real benefit to having the tests as a seperate
>>> PR that gets opened before the fix. Instead I propose that tests that do
>>> not pass yet are linked/co-ordinated under the relevant JIRA, and when the
>>> fix is submitted the tests can be submitted as part of the same PR.
>>>
>>> Does anyone have any objections to this?
>>>
>>> Stuart
>>> _______________________________________________
>>> wildfly-dev mailing list
>>> wildfly-dev at lists.jboss.org
>>> https://lists.jboss.org/mailman/listinfo/wildfly-dev
>>
>>
>
> _______________________________________________
> wildfly-dev mailing listwildfly-dev at lists.jboss.orghttps://lists.jboss.org/mailman/listinfo/wildfly-dev
>
>
>
> _______________________________________________
> wildfly-dev mailing list
> wildfly-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/wildfly-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/wildfly-dev/attachments/20180419/6f62efdd/attachment.html 


More information about the wildfly-dev mailing list