Skip to content

Conversation

@timhuynh94
Copy link
Contributor

@timhuynh94 timhuynh94 commented Mar 12, 2025

Functionality to retrieve storage credentials from server
Depends on go-vela/server#1271

@codecov
Copy link

codecov bot commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.31%. Comparing base (0f27f92) to head (da3d004).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #351   +/-   ##
=======================================
  Coverage   91.31%   91.31%           
=======================================
  Files          22       22           
  Lines        1635     1635           
=======================================
  Hits         1493     1493           
  Misses        106      106           
  Partials       36       36           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@timhuynh94 timhuynh94 marked this pull request as ready for review April 9, 2025 02:41
@timhuynh94 timhuynh94 requested a review from a team as a code owner April 9, 2025 03:01
@wass3rw3rk wass3rw3rk marked this pull request as draft May 14, 2025 15:02
@@ -0,0 +1,41 @@
// SPDX-License-Identifier: Apache-2.0

package vela
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
3-41 lines are duplicate of vela/testreport.go:3-41 (dupl)

@@ -0,0 +1,41 @@
// SPDX-License-Identifier: Apache-2.0

package vela
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
3-41 lines are duplicate of vela/testattachment.go:3-41 (dupl)


import (
"encoding/json"
"github.com/gin-gonic/gin"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
File is not properly formatted (gci)


import (
"encoding/json"
"github.com/gin-gonic/gin"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
File is not properly formatted (gci)

// from the server methods of the Vela API.
AdminSettingsService service

// AdminStorageSettingsService handles managing admin storage settings functionality
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Comment should end in a period (godot)

@@ -0,0 +1,58 @@
package vela
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Missed header for check (goheader)

@@ -0,0 +1,58 @@
package vela
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
Missed header for check (goheader)

s := httptest.NewServer(server.FakeHandler())
c, _ := NewClient(s.URL, "", nil)
c.Authentication.SetPersonalAccessTokenAuth("token")
data := []byte(server.StorageInfoResp)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
missing whitespace above this line (invalid statement above assign) (wsl_v5)

if err == nil {
t.Errorf("GetInfo should have returned err %v", resp.StatusCode)
}
if resp.StatusCode != http.StatusUnauthorized {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
missing whitespace above this line (invalid statement above if) (wsl_v5)

s := httptest.NewServer(server.FakeHandler())
c, _ := NewClient(s.URL, "", nil)
c.Authentication.SetPersonalAccessTokenAuth("token")
data := []byte(server.TestReportResp)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 [golangci] reported by reviewdog 🐶
missing whitespace above this line (invalid statement above assign) (wsl_v5)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants