> On 29 Mar 2019, at 10:37, Vincent Massol <vinc...@massol.net> wrote:
>
>
>
>> On 29 Mar 2019, at 10:33, Simon Urli <simon.u...@xwiki.com> wrote:
>>
>> Hi Vincent,
>>
>> On 29/03/2019 10:27, Vincent Massol wrote:
>>> Hi devs,
>>> I'd like to discuss about introducing a checker in the tests to fail the
>>> test if there's a warning message about a deprecated APIs being used in
>>> scripts.
>>> For example:
>>> ```
>>> 23:59:28.308 [main] INFO org.xwiki.test.ui.TestDebugger -
>>> GroupIT-addUserAndSubgroupToGroup started
>>> 23:59:32.593 [Exec Stream Pumper] ERROR o.x.t.i.XWikiLogOutputStream -
>>> 2019-03-28 23:59:32,593
>>> [http://localhost:8080/xwiki/bin/view/XWiki/XWikiPreferences?xpage=getgroups&offset=1&limit=15&reqNo=1]
>>> WARN o.x.v.i.DefaultVelocityEngine - Deprecated usage of method
>>> [com.xpn.xwiki.plugin.rightsmanager.RightsManagerPluginApi.countAllMembersNamesForGroup]
>>> in 21:/templates/getgroups.vm@62,37
>>> 23:59:35.824 [Exec Stream Pumper] ERROR o.x.t.i.XWikiLogOutputStream -
>>> 2019-03-28 23:59:35,824
>>> [http://localhost:8080/xwiki/bin/view/XWiki/XWikiPreferences?xpage=getgroups&offset=1&limit=15&reqNo=2]
>>> WARN o.x.v.i.DefaultVelocityEngine - Deprecated usage of method
>>> [com.xpn.xwiki.plugin.rightsmanager.RightsManagerPluginApi.countAllMembersNamesForGroup]
>>> in 18:/templates/getgroups.vm@62,37
>>> 23:59:41.349 [Exec Stream Pumper] ERROR o.x.t.i.XWikiLogOutputStream -
>>> 2019-03-28 23:59:41,348
>>> [http://localhost:8080/xwiki/bin/view/XWiki/XWikiPreferences?xpage=getgroups&offset=1&limit=15&reqNo=3]
>>> WARN o.x.v.i.DefaultVelocityEngine - Deprecated usage of method
>>> [com.xpn.xwiki.plugin.rightsmanager.RightsManagerPluginApi.countAllMembersNamesForGroup]
>>> in 21:/templates/getgroups.vm@62,37
>>> 23:59:58.503 [main] INFO org.xwiki.test.ui.TestDebugger -
>>> GroupIT-addUserAndSubgroupToGroup passed
>>> ```
>>> Rationale:
>>> * This adds warnings in the xwiki logs when users navigate to those pages
>>> which isn’t nice.
>>> * It also helps reducing the number of deprecated methods we use (I have
>>> the feeling this is not reducing) and helps us move towards being able to
>>> move the deprecated code to legacy.
>>> WDYT?
>>
>> Is it possible to start by doing this in a specific job and see the result?
>> If there's not much to fix then we can fix them and add this to the main
>> build.
>>
>> I'm a bit afraid that we reproduce what happens with wcag: lots of failing
>> test coming and not much bandwidth to fix them all so starting to "accept"
>> failing tests where we shouldn’t.
>
> We can do what we usually do for checks: we have the ability to
> enable/disable them per module. So the strategy is:
>
> * Enable the check by default
> * Set it to not execute for all modules
> * Check which modules pass when it’s enabled and remove the property from
> their pom.xml. This allows to no longer have deprecation regressions in that
> module.
> * Then start fixing the deprecations module per module and remove the
> property there too.
Forget to mention that this strategy allows new modules to have the check on by
default.
Thanks
-Vincent
>
> But first, we need to decide if we agree about doing this. Do you agree?
>
> Thanks
> -Vincent
>
>>
>> Simon
>>> Thanks
>>> -Vincent
>>
>> --
>> Simon Urli
>> Software Engineer at XWiki SAS
>> simon.u...@xwiki.com
>> More about us at http://www.xwiki.com