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

Simplify tests, run in same backend service #152

Merged
merged 1 commit into from
Apr 19, 2020
Merged

Conversation

tiangolo
Copy link
Member

♻️ Simplify tests, run in the same backend service, remove backend-tests.

Simplify the setup, one Docker Compose file less. One container less. This is the first step to running the tests with the TestClient, generating coverage, etc.

@tiangolo tiangolo merged commit 283bc7c into master Apr 19, 2020
@tiangolo tiangolo deleted the tests-in-backend branch April 19, 2020 10:34
ebreton added a commit to ebreton/full-stack-fastapi-postgresql that referenced this pull request Jun 1, 2020
…-postgresql into tiangolo-master

* 'master' of git://github.com/tiangolo/full-stack-fastapi-postgresql: (63 commits)
  📝 Update release notes
  ✨ Add GitHub Sponsors button (fastapi#201)
  📝 Update release notes
  🔊 Add consistent errors for env vars not set (fastapi#200)
  📝 Update release notes
  ✨ Upgrade Traefik to version 2 (fastapi#199)
  📝 Update release notes
  📝 Add docs about reporting coverage in HTML (fastapi#161)
  📝 Update relase notes
  ✨ Test using the TestClient (fastapi#160)
  🎨 Format fixes (fastapi#159)
  📝 Update release notes
  ♻️ Refactor backend, settings, DB sessions, types, configs, plugins (fastapi#158)
  📝 Update release notes
  📝 Add docs about removing the frontend (fastapi#156)
  📝 Update release notes
  🔧 Simplify scripts and development (fastapi#155)
  📝 Update release notes
  ♻️ Simplify Docker Compose files and deployment (fastapi#153)
  ♻️ Simplify tests, run in same backend service (fastapi#152)
  ...
ebreton added a commit to ebreton/full-stack-fastapi-postgresql that referenced this pull request Jun 1, 2020
* tiangolo-master: (63 commits)
  📝 Update release notes
  ✨ Add GitHub Sponsors button (fastapi#201)
  📝 Update release notes
  🔊 Add consistent errors for env vars not set (fastapi#200)
  📝 Update release notes
  ✨ Upgrade Traefik to version 2 (fastapi#199)
  📝 Update release notes
  📝 Add docs about reporting coverage in HTML (fastapi#161)
  📝 Update relase notes
  ✨ Test using the TestClient (fastapi#160)
  🎨 Format fixes (fastapi#159)
  📝 Update release notes
  ♻️ Refactor backend, settings, DB sessions, types, configs, plugins (fastapi#158)
  📝 Update release notes
  📝 Add docs about removing the frontend (fastapi#156)
  📝 Update release notes
  🔧 Simplify scripts and development (fastapi#155)
  📝 Update release notes
  ♻️ Simplify Docker Compose files and deployment (fastapi#153)
  ♻️ Simplify tests, run in same backend service (fastapi#152)
  ...
gusevyaroslove pushed a commit to gusevyaroslove/fastapi-template that referenced this pull request Aug 4, 2024
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.

1 participant