Skip to content

CI: Use azure-pipelines reporting for pytest data #26601

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
datapythonista opened this issue Jun 1, 2019 · 1 comment · Fixed by #26620
Closed

CI: Use azure-pipelines reporting for pytest data #26601

datapythonista opened this issue Jun 1, 2019 · 1 comment · Fixed by #26620
Labels
CI Continuous Integration Testing pandas testing functions or related to the test suite
Milestone

Comments

@datapythonista
Copy link
Member

Azure Pipelines can show errors and warnings clearer, by adding some specific syntax to the log.

We are already using this for the linting, where we output flake8 errors like ##vso[task.logissue type=error;sourcepath=%(path)s;linenumber=%(row)s;columnnumber=%(col)s;code=%(code)s;]%(text)s (see ci/code_checks.sh).

This makes it easier to find the errors in the logs, as they appear in the summary. See for example the Checks_and_docs build in https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=12078

We should be using the same for tests, and also "upload" to Azure pipelines the summary of the tests.

This is implemented in this pytest plugin: https://github.com/tonybaloney/pytest-azurepipelines

I guess the best option is to install the plugin (may be install in the azure yaml files, as the dependency is not useful outside of azure). The plugin is quite simple, if it makes sense we may prefer to implement something specific to our needs based on the plugin code: https://github.com/tonybaloney/pytest-azurepipelines/blob/master/pytest_azurepipelines.py

@datapythonista datapythonista added Testing pandas testing functions or related to the test suite CI Continuous Integration labels Jun 1, 2019
@TomAugspurger
Copy link
Contributor

TomAugspurger commented Jun 1, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants