Fix commented out test

Author: JillRegan

internal/onepassword/client.go

@@ -41,5 +41,5 @@ func NewClient(ctx context.Context, config ClientConfig) (Client, error) {
 			ProviderUserAgent: config.ProviderUserAgent,
 		}), nil
 	}
-	return nil, errors.New("Invalid provider configuration. Either Connect credentials (\"connect_token\" and \"connect_url\") or Service Account (\"service_account_token\" or \"account\") credentials should be set.")
+	return nil, errors.New("Invalid provider configuration. Either Connect credentials (\"connect_token\" and \"connect_url\") or Service Account (\"service_account_token\") or \"account\"  should be set.")
 }

internal/provider/provider.go

@@ -52,25 +52,11 @@ func (p *OnePasswordProvider) Schema(ctx context.Context, req provider.SchemaReq
 			"connect_url": schema.StringAttribute{
 				MarkdownDescription: "The HTTP(S) URL where your 1Password Connect server can be found. Can also be sourced `OP_CONNECT_HOST` environment variable. Provider will use 1Password Connect server if set.",
 				Optional:            true,
-				Validators: []validator.String{
-					stringvalidator.ConflictsWith(
-						path.Expressions{
-							path.MatchRoot("url"),
-						}...,
-					),
-				},
 			},
 			"connect_token": schema.StringAttribute{
 				MarkdownDescription: "A valid token for your 1Password Connect server. Can also be sourced from `OP_CONNECT_TOKEN` environment variable. Provider will use 1Password Connect server if set.",
 				Optional:            true,
 				Sensitive:           true,
-				Validators: []validator.String{
-					stringvalidator.ConflictsWith(
-						path.Expressions{
-							path.MatchRoot("token"),
-						}...,
-					),
-				},
 			},
 			"url": schema.StringAttribute{
 				MarkdownDescription: "The HTTP(S) URL where your 1Password Connect server can be found. Can also be sourced `OP_CONNECT_HOST` environment variable. Provider will use 1Password Connect server if set. Deprecated: Use `connect_url` instead.",
@@ -82,6 +68,7 @@ func (p *OnePasswordProvider) Schema(ctx context.Context, req provider.SchemaReq
 						}...,
 					),
 				},
+				DeprecationMessage: "The \"url\" field is deprecated and will be removed in a future version. Use \"connect_url\" instead.",
 			},
 			"token": schema.StringAttribute{
 				MarkdownDescription: "A valid token for your 1Password Connect server. Can also be sourced from `OP_CONNECT_TOKEN` environment variable. Provider will use 1Password Connect server if set. Deprecated: Use `connect_token` instead.",
@@ -94,6 +81,7 @@ func (p *OnePasswordProvider) Schema(ctx context.Context, req provider.SchemaReq
 						}...,
 					),
 				},
+				DeprecationMessage: "The \"token\" field is deprecated and will be removed in a future version. Use \"connect_token\" instead.",
 			},
 			"service_account_token": schema.StringAttribute{
 				MarkdownDescription: "A valid 1Password service account token. Can also be sourced from `OP_SERVICE_ACCOUNT_TOKEN` environment variable.",
@@ -134,11 +122,9 @@ func (p *OnePasswordProvider) Configure(ctx context.Context, req provider.Config
 
 	// Old field names - these are deprecated and will be removed in a future version.
 	if !config.ConnectHostOld.IsNull() {
-		resp.Diagnostics.AddWarning("Deprecated field", "The \"url\" field is deprecated and will be removed in a future version. Use \"connect_url\" instead.")
 		connectHost = config.ConnectHostOld.ValueString()
 	}
 	if !config.ConnectTokenOld.IsNull() {
-		resp.Diagnostics.AddWarning("Deprecated field", "The \"token\" field is deprecated and will be removed in a future version. Use \"connect_token\" instead.")
 		connectToken = config.ConnectTokenOld.ValueString()
 	}
 

test/e2e/item_resource_test.go

@@ -554,8 +554,8 @@ func TestAccItemResourceTags(t *testing.T) {
 		tags []string
 	}{
 		{"CREATE_ITEM_WITH_2_TAGS", []string{"firstTestTag", "secondTestTag"}},
-		// {"ADD_3RD_TAG", []string{"firstTestTag", "secondTestTag", "thirdTestTag"}},
-		// {"REMOVE_2_TAGS", []string{"firstTestTag"}},
+		{"ADD_3RD_TAG", []string{"firstTestTag", "secondTestTag", "thirdTestTag"}},
+		{"REMOVE_2_TAGS", []string{"firstTestTag"}},
 	}
 
 	testVaultID := vault.GetTestVaultID(t)