all: oss: ListObjects should treat prefix as a dir

[scbizu]

See:

• fix: ensure proper prefix for tenant repositories in storage operations helm/chartmuseum#1082
• Unable to refresh tenant index and get new chart versions helm/chartmuseum#794

[Copilot]

Pull Request Overview

This PR ensures that the prefix parameter is consistently treated as a directory in all ListObjects methods by appending a trailing slash when needed.

• Introduced a normalizePath helper to add a trailing slash for non-empty paths.
• Updated all storage backends’ ListObjects to call normalizePath after joining the prefix.
• Added unit tests for the new normalizePath function.

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
storage.go	Added normalizePath helper function
storage_test.go	Added unit tests for normalizePath
tencent.go	Applied normalizePath in ListObjects
oracle.go	Applied normalizePath in ListObjects
openstack.go	Applied normalizePath in ListObjects
microsoft.go	Applied normalizePath in ListObjects
google.go	Applied normalizePath in ListObjects
etcd.go	Applied normalizePath in ListObjects
baidu.go	Applied normalizePath in ListObjects
amazon.go	Applied normalizePath in ListObjects
alibaba.go	Applied normalizePath in ListObjects

Comments suppressed due to low confidence (3)

storage.go:100

• [nitpick] Add a comment above normalizePath to describe its behavior (e.g., "// normalizePath ensures a non-empty path ends with a slash").

func normalizePath(path string) string {

storage_test.go:184

• Consider adding test cases for nested paths (e.g., normalizePath("a/b") => "a/b/") to improve coverage for multi-segment prefixes.

	suite.Equal("prefix/", normalizePath("prefix"), "path without trailing slash normalized to path with trailing slash")

tencent.go:90

• [nitpick] This join-and-normalize pattern is repeated across multiple backends; consider extracting a helper like joinAndNormalizePrefix to reduce duplication and ensure consistency.

	prefix = normalizePath(prefix)

[nerdeveloper]

LGTM