Skip to content

Commit b4e4003

Browse files
authored
test: add integration build tag (#2831)
This test verify the integration of the librarian tool with external systems, GitHub and Artifact Registry. It has expectation setups in env vars and auth to properly work. With an `integration` build tag, all tests in system_test.go will be skipped when running `go test ./...` locally. Which is same behavior for most developers who does not have relevant env vars setup. Also remove the temp skip for TestFindLatestImage because it is no longer triggered with standard `go test ./...`. Added a note to make debugging easier. Fixes #2720
1 parent 05c9b65 commit b4e4003

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

system_test.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,8 @@
1212
// See the License for the specific language governing permissions and
1313
// limitations under the License.
1414

15+
//go:build integration
16+
1517
package librarian
1618

1719
import (
@@ -446,7 +448,7 @@ func TestCreateRelease(t *testing.T) {
446448
}
447449

448450
func TestFindLatestImage(t *testing.T) {
449-
t.Skip("Temporarily skipping until we can run it reliably. See https://github.com/googleapis/librarian/issues/2720")
451+
t.Logf("Note: This test requires authentication with the Artifact Registry in project 'cloud-sdk-librarian-prod'.")
450452

451453
// If we are able to configure system tests on GitHub actions, then update this
452454
// guard clause.

0 commit comments

Comments
 (0)