Skip to content

Updating some Action versions and disabling KBase auth2 server tests #222

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.

Sign up for GitHub

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

Merged
jeff-cohere merged 3 commits into from
Jun 3, 2026
Merged

Updating some Action versions and disabling KBase auth2 server tests #222

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
0f1f438
Bumping some action versions.
jeff-cohere Jun 3, 2026
ba9373e
Disabling KBase auth2 server tests (disallowed by Cloudflare bot stor…
jeff-cohere Jun 3, 2026
94aeb5a
Moving t.Skip statements upward.
jeff-cohere Jun 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 4 additions & 6 deletions .github/workflows/autotest_prs.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,6 @@ on:
branches: [ main ]
workflow_dispatch:



jobs:

build:
Expand All @@ -19,14 +17,14 @@ jobs:
pull-requests: write
steps:

- name: Setting up Go 1.21
uses: actions/setup-go@v4
- name: Setting up Go 1.24
uses: actions/setup-go@v6
with:
go-version: ^1.21
go-version: ^1.24
id: go

- name: Checking out DTS
uses: actions/checkout@v4
uses: actions/checkout@v5

- name: Vetting DTS
run: go vet -v ./...
Expand Down
5 changes: 5 additions & 0 deletions auth/kbase_auth_server_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ func getKbaseToken() (string, bool) {
// tests whether a proxy for the KBase authentication server can be
// constructed
func TestNewKBaseAuthServer(t *testing.T) {
t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)")
assert := assert.New(t)

devToken, ok := getKbaseToken()
Expand All @@ -158,6 +159,7 @@ func TestNewKBaseAuthServer(t *testing.T) {
// tests whether an invalid KBase token prevents a proxy for the auth server
// from being constructed
func TestInvalidToken(t *testing.T) {
t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)")
assert := assert.New(t)

_, ok := getKbaseToken()
Expand All @@ -180,6 +182,7 @@ func TestInvalidToken(t *testing.T) {

// tests that the proxy handles missing identifiers correctly
func TestNoIdentifiers(t *testing.T) {
t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)")
assert := assert.New(t)

_, ok := getKbaseToken()
Expand All @@ -198,6 +201,7 @@ func TestNoIdentifiers(t *testing.T) {

// tests that the proxy handles missing OrcID identifiers correctly
func TestNoOrcID(t *testing.T) {
t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)")
assert := assert.New(t)

_, ok := getKbaseToken()
Expand All @@ -217,6 +221,7 @@ func TestNoOrcID(t *testing.T) {
// tests whether the authentication server can return information for the
// client (the user associated with the specified developer token)
func TestClient(t *testing.T) {
t.Skip("Skipping KBase auth2 server test (access denied by Cloudflare bot storm security)")
assert := assert.New(t)

devToken, ok := getKbaseToken()
Expand Down
Loading