Skip to content
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

[WIP] Create new sanity test that checks for tests #60612

Open
wants to merge 2 commits into
base: devel
from

Conversation

@samdoran
Copy link
Member

commented Aug 14, 2019

SUMMARY

Create a new sanity test that checks that modules have an integration on unit test.

Fixes #57699

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

test/lib/ansible_test/_internal/sanity/has_test.py

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Aug 14, 2019

@ansibot

This comment has been minimized.

Copy link
Contributor

commented Aug 14, 2019

The test ansible-test sanity --test pylint [explain] failed with 1 error:

test/lib/ansible_test/_internal/sanity/has_test.py:41:0: missing-docstring Missing class docstring

The test ansible-test sanity --test docs-build [explain] failed with 1 error:

docs/docsite/rst/dev_guide/testing/sanity/index.rst:11:0: toc-tree-missing-document: toctree contains reference to nonexisting document 'dev_guide/testing/sanity/test-testexists'

The test ansible-test sanity --test sanity-docs [explain] failed with 1 error:

docs/docsite/rst/dev_guide/testing/sanity/test-testexists.rst:0:0: missing docs for ansible-test sanity --test test-testexists

click here for bot help

@sivel sivel removed the needs_triage label Aug 15, 2019

@samdoran

This comment has been minimized.

Copy link
Member Author

commented Aug 15, 2019

bot_skip

@@ -0,0 +1,96 @@
"""Sanity test to check that an integration test exists"""

This comment has been minimized.

Copy link
@gundalow

gundalow Aug 27, 2019

Contributor
Suggested change
"""Sanity test to check that an integration test exists"""
"""Sanity test to check that a unit or integration test exists"""
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.