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

Make repo private to not interfere with other tests #2467

Merged
merged 1 commit into from
Sep 5, 2017

Conversation

sapk
Copy link
Member

@sapk sapk commented Sep 5, 2017

Since dev of #2266 some tests were added. Change made in PR #2266 add a repo fixture that interfere with thoses tests.

To fix that, I made the repo fixture private.

@sapk sapk changed the title Make repo private to no interfere with other tests Make gpg test repo private to no interfere with other tests Sep 5, 2017
@sapk sapk changed the title Make gpg test repo private to no interfere with other tests Make repo private to not interfere with other tests Sep 5, 2017
@strk
Copy link
Member

strk commented Sep 5, 2017

trusted LGTM

@tboerger tboerger added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Sep 5, 2017
@lafriks
Copy link
Member

lafriks commented Sep 5, 2017

LGTM

@tboerger tboerger added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 5, 2017
@lafriks lafriks merged commit 4f9a28e into go-gitea:master Sep 5, 2017
@lunny lunny added this to the 1.3.0 milestone Sep 6, 2017
@sapk
Copy link
Member Author

sapk commented Sep 6, 2017

Even if PR CI and local tests passed, It seems that it make master failed on gpg test ...

@lunny
Copy link
Member

lunny commented Sep 6, 2017

I have tested master(95c408f), tests is work.

@sapk
Copy link
Member Author

sapk commented Sep 6, 2017

@lunny It's seems that tests are randomly failing : https://drone.gitea.io/go-gitea/gitea/455 😢 so maybe not related.

@sapk sapk mentioned this pull request Sep 8, 2017
1 task
sapk added a commit to sapk-fork/gitea that referenced this pull request Oct 26, 2017
lunny pushed a commit that referenced this pull request Oct 27, 2017
* Only check at least one email gpg key (#2266)

* Only require one email (possibly not yet validated)

* Update message error and check validation of commit

* Add integrations tests

* Complete integration for import

* Add pre-check/optimization

* Add some test (not finished)

* Finish

* Fix fixtures

* Fix typo

* Don't guess key ID

* Make repo private to no interfere with other tests (#2467)

* GPG key email verification no longer case sensitive (#2661) (#2663)

* GPG key email verification no longer case sensitive (#2661)

* case insensitive GPG key email verification now cached (#2661)

Signed-off-by: Julian Scholle <julian.scholle@googlemail.com>
@sapk sapk deleted the fix-test-after-2266 branch December 9, 2017 21:47
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants