Description: Briefly describe the changes in this pull request.
Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
Testing: Have you written and run sufficient tests for the changes?
Code Review: Have you self-reviewed your code and addressed any coding standard issues?
Description
Adds a new test in CI that runs the backend with SQLite and Postgres, and verifies that they start correctly, which will be key to ensuring we don't break compatibility with external databases when making changes to DB schema.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/open-webui/open-webui/pull/1830
**Author:** [@cheahjs](https://github.com/cheahjs)
**Created:** 4/28/2024
**Status:** ✅ Merged
**Merged:** 4/28/2024
**Merged by:** [@tjbck](https://github.com/tjbck)
**Base:** `dev` ← **Head:** `feat/db-migration-tests`
---
### 📝 Commits (3)
- [`167b571`](https://github.com/open-webui/open-webui/commit/167b5712ea097efcd21895be6499bcd0274a0357) feat: add tests for db migration on sqlite, postgres, and mysql
- [`eed6c71`](https://github.com/open-webui/open-webui/commit/eed6c7194b7009f613cf6aea313e8dd07e9a83ac) fix: disable mysql tests
- [`f561e94`](https://github.com/open-webui/open-webui/commit/f561e942442d8efa2ee3a3bf4ee55a13ace6572d) fix: refactor db migration tests
### 📊 Changes
**1 file changed** (+131 additions, -0 deletions)
<details>
<summary>View changed files</summary>
📝 `.github/workflows/integration-test.yml` (+131 -0)
</details>
### 📄 Description
## Pull Request Checklist
- [x] **Description:** Briefly describe the changes in this pull request.
- [ ] **Changelog:** Ensure a changelog entry following the format of [Keep a Changelog](https://keepachangelog.com/) is added at the bottom of the PR description.
- [ ] **Documentation:** Have you updated relevant documentation [Open WebUI Docs](https://github.com/open-webui/docs), or other documentation sources?
- [ ] **Dependencies:** Are there any new dependencies? Have you updated the dependency versions in the documentation?
- [x] **Testing:** Have you written and run sufficient tests for the changes?
- [x] **Code Review:** Have you self-reviewed your code and addressed any coding standard issues?
---
## Description
Adds a new test in CI that runs the backend with SQLite and Postgres, and verifies that they start correctly, which will be key to ensuring we don't break compatibility with external databases when making changes to DB schema.
Example run: https://github.com/open-webui/open-webui/actions/runs/8868769049/job/24348714090?pr=1830
There's a test for MySQL as well, but it was failing, so I've commented it out for now: https://github.com/cheahjs/open-webui-fork/actions/runs/8868578535/job/24348304494
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/open-webui/open-webui/pull/1830
Author: @cheahjs
Created: 4/28/2024
Status: ✅ Merged
Merged: 4/28/2024
Merged by: @tjbck
Base:
dev← Head:feat/db-migration-tests📝 Commits (3)
167b571feat: add tests for db migration on sqlite, postgres, and mysqleed6c71fix: disable mysql testsf561e94fix: refactor db migration tests📊 Changes
1 file changed (+131 additions, -0 deletions)
View changed files
📝
.github/workflows/integration-test.yml(+131 -0)📄 Description
Pull Request Checklist
Description
Adds a new test in CI that runs the backend with SQLite and Postgres, and verifies that they start correctly, which will be key to ensuring we don't break compatibility with external databases when making changes to DB schema.
Example run: https://github.com/open-webui/open-webui/actions/runs/8868769049/job/24348714090?pr=1830
There's a test for MySQL as well, but it was failing, so I've commented it out for now: https://github.com/cheahjs/open-webui-fork/actions/runs/8868578535/job/24348304494
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.