fix: explicit depends on for budget project data source

[bharathkkb]

fixes #555

[comment-bot-dev]

Thanks for the PR! 🚀
✅ Lint checks have passed.

[morgante]

@bharathkkb Were you able to reproduce and confirm this fixes it?

[bharathkkb]

@morgante I tested dynamic_shared_vpc locally.
On main when I plan, it errors with

Refreshing Terraform state in-memory prior to plan...
The refreshed state will be used to calculate this plan, but will not be
persisted to local or remote state storage.
------------------------------------------------------------------------
Error: the user does not have permission to access Project "b-pf-ci-shared2-svc-ab19df" or it may not exist



Error: the user does not have permission to access Project "pf-ci-shared2-svc-ab19df" or it may not exist

With this fix looks like it is deferring the read correctly

An execution plan has been generated and is shown below.
Resource actions are indicated with the following symbols:
  + create
 <= read (data resources)

Terraform will perform the following actions:

...

  # module.example.module.host-project.module.budget.data.google_project.project[0] will be read during apply
  # (config refers to values not yet known)
 <= data "google_project" "project"  {
      + auto_create_network = (known after apply)
      + billing_account     = (known after apply)
      + folder_id           = (known after apply)
      + id                  = (known after apply)
      + labels              = (known after apply)
      + name                = (known after apply)
      + number              = (known after apply)
      + org_id              = (known after apply)
      + project_id          = (known after apply)
      + skip_delete         = (known after apply)
    }

and looks like the integrations tests passed too.

[morgante]

Great, thanks for the fix.