[PR #1830] [MERGED] feat: add tests for db migration on sqlite and postgres #20808

Closed
opened 2026-04-20 03:11:42 -05:00 by GiteaMirror · 0 comments
Owner

📋 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: devHead: feat/db-migration-tests


📝 Commits (3)

  • 167b571 feat: add tests for db migration on sqlite, postgres, and mysql
  • eed6c71 fix: disable mysql tests
  • f561e94 fix: 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: 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.

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.

## 📋 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>
GiteaMirror added the pull-request label 2026-04-20 03:11:42 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/open-webui#20808