Re: [MNG-6130] Loss of profile information in workaround for MNG-4900

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [MNG-6130] Loss of profile information in workaround for MNG-4900

Michael Osipov-2
Am 2017-05-15 um 22:29 schrieb Michael Osipov:
> Am 2017-05-15 um 22:10 schrieb Robert Scholte:
>> I think I miss a unittest or integration-test, just to be sure.
>
> The reporter says: "It's very tricky and hopefully not necessary, since
> the 1-line fix is provided"

The issue reporter isn't able to provide an IT and there is only the
patch, but no sample project for this issue.

What to do? Merge this oneline fix anyway?


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [MNG-6130] Loss of profile information in workaround for MNG-4900

Michael Osipov-2
Am 2017-07-02 um 20:13 schrieb Robert Scholte:
> So it's back to us, which means we should write the UT/IT if we want to
> apply the patch.
>
> As long as there's no proof that this patch is required, I'm -1

Sharing the view. I will reask the reporter.

> On Sun, 02 Jul 2017 18:57:07 +0200, Michael Osipov <[hidden email]>
> wrote:
>
>> Am 2017-05-15 um 22:29 schrieb Michael Osipov:
>>> Am 2017-05-15 um 22:10 schrieb Robert Scholte:
>>>> I think I miss a unittest or integration-test, just to be sure.
>>>
>>> The reporter says: "It's very tricky and hopefully not necessary, since
>>> the 1-line fix is provided"
>>
>> The issue reporter isn't able to provide an IT and there is only the
>> patch, but no sample project for this issue.
>>
>> What to do? Merge this oneline fix anyway?
>>
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [hidden email]
>> For additional commands, e-mail: [hidden email]
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [hidden email]
> For additional commands, e-mail: [hidden email]
>
>


---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Loading...