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 Playwright tests #81

Merged
merged 43 commits into from
Nov 12, 2024
Merged

Add Playwright tests #81

merged 43 commits into from
Nov 12, 2024

Conversation

Abdullah13521
Copy link
Contributor

@Abdullah13521 Abdullah13521 commented Nov 11, 2024

Description

Adds playwright tests to test the frontend of Colivara. Addresses this issue.

Type of change

  • 🚨 Breaking change - causes existing functionality to break (triggers MAJOR version bump)
  • ✨ New feature - adds functionality in a non-breaking way (triggers MINOR version bump)
  • 🐛 Bug fix - fixes an issue without changing functionality (triggers PATCH version bump)
  • 🏠 Internal - changes that don't affect external functionality (no version bump)
    (e.g., refactoring, docs, tests, admin site changes)

Release Impact

  • Current version:
  • Next version will be:

Checklist

  • My code follows the style guidelines
  • My code is tested with 99%+ coverage
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have updated the documentation to reflect my changes
  • I have added/updated tests to cover my changes

Breaking Changes

Migration Guide

Additional Notes

@Jonathan-Adly
Copy link
Contributor

@Abdullah13521 - review my changes.

  • Added a pytest-ui.ini to avoid issues with the API tests
  • Add the stripe CLI to docker-compose, so stripe works without logging in local
  • Added the management commands before and after running the tests.

To do:

  • Modify the github action that deploys cloud to run the tests then deploy. I will do it, just FYI.

@Jonathan-Adly
Copy link
Contributor

closes #58

@Jonathan-Adly Jonathan-Adly merged commit 08db754 into cloud Nov 12, 2024
1 check passed
@Jonathan-Adly Jonathan-Adly deleted the playwright branch November 12, 2024 16:11
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

Successfully merging this pull request may close these issues.

2 participants