[PR #4282] [MERGED] Added test environment for mssql #17315

Closed
opened 2025-11-02 12:32:24 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/4282
Author: @kolaente
Created: 6/20/2018
Status: Merged
Merged: 12/12/2018
Merged by: @lunny

Base: masterHead: mssql-test


📝 Commits (10+)

  • 6570d21 Added test environment for m$sql
  • d365cd6 Added template for test environment for m$sql
  • 2aa2f19 Fix password
  • 38e8b9c Fix password (again)
  • 6e3e458 Fix password (again again)
  • f03fc16 Fix db
  • 002b300 Ci trigger (Looking at you drone....)
  • fa731e2 Ci trigger (Looking at you drone....)
  • 8b2e67e Ci trigger (Looking at you drone....)
  • f5069eb Ci trigger (Looking at you drone....)

📊 Changes

27 files changed (+383 additions, -66 deletions)

View changed files

📝 .drone.yml (+23 -0)
📝 .gitignore (+3 -0)
📝 Gopkg.lock (+2 -2)
📝 Gopkg.toml (+1 -2)
📝 Makefile (+16 -3)
📝 integrations/integration_test.go (+13 -0)
integrations/mssql.ini.tmpl (+72 -0)
📝 models/models.go (+4 -3)
vendor/github.com/go-xorm/xorm/context_cache.go (+30 -0)
📝 vendor/github.com/go-xorm/xorm/dialect_mssql.go (+1 -1)
📝 vendor/github.com/go-xorm/xorm/dialect_mysql.go (+4 -1)
📝 vendor/github.com/go-xorm/xorm/dialect_sqlite3.go (+5 -1)
📝 vendor/github.com/go-xorm/xorm/engine.go (+15 -2)
📝 vendor/github.com/go-xorm/xorm/engine_group.go (+10 -0)
vendor/github.com/go-xorm/xorm/engine_maxlife.go (+0 -22)
📝 vendor/github.com/go-xorm/xorm/interface.go (+8 -1)
📝 vendor/github.com/go-xorm/xorm/session.go (+15 -0)
📝 vendor/github.com/go-xorm/xorm/session_delete.go (+1 -1)
📝 vendor/github.com/go-xorm/xorm/session_find.go (+3 -3)
📝 vendor/github.com/go-xorm/xorm/session_get.go (+27 -2)

...and 7 more files

📄 Description

This pr adds a test step in drone to run tests with mssql db.

Even if you dont like Microsoft (like me), IMHO we should at least have tests for mssql dbs or not support it at all.


🔄 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/go-gitea/gitea/pull/4282 **Author:** [@kolaente](https://github.com/kolaente) **Created:** 6/20/2018 **Status:** ✅ Merged **Merged:** 12/12/2018 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `mssql-test` --- ### 📝 Commits (10+) - [`6570d21`](https://github.com/go-gitea/gitea/commit/6570d21d07f259b110f87baa4369cce5a6b4349d) Added test environment for m$sql - [`d365cd6`](https://github.com/go-gitea/gitea/commit/d365cd6bfe84c5ff5ab7e01625df97622db9a2a0) Added template for test environment for m$sql - [`2aa2f19`](https://github.com/go-gitea/gitea/commit/2aa2f1988def804011d6e06e6d507a02597412ff) Fix password - [`38e8b9c`](https://github.com/go-gitea/gitea/commit/38e8b9c4d312f13e41108ea3a92f8d9212974c74) Fix password (again) - [`6e3e458`](https://github.com/go-gitea/gitea/commit/6e3e4587153569273f9068186e4bc4f4cf23080d) Fix password (again again) - [`f03fc16`](https://github.com/go-gitea/gitea/commit/f03fc160ba0ddaa50f3a4f327e0334d6ad81dbc5) Fix db - [`002b300`](https://github.com/go-gitea/gitea/commit/002b30056fe569fbfc84dccabc4afde8036c01ba) Ci trigger (Looking at you drone....) - [`fa731e2`](https://github.com/go-gitea/gitea/commit/fa731e2bcee4173e96f10ec8a3f2651390f49978) Ci trigger (Looking at you drone....) - [`8b2e67e`](https://github.com/go-gitea/gitea/commit/8b2e67ec9a121b35cd290bf2eb15ef8a5c9c0709) Ci trigger (Looking at you drone....) - [`f5069eb`](https://github.com/go-gitea/gitea/commit/f5069eb574f8c713821b21ee9c5c28d026e17ac9) Ci trigger (Looking at you drone....) ### 📊 Changes **27 files changed** (+383 additions, -66 deletions) <details> <summary>View changed files</summary> 📝 `.drone.yml` (+23 -0) 📝 `.gitignore` (+3 -0) 📝 `Gopkg.lock` (+2 -2) 📝 `Gopkg.toml` (+1 -2) 📝 `Makefile` (+16 -3) 📝 `integrations/integration_test.go` (+13 -0) ➕ `integrations/mssql.ini.tmpl` (+72 -0) 📝 `models/models.go` (+4 -3) ➕ `vendor/github.com/go-xorm/xorm/context_cache.go` (+30 -0) 📝 `vendor/github.com/go-xorm/xorm/dialect_mssql.go` (+1 -1) 📝 `vendor/github.com/go-xorm/xorm/dialect_mysql.go` (+4 -1) 📝 `vendor/github.com/go-xorm/xorm/dialect_sqlite3.go` (+5 -1) 📝 `vendor/github.com/go-xorm/xorm/engine.go` (+15 -2) 📝 `vendor/github.com/go-xorm/xorm/engine_group.go` (+10 -0) ➖ `vendor/github.com/go-xorm/xorm/engine_maxlife.go` (+0 -22) 📝 `vendor/github.com/go-xorm/xorm/interface.go` (+8 -1) 📝 `vendor/github.com/go-xorm/xorm/session.go` (+15 -0) 📝 `vendor/github.com/go-xorm/xorm/session_delete.go` (+1 -1) 📝 `vendor/github.com/go-xorm/xorm/session_find.go` (+3 -3) 📝 `vendor/github.com/go-xorm/xorm/session_get.go` (+27 -2) _...and 7 more files_ </details> ### 📄 Description This pr adds a test step in drone to run tests with mssql db. Even if you dont like Microsoft (like me), IMHO we should at least have tests for mssql dbs or not support it at all. --- <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 2025-11-02 12:32:24 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#17315