-
-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Added test environment for mssql #4282
Changes from 3 commits
6570d21
d365cd6
2aa2f19
38e8b9c
6e3e458
f03fc16
002b300
fa731e2
8b2e67e
f5069eb
3fc4bb3
6ebc478
1c7c44f
729a512
8ba423a
d975ba3
ed8008d
63681b7
d1907f9
c5e48e4
b889df5
b377bea
f104ce7
b07270d
f14225e
f02366c
4f3508b
98d0f22
1d0c7dd
b2ac9c6
f429734
5318157
b3fa696
e5efe98
ee146b0
543e997
15715d6
46da16e
b2196d8
d6ad629
1a27388
6f5200c
96f7f86
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -172,6 +172,20 @@ pipeline: | |
when: | ||
event: [ push, tag, pull_request ] | ||
|
||
test-mssql: | ||
image: golang:1.10 | ||
pull: true | ||
group: test | ||
environment: | ||
TAGS: bindata | ||
TEST_LDAP: "1" | ||
commands: | ||
- curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | bash | ||
- apt-get install -y git-lfs | ||
- make test-mssql | ||
when: | ||
event: [ push, tag, pull_request ] | ||
|
||
generate-coverage: | ||
image: golang:1.10 | ||
pull: true | ||
|
@@ -342,6 +356,15 @@ services: | |
when: | ||
event: [ push, tag, pull_request ] | ||
|
||
mssql: | ||
image: microsoft/mssql-server-linux:latest | ||
environment: | ||
- ACCEPT_EULA=Y | ||
- SA_PASSWORD=M$wantsaSecurePassword1 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think you need to escape There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. but maybe not it this file but in app.ini There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks for pointing that out, I thinks thats the issue. |
||
- MSSQL_PID=Standard | ||
when: | ||
event: [ push, tag, pull_request ] | ||
|
||
ldap: | ||
image: gitea/test-openldap:latest | ||
when: | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,72 @@ | ||
APP_NAME = Gitea: Git with a cup of tea | ||
RUN_MODE = prod | ||
|
||
[database] | ||
DB_TYPE = mssql | ||
HOST = {{TEST_MSSQL_HOST}} | ||
NAME = {{TEST_MSSQL_DBNAME}} | ||
USER = {{TEST_MSSQL_USERNAME}} | ||
PASSWD = {{TEST_MSSQL_PASSWORD}} | ||
SSL_MODE = disable | ||
|
||
[indexer] | ||
ISSUE_INDEXER_PATH = integrations/indexers-mssql/issues.bleve | ||
REPO_INDEXER_ENABLED = true | ||
REPO_INDEXER_PATH = integrations/indexers-mssql/repos.bleve | ||
|
||
[repository] | ||
ROOT = integrations/gitea-integration-mssql/gitea-repositories | ||
|
||
[repository.local] | ||
LOCAL_COPY_PATH = tmp/local-repo-mssql | ||
LOCAL_WIKI_PATH = tmp/local-wiki-mssql | ||
|
||
[server] | ||
SSH_DOMAIN = localhost | ||
HTTP_PORT = 3003 | ||
ROOT_URL = https://localhost:3003/ | ||
DISABLE_SSH = false | ||
SSH_LISTEN_HOST = localhost | ||
SSH_PORT = 2201 | ||
START_SSH_SERVER = true | ||
LFS_START_SERVER = true | ||
LFS_CONTENT_PATH = data/lfs-mssql | ||
OFFLINE_MODE = false | ||
LFS_JWT_SECRET = Tv_MjmZuHqpIY6GFl12ebgkRAMt4RlWt0v4EHKSXO0w | ||
APP_DATA_PATH = integrations/gitea-integration-mssql/data | ||
|
||
[mailer] | ||
ENABLED = false | ||
|
||
[service] | ||
REGISTER_EMAIL_CONFIRM = false | ||
ENABLE_NOTIFY_MAIL = false | ||
DISABLE_REGISTRATION = false | ||
ENABLE_CAPTCHA = false | ||
REQUIRE_SIGNIN_VIEW = false | ||
DEFAULT_KEEP_EMAIL_PRIVATE = false | ||
DEFAULT_ALLOW_CREATE_ORGANIZATION = true | ||
NO_REPLY_ADDRESS = noreply.example.org | ||
|
||
[picture] | ||
DISABLE_GRAVATAR = false | ||
ENABLE_FEDERATED_AVATAR = false | ||
|
||
[session] | ||
PROVIDER = file | ||
PROVIDER_CONFIG = data/sessions-mssql | ||
|
||
[log] | ||
MODE = console,file | ||
ROOT_PATH = mssql-log | ||
|
||
[log.console] | ||
LEVEL = Warn | ||
|
||
[log.file] | ||
LEVEL = Debug | ||
|
||
[security] | ||
INSTALL_LOCK = true | ||
SECRET_KEY = 9pCviYTWSb | ||
INTERNAL_TOKEN = eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJuYmYiOjE0OTU1NTE2MTh9.hhSVGOANkaKk3vfCd2jDOIww4pUk0xtg9JRde5UogyQ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Edit: I was being lazy, pay attention to @JonasFranzDEV as his solution is better than mine.
before the test steps (look at group test) you could have a step that creates the DB like:I haven't tested the above (or indentation) but something like it should workThere was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The better option might be to create the database in code like we do for pgsql and mysql:
https://github.com/go-gitea/gitea/blob/master/integrations/integration_test.go#L110
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've implemented it and the test works now