fix: add fail index to streaming sequence

[leahecole]

Gal was working on this before he moved to another team and I am opening this to make sure I tie up loose ends by finishing his work. I added a test and then I regenerated the support files because of the proto change. It seemed to regenerate more than just my service, but I removed anything unrelated to sequence. If I should re-include those, let me know. cc @galz10

[noahdietz]

I don't think you need to include the generated code changes in this PR. They should be automatically included in CI for testing, then contributed back via an automatic PR upon merge.

[blakeli0]

@noahdietz Is this a valid resource reference? I think the field reference another resource should be of String type per AIP-122, also the Sequence resource is of pattern sequences/{sequence} which seems do not fit here.

[noahdietz]

Oh yeah definitely not.

[leahecole]

good catch! I accepted Noah's suggestion which iiuc just removed what used to be line 30 with this reference type. I added a clarifying comment, good call @blakeli0. I want to make sure I'm interpreting these comments and the code suggestion from @noahdietz correctly though - do I not need to include a resource type at all? Or, should I be linking to something like

gapic-showcase/schema/google/showcase/v1beta1/sequence.proto

Line 233 in 3188c8a

string content = 1;

?

[noahdietz]

Nope, not necessary. The resource here is the Sequence being attempted. The content is just data.