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

TestProject.test_conf_file_not_found fails on master #3699

Closed
xrmx opened this issue Feb 28, 2018 · 5 comments
Closed

TestProject.test_conf_file_not_found fails on master #3699

xrmx opened this issue Feb 28, 2018 · 5 comments

Comments

@xrmx
Copy link
Contributor

xrmx commented Feb 28, 2018

Details

With 09f8282be35c20dcca42c85d3f845bba8b784cfe

tox -e py27

fails

Expected Result

Tests green

Actual Result

TestProject.test_conf_file_not_found 

self = <readthedocs.rtd_tests.tests.test_project.TestProject testMethod=test_conf_file_not_found>

    def test_conf_file_not_found(self):
        with self.assertRaisesMessage(
                ProjectConfigurationError,
                ProjectConfigurationError.NOT_FOUND) as cm:
>           self.pip.conf_file()

rtd_tests/tests/test_project.py:146: 
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
/usr/lib/python2.7/contextlib.py:24: in __exit__
    self.gen.next()
../.tox/py27/local/lib/python2.7/site-packages/django/test/testcases.py:616: in _assert_raises_message_cm
    yield cm
E   AssertionError: ProjectConfigurationError not raised
@xrmx
Copy link
Contributor Author

xrmx commented Feb 28, 2018

Can this be caused by running the tests on an instance that has built pip documentation?

@stsewd
Copy link
Member

stsewd commented Feb 28, 2018

@xrmx I think that for the tests a separate database is used, I can't replicate this issue on my local instance and the tests are all green on travis. What OS do you have?

@xrmx
Copy link
Contributor Author

xrmx commented Feb 28, 2018

@stsewd was thinking more of the fact that there's a checkout of the pip docs or a build of the same around. The os is debian sid

@stsewd
Copy link
Member

stsewd commented Mar 1, 2018

@xrmx also make sure to run the tests while the instance isn't running (some tests fail because of this).

@xrmx
Copy link
Contributor Author

xrmx commented Mar 1, 2018

@stsewd wasn't running.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants