Skip to content

Commit e8608b1

Browse files
yyyu-googlecopybara-github
authored andcommitted
feat!: consolidate NewUserContentFrom* and NewModelContentFrom* functions into NewContentFrom* to make API simpler
PiperOrigin-RevId: 741683659
1 parent a533e85 commit e8608b1

File tree

4 files changed

+40
-101
lines changed

4 files changed

+40
-101
lines changed

go.mod

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ require (
77
cloud.google.com/go/auth v0.9.3
88
github.com/google/go-cmp v0.6.0
99
github.com/gorilla/websocket v1.5.3
10-
golang.org/x/oauth2 v0.23.0
1110
)
1211

1312
require (

go.sum

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,6 @@ golang.org/x/net v0.0.0-20201110031124-69a78807bb2b/go.mod h1:sp8m0HH+o8qH0wwXwY
7171
golang.org/x/net v0.29.0 h1:5ORfpBpCs4HzDYoodCDBbwHzdR5UrLBZ3sOnUJmFoHo=
7272
golang.org/x/net v0.29.0/go.mod h1:gLkgy8jTGERgjzMic6DS9+SP0ajcu6Xu3Orq/SpETg0=
7373
golang.org/x/oauth2 v0.0.0-20180821212333-d2e6202438be/go.mod h1:N/0e6XlmueqKjAGxoOufVs8QHGRruUQn6yWY3a++T0U=
74-
golang.org/x/oauth2 v0.23.0 h1:PbgcYx2W7i4LvjJWEbf0ngHV6qJYr86PkAV3bXdLEbs=
75-
golang.org/x/oauth2 v0.23.0/go.mod h1:XYTD2NtWslqkgxebSiOHnXEap4TF09sJSc7H1sXbhtI=
7674
golang.org/x/sync v0.0.0-20180314180146-1d60e4601c6f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
7775
golang.org/x/sync v0.0.0-20181108010431-42b317875d0f/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=
7876
golang.org/x/sync v0.0.0-20190423024810-112230192c58/go.mod h1:RxMgew5VJxzue5/jJTE5uejpjVlOe/izrB70Jof72aM=

types.go

Lines changed: 26 additions & 84 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

types_test.go

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -375,7 +375,7 @@ func TestNewUserContentFromParts(t *testing.T) {
375375
Role: "user",
376376
}
377377

378-
result := NewUserContentFromParts(parts)
378+
result := NewContentFromParts(parts, "user")
379379
if !reflect.DeepEqual(result, expected) {
380380
t.Fatalf("expected %v, got %v", expected, result)
381381
}
@@ -390,7 +390,7 @@ func TestNewUserContentFromText(t *testing.T) {
390390
Role: "user",
391391
}
392392

393-
result := NewUserContentFromText(text)
393+
result := NewContentFromText(text, "user")
394394
if !reflect.DeepEqual(result, expected) {
395395
t.Fatalf("expected %v, got %v", expected, result)
396396
}
@@ -406,7 +406,7 @@ func TestNewUserContentFromBytes(t *testing.T) {
406406
Role: "user",
407407
}
408408

409-
result := NewUserContentFromBytes(data, mimeType)
409+
result := NewContentFromBytes(data, mimeType, "user")
410410
if !reflect.DeepEqual(result, expected) {
411411
t.Fatalf("expected %v, got %v", expected, result)
412412
}
@@ -422,7 +422,7 @@ func TestNewUserContentFromURI(t *testing.T) {
422422
Role: "user",
423423
}
424424

425-
result := NewUserContentFromURI(fileURI, mimeType)
425+
result := NewContentFromURI(fileURI, mimeType, "user")
426426
if !reflect.DeepEqual(result, expected) {
427427
t.Fatalf("expected %v, got %v", expected, result)
428428
}
@@ -438,7 +438,7 @@ func TestNewUserContentFromFunctionResponse(t *testing.T) {
438438
Role: "user",
439439
}
440440

441-
result := NewUserContentFromFunctionResponse(funcName, response)
441+
result := NewContentFromFunctionResponse(funcName, response, "user")
442442
if !reflect.DeepEqual(result, expected) {
443443
t.Fatalf("expected %v, got %v", expected, result)
444444
}
@@ -454,7 +454,7 @@ func TestNewUserContentFromExecutableCode(t *testing.T) {
454454
Role: "user",
455455
}
456456

457-
result := NewUserContentFromExecutableCode(code, language)
457+
result := NewContentFromExecutableCode(code, language, "user")
458458
if !reflect.DeepEqual(result, expected) {
459459
t.Fatalf("expected %v, got %v", expected, result)
460460
}
@@ -470,7 +470,7 @@ func TestNewUserContentFromCodeExecutionResult(t *testing.T) {
470470
Role: "user",
471471
}
472472

473-
result := NewUserContentFromCodeExecutionResult(outcome, output)
473+
result := NewContentFromCodeExecutionResult(outcome, output, "user")
474474
if !reflect.DeepEqual(result, expected) {
475475
t.Fatalf("expected %v, got %v", expected, result)
476476
}
@@ -486,7 +486,7 @@ func TestNewModelContentFromParts(t *testing.T) {
486486
Role: "model",
487487
}
488488

489-
result := NewModelContentFromParts(parts)
489+
result := NewContentFromParts(parts, "model")
490490
if !reflect.DeepEqual(result, expected) {
491491
t.Fatalf("expected %v, got %v", expected, result)
492492
}
@@ -501,7 +501,7 @@ func TestNewModelContentFromText(t *testing.T) {
501501
Role: "model",
502502
}
503503

504-
result := NewModelContentFromText(text)
504+
result := NewContentFromText(text, "model")
505505
if !reflect.DeepEqual(result, expected) {
506506
t.Fatalf("expected %v, got %v", expected, result)
507507
}
@@ -517,7 +517,7 @@ func TestNewModelContentFromBytes(t *testing.T) {
517517
Role: "model",
518518
}
519519

520-
result := NewModelContentFromBytes(data, mimeType)
520+
result := NewContentFromBytes(data, mimeType, "model")
521521
if !reflect.DeepEqual(result, expected) {
522522
t.Fatalf("expected %v, got %v", expected, result)
523523
}
@@ -533,7 +533,7 @@ func TestNewModelContentFromURI(t *testing.T) {
533533
Role: "model",
534534
}
535535

536-
result := NewModelContentFromURI(fileURI, mimeType)
536+
result := NewContentFromURI(fileURI, mimeType, "model")
537537
if !reflect.DeepEqual(result, expected) {
538538
t.Fatalf("expected %v, got %v", expected, result)
539539
}
@@ -549,7 +549,7 @@ func TestNewModelContentFromFunctionCall(t *testing.T) {
549549
Role: "model",
550550
}
551551

552-
result := NewModelContentFromFunctionCall(funcName, args)
552+
result := NewContentFromFunctionCall(funcName, args, "model")
553553
if !reflect.DeepEqual(result, expected) {
554554
t.Fatalf("expected %v, got %v", expected, result)
555555
}
@@ -565,7 +565,7 @@ func TestNewModelContentFromExecutableCode(t *testing.T) {
565565
Role: "model",
566566
}
567567

568-
result := NewModelContentFromExecutableCode(code, language)
568+
result := NewContentFromExecutableCode(code, language, "model")
569569
if !reflect.DeepEqual(result, expected) {
570570
t.Fatalf("expected %v, got %v", expected, result)
571571
}
@@ -581,7 +581,7 @@ func TestNewModelContentFromCodeExecutionResult(t *testing.T) {
581581
Role: "model",
582582
}
583583

584-
result := NewModelContentFromCodeExecutionResult(outcome, output)
584+
result := NewContentFromCodeExecutionResult(outcome, output, "model")
585585
if !reflect.DeepEqual(result, expected) {
586586
t.Fatalf("expected %v, got %v", expected, result)
587587
}

0 commit comments

Comments
 (0)