Run testarchive unit tests on PRs (#3500)

## Why
Feedback in
#3453 (comment)
recommended not using build tags. This PR enables these tests by
default.

These tests will be skipped when `-short` flag is provided. Note: We do
not specify the `-short` flag in CI so these tests will run by default.

Author: shreyas-goenka

internal/testarchive/archive_test.go

@@ -1,5 +1,3 @@
-//go:build dbr_only
-
 package main
 
 import (
@@ -11,6 +9,12 @@ import (
 )
 
 func TestArchive(t *testing.T) {
+	if testing.Short() {
+		t.Skip("Skipping test in short mode")
+	}
+
+	t.Parallel()
+
 	archiveDir := t.TempDir()
 	binDir := t.TempDir()
 	repoRoot := "../.."

internal/testarchive/downloader_test.go

@@ -1,5 +1,3 @@
-//go:build dbr_only
-
 package main
 
 import (
@@ -12,6 +10,12 @@ import (
 )
 
 func TestUvDownloader(t *testing.T) {
+	if testing.Short() {
+		t.Skip("Skipping test in short mode")
+	}
+
+	t.Parallel()
+
 	tmpDir := t.TempDir()
 
 	for _, arch := range []string{"arm64", "amd64"} {
@@ -27,6 +31,12 @@ func TestUvDownloader(t *testing.T) {
 }
 
 func TestJqDownloader(t *testing.T) {
+	if testing.Short() {
+		t.Skip("Skipping test in short mode")
+	}
+
+	t.Parallel()
+
 	tmpDir := t.TempDir()
 
 	for _, arch := range []string{"arm64", "amd64"} {
@@ -42,6 +52,12 @@ func TestJqDownloader(t *testing.T) {
 }
 
 func TestGoDownloader(t *testing.T) {
+	if testing.Short() {
+		t.Skip("Skipping test in short mode")
+	}
+
+	t.Parallel()
+
 	tmpDir := t.TempDir()
 
 	for _, arch := range []string{"arm64", "amd64"} {