test: fix TestAcquire/bigquery/resourceid-userspecifiedresourceid

[jingyih]

Add back the projectRef, which was likely removed unintentionally in a previous PR #2814.

Fixes #3170

[yuwenma]

/lgtm
/approve

Thank you for the fix! @xiaoweim Could you investigate why the presubmit cannot catch this? (I think porjectRef is required)

[google-oss-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: yuwenma

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [yuwenma]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[jingyih]

Thank you for the fix! @xiaoweim Could you investigate why the presubmit cannot catch this? (I think porjectRef is required)

I think the "container_annotation_handler" webhook currently adds projectRef for this resource.

I took a closer look, TestAcquire has a different test flow than other dynamic integration e2e tests (e.g. the most common test TestCreateNoChangeUpdateDelete). In TestAcquire, the test performs a TF get (direct controller for this resource is not enabled by default yet) before the webhooks get a chance to add projectRef to the resource. Therefore causing the error.

[yuwenma]

Thank you for the fix! @xiaoweim Could you investigate why the presubmit cannot catch this? (I think porjectRef is required)

I think the "container_annotation_handler" webhook currently adds projectRef for this resource.

I took a closer look, TestAcquire has a different test flow than other dynamic integration e2e tests (e.g. the most common test TestCreateNoChangeUpdateDelete). In TestAcquire, the test performs a TF get (direct controller for this resource is not enabled by default yet) before the webhooks get a chance to add projectRef to the resource. Therefore causing the error.

Thank you for the investigation, @jingyih! Add @maqiuyujoyce to the thread, this is something the dynamic test can improve