Disable V8-dependent API tests for V2 since future arangods will ship without V8

Author: bluepal-prasanthi-moparthi

v2/tests/asyncjob_test.go

@@ -111,6 +111,8 @@ func TestAsyncJobListPending(t *testing.T) {
 					skipBelowVersion(client, ctx, "3.11.1", t)
 					skipResilientSingleMode(t)
 
+					requireV8Enabled(client, ctx, t)
+
 					ctxAsync := connection.WithAsync(context.Background())
 
 					idTransaction := runLongRequest(t, ctxAsync, db, 2, col.Name())
@@ -254,6 +256,7 @@ func TestAsyncJobDelete(t *testing.T) {
 					})
 
 					t.Run("delete pending job", func(t *testing.T) {
+						requireV8Enabled(client, ctx, t)
 						idTransaction := runLongRequest(t, ctxAsync, db, 10, col.Name())
 						require.NotEmpty(t, idTransaction)
 
@@ -275,6 +278,7 @@ func TestAsyncJobDelete(t *testing.T) {
 					})
 
 					t.Run("delete expired jobs", func(t *testing.T) {
+						requireV8Enabled(client, ctx, t)
 						idTransaction := runLongRequest(t, ctxAsync, db, 10, col.Name())
 						require.NotEmpty(t, idTransaction)
 

v2/tests/database_query_test.go

@@ -1302,6 +1302,7 @@ func Test_UserDefinedFunctions(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				// Define UDF details
 				namespace := "myfunctions::temperature::" + StringWithCharset(16, charset)
 				functionName := namespace + "::celsiustofahrenheit"

v2/tests/database_transactionsjs_test.go

@@ -34,7 +34,7 @@ func Test_DatabaseTransactionsJS(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			WithCollectionV2(t, db, nil, func(col arangodb.Collection) {
-
+				requireV8Enabled(client, context.Background(), t)
 				t.Run("Transaction ReturnValue", func(t *testing.T) {
 					withContextT(t, defaultTestTimeout, func(ctx context.Context, t testing.TB) {
 						txJSOptions := arangodb.TransactionJSOptions{

v2/tests/foxx_test.go

@@ -36,6 +36,7 @@ func Test_FoxxItzpapalotlService(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				if os.Getenv("TEST_CONNECTION") == "vst" {
 					skipBelowVersion(client, ctx, "3.6", t)
 				}
@@ -298,6 +299,7 @@ func Test_ListInstalledFoxxServices(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				services, err := client.ListInstalledFoxxServices(ctx, db.Name(), nil)
 				require.NoError(t, err)
 				require.NotEmpty(t, services)
@@ -325,6 +327,7 @@ func Test_GetInstalledFoxxService(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				mount := "/_api/foxx"
 				serviceDetails, err := client.GetInstalledFoxxService(ctx, db.Name(), &mount)
 				require.NoError(t, err)

v2/tests/tasks_test.go

@@ -55,6 +55,7 @@ func Test_CreateNewTask(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				testCases := map[string]*arangodb.TaskOptions{
 					"taskWithParams": {
 						Name:    utils.NewType("taskWithParams"),
@@ -128,6 +129,7 @@ func Test_ValidationsForCreateNewTask(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				testCases := map[string]*arangodb.TaskOptions{
 					"taskWithoutCommand": {
 						Name:   utils.NewType("taskWithoutCommand"),
@@ -160,6 +162,7 @@ func Test_TaskCreationWithId(t *testing.T) {
 	Wrap(t, func(t *testing.T, client arangodb.Client) {
 		WithDatabase(t, client, nil, func(db arangodb.Database) {
 			withContextT(t, defaultTestTimeout, func(ctx context.Context, tb testing.TB) {
+				requireV8Enabled(client, ctx, tb)
 				taskID := "test-task-id" + StringWithCharset(16, charset)
 				options := &arangodb.TaskOptions{
 					ID:      &taskID, // Optional if CreateTaskWithID sets it, but safe to keep

v2/tests/util_test.go

@@ -29,6 +29,7 @@ import (
 	"github.com/stretchr/testify/require"
 
 	"github.com/arangodb/go-driver/v2/arangodb"
+	"github.com/arangodb/go-driver/v2/utils"
 )
 
 func getTestMode() string {
@@ -136,3 +137,26 @@ func skipVersionNotInRange(c arangodb.Client, ctx context.Context, minVersion, m
 	}
 	return x
 }
+
+// requireV8Enabled skips the test if V8 is disabled in the ArangoDB server.
+// V8 is required for features like tasks, UDFs, Foxx, JS transactions, and simple queries.
+// This function checks the v8-version field in the version details.
+// If v8-version is "none", V8 is disabled and the test will be skipped.
+func requireV8Enabled(c arangodb.Client, ctx context.Context, t testing.TB) {
+	versionInfo, err := c.VersionWithOptions(ctx, &arangodb.GetVersionOptions{
+		Details: utils.NewType(true),
+	})
+	if err != nil {
+		t.Fatalf("Failed to get version info with details: %s", err)
+	}
+
+	// Check if v8-version exists in Details and if it's "none"
+	if versionInfo.Details != nil {
+		if v8Version, ok := versionInfo.Details["v8-version"]; ok {
+			if v8VersionStr, ok := v8Version.(string); ok && v8VersionStr == "none" {
+				t.Skip("Skipping test: V8 is disabled in this ArangoDB server (v8-version: none). " +
+					"This test requires V8 enabled.")
+			}
+		}
+	}
+}