[PATCH 2 of 2] tests: make run-tests.py invokes tests with needed variables only (issue5069)

Pierre-Yves David pierre-yves.david at ens-lyon.org
Sun Mar 27 15:36:30 EDT 2016



On 03/26/2016 04:45 AM, liscju wrote:
> # HG changeset patch
> # User liscju <piotr.listkiewicz at gmail.com>
> # Date 1458986707 -3600
> #      Sat Mar 26 11:05:07 2016 +0100
> # Node ID 19a08baae8794c77f4876161ea4bbe3f5cb6b6ff
> # Parent  ffe9bc7aea5a63bc82e7322da9ff3448f13abaec
> tests: make run-tests.py invokes tests with needed variables only (issue5069)
>
> Before this patch tests were invoked with all environment variables
> except fixed set of known problematic environment variables. This
> commit makes run-tests.py pass to tests only that variables that
> are actually used in run-tests.py
>
> This commit is only proposal of solution. Class rememberusedkeyenv
> is not implementing all dictionary methods, only ones that are
> actually used in run-tests.py when accessing environment variables.
>
> Other possible solution would be to make a fixed list of used
> environment variables in run-tests.py.
>
> I am looking forward for suggestions how this should be implemented.

This magic object seems too magical to me. I would prefer to have an 
explicit list of white listed environment variable.

Keep also in mind that the test runner is used by other third party 
extension that may have there own need for environment variable. What is 
the current story for them to expend this list? We should probably make 
the run-tests.py test checks for this.

Cheers,

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list