Hi Udi,
Are the temporary pytest tasks in use for pre-commit check or anything
currently?
I see there is still WIP for BEAM-3713
<https://issues.apache.org/jira/browse/BEAM-3713>.
There is only one task "pythonPreCommitPytest" depending on the pytest
tasks using the pytest environment configs.
And it's invoked here:
PrecommitJobBuilder builderPytest = new PrecommitJobBuilder(
scope: this,
nameBase: 'Python_pytest',
gradleTask: ':pythonPreCommitPytest',
commitTriggering: false,
timeoutMins: 180,
)
builderPytest.build {...}
On Wed, Dec 4, 2019 at 5:51 PM Ning Kang <ni...@google.com> wrote:
> Thanks for the heads up! I was wondering why the interactive tests are
> skipped, lol.
> So we are moving away from the deprecated pytest-runner (with the changes
> in setup.py) but still sticking to pytest since it's replacing nosetest.
>
> Can I add "interactive" as "extras" to testenv "py37-pytest" and
> "py36-pytest" in tox.ini
> <https://github.com/apache/beam/blob/master/sdks/python/tox.ini#L100>
> then?
>
> @Ahmet Altay <al...@google.com> fyi
>
> On Wed, Dec 4, 2019 at 5:22 PM Pablo Estrada <pabl...@google.com> wrote:
>
>> +Ning Kang <ni...@google.com> +Sam Rohde <sro...@google.com> fyi
>>
>> On Wed, Nov 27, 2019 at 5:09 PM Udi Meiri <eh...@google.com> wrote:
>>
>>> As part of a move to stop using the deprecated (and racey) setup.py
>>> keywords setup_requires and test_require, interactive runner dependencies
>>> have been removed from tests in
>>> https://github.com/apache/beam/pull/10227
>>>
>>> If this breaks any tests, please let me know.
>>>
>>
