Hi Benjamin, AFAIK nose is only used for integration tests (unit tests were converted to pytest a while back). These ITs should all be running periodically (except maybe the release related ones?).
I would start with selecting one of the Jenkins jobs and converting the ITs in it to pytest. Good place to start: https://ci-beam.apache.org/job/beam_PreCommit_Python_Cron/ I would prioritize converting the Python jobs listed here: https://github.com/apache/beam/blob/master/.github/PULL_REQUEST_TEMPLATE.md There's a fairly old abandoned PR with some ideas: https://github.com/apache/beam/pull/7949/files Have a look at: sdks/python/scripts/run_integration_test.sh sdks/python/pytest.ini sdks/python/conftest.py My idea in that PR was to replace the nose @attr('IT') decorators with 1 or more: @pytest.mark.it_postcommit, @pytest.mark.no_direct, etc. These decorators tell nose/pytest which tests to run. So if I wanted to run post-commit tests on direct runner I would use this pytest flag: "-m 'it_postcommit and not no_direct'". On Wed, Mar 24, 2021 at 5:41 PM Ahmet Altay <al...@google.com> wrote: > All PRs look either merged or closed. > > +Udi Meiri <eh...@google.com> might have more information about the > remaining work. > > On Wed, Mar 24, 2021 at 5:29 PM Benjamin Gonzalez Delgado < > benjamin.gonza...@wizeline.com> wrote: > >> Hi team, >> I am planning to work in BEAM-3713 >> <https://issues.apache.org/jira/browse/BEAM-3713>, but I see there are >> PRs related to the task. >> Could someone guide me on the work that remains missing regarding the >> migration from nose to pytest? >> Any guidance on this would be appreciated. >> >> Thanks! >> Benjamin >> >> >> >> >> >> >> >> >> *This email and its contents (including any attachments) are being sent >> toyou on the condition of confidentiality and may be protected by >> legalprivilege. Access to this email by anyone other than the intended >> recipientis unauthorized. If you are not the intended recipient, please >> immediatelynotify the sender by replying to this message and delete the >> materialimmediately from your system. Any further use, dissemination, >> distributionor reproduction of this email is strictly prohibited. Further, >> norepresentation is made with respect to any content contained in this >> email.* > >
smime.p7s
Description: S/MIME Cryptographic Signature
