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

Fix Test Suite #65

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Fix Test Suite #65

wants to merge 6 commits into from

Conversation

mattezra
Copy link
Contributor

@mattezra mattezra commented Dec 6, 2024

Fixes #64

Updates done:

  • updated media URLs to use the new URLs
  • created a dedicated instance that will be used for the tests
  • added TEST_PREVIEW_LOCK_PASSWORD in the .env to get uncached content from the updateView test
  • cleaned up before hook tests to only be executed once for each test file
  • added deletion on resource creation tests to cleanup instance
  • skipped model and field creation test to ensure that we don't hit the limit of fields allowed in a model
  • fixed findAndReplace test

Note:
Sent to keybase the env variables that the tests will be ran against (cc @markelmad)

@mattezra mattezra self-assigned this Dec 6, 2024
@mattezra mattezra linked an issue Dec 6, 2024 that may be closed by this pull request
6 tasks
@mattezra mattezra requested a review from shrunyan December 17, 2024 07:29
@mattezra mattezra added enhancement New feature or request ready labels Dec 17, 2024
@mattezra mattezra marked this pull request as ready for review December 17, 2024 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Media and Instance APIs 500 Service Errors
1 participant