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

Add running of tests on commit & PRs #16

Closed
wants to merge 9 commits into from

Conversation

duckduckgrayduck
Copy link

No description provided.

@duckduckgrayduck duckduckgrayduck added the enhancement New feature or request label Oct 31, 2023
@duckduckgrayduck
Copy link
Author

When the containers (squarelet, DocumentCloud, documentcloud-frontend) are down, the tests send me one error message for teardown of tests.
Here is when the tests are run locally with make test with all of the tests pre-recorded

ContainersUp

Here is when I take the containers down (like when the tests would run in an automated way:
ContainersDown

And for good measure, I removed test_users altogether and still got an error on the last test, this time toolbox

RemoveTestUsers

@duckduckgrayduck
Copy link
Author

The teardown error is due to trying to take down the document and project fixtures with deletes(), which require network access to do so.

@duckduckgrayduck
Copy link
Author

Going to close this one, as there are merge conflcits. Pushed Mitch's changes to a new branch to do a PR.

@duckduckgrayduck duckduckgrayduck deleted the duckduckgrayduck-tests branch November 3, 2023 21:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants