cleanup unused code

.gitignore

@@ -1,6 +1,5 @@
 *.test
 .astro/
-.Dockerfile.sql_cli
 .DS_Store
 .env
 .idea/

Makefile

@@ -48,9 +48,6 @@ test:
 temp-astro:
 	cd $(shell mktemp -d) && ${PWD}/astro dev init
 
-temp-astro-flow:
-	./astro flow init $(shell mktemp -d)
-
 mock: mockery mock_airflow mock_houston mock_astro mock_pkg mock_astro_core mock_airflow_api
 
 mock_houston:

pkg/util/util.go

@@ -9,9 +9,6 @@ import (
 	"github.com/Masterminds/semver"
 	"github.com/golang-jwt/jwt/v4"
 	"github.com/pkg/errors"
-
-	// "Masterminds/semver" does not support the format of pre-release tags for SQL CLI, so we're using "hashicorp/go-version"
-	goVersion "github.com/hashicorp/go-version"
 )
 
 type CustomClaims struct {
@@ -103,21 +100,6 @@ func IsM1(myOS, myArch string) bool {
 	return false
 }
 
-func IsRequiredVersionMet(currentVersion, requiredVersion string) (bool, error) {
-	v1, err := goVersion.NewVersion(currentVersion)
-	if err != nil {
-		return false, err
-	}
-	constraints, err := goVersion.NewConstraint(requiredVersion)
-	if err != nil {
-		return false, err
-	}
-	if constraints.Check(v1) {
-		return true, nil
-	}
-	return false, nil
-}
-
 func ParseAPIToken(astroAPIToken string) (*CustomClaims, error) {
 	// Parse the token to peek at the custom claims
 	jwtParser := jwt.NewParser()

pkg/util/util_test.go

@@ -1,7 +1,6 @@
 package util
 
 import (
-	"errors"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
@@ -229,53 +228,3 @@ func TestIsM1(t *testing.T) {
 		assert.False(t, IsM1("windows", "amd64"))
 	})
 }
-
-var (
-	errMalformedCurrentVersion = errors.New("Malformed version: invalid current version") //nolint:stylecheck
-	errMalformedConstraint     = errors.New("Malformed constraint: invalid constraint")   //nolint:stylecheck
-)
-
-func TestIsRequiredVersionMet(t *testing.T) {
-	type args struct {
-		currentVersion  string
-		requiredVersion string
-	}
-	type result struct {
-		valid bool
-		err   error
-	}
-
-	tests := []struct {
-		name string
-		args args
-		want result
-	}{
-		{
-			name: "first true case",
-			args: args{"7.1.0", ">7.0.0, <8.0.0"},
-			want: result{true, nil},
-		},
-		{
-			name: "first false case",
-			args: args{"7.1.0", ">7.1.0"},
-			want: result{false, nil},
-		},
-		{
-			name: "first error case",
-			args: args{"invalid current version", ">7.0.0, <8.0.0"},
-			want: result{false, errMalformedCurrentVersion},
-		},
-		{
-			name: "second error case",
-			args: args{"7.1.0", "invalid constraint"},
-			want: result{false, errMalformedConstraint},
-		},
-	}
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			if versionMet, err := IsRequiredVersionMet(tt.args.currentVersion, tt.args.requiredVersion); versionMet != tt.want.valid || (err != nil && err.Error() != tt.want.err.Error()) {
-				t.Errorf("IsRequiredVersionMet() = %v, %v; want %v, %v", versionMet, err, tt.want.valid, tt.want.err)
-			}
-		})
-	}
-}