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

auth/pam test fails with v1.23.0 #33168

Closed
Antiz96 opened this issue Jan 9, 2025 · 3 comments · Fixed by #33169
Closed

auth/pam test fails with v1.23.0 #33168

Antiz96 opened this issue Jan 9, 2025 · 3 comments · Fixed by #33169
Labels

Comments

@Antiz96
Copy link

Antiz96 commented Jan 9, 2025

Description

Hi,

When building Gitea 1.23.0, the auth/pam test is failing when running make test:

[...]
modules/auth/pam/pam_test.go:18:16: not enough arguments in call to assert.Len
[...]
FAIL    code.gitea.io/gitea/modules/auth/pam [build failed]
[...]
FAIL
make: *** [Makefile:473: test-backend] Error 1

It passes fine when building previous versions however (e.g. 1.22.6).

The build and test instructions we use are available here.

I remain available if needed :)

Gitea Version

1.23.0

Can you reproduce the bug on the Gitea demo site?

N/A

Log Gist

Provided in the description

Screenshots

N/A

Git Version

The build and test env is using git 2.47.1

Operating System

Arch Linux

How are you running Gitea?

N/A

Database

N/A

@TheFox0x7
Copy link
Contributor

my bad, not sure how it didn't trigger any lint issues.

Is pam test not run in CI?

@Antiz96
Copy link
Author

Antiz96 commented Jan 9, 2025

my bad, not sure how it didn't trigger any lint issues.

No problem, thanks for the very quick actions! 🙏
I can confirm the fix provided in #33169 fixes the issue 😄

@wxiaoguang
Copy link
Contributor

wxiaoguang commented Jan 9, 2025

Is pam test not run in CI?

Hmm .... it seems that pam test doesn't run in CI because there is no pam build tag.

GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue Jan 9, 2025
wxiaoguang pushed a commit that referenced this issue Jan 9, 2025
Backport #33169 by TheFox0x7

fixes: #33168

Co-authored-by: TheFox0x7 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants