Skip to content

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 <[email protected]>
@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