chore(gcp-resource-detector): unit test to prevent a regression and ensure previously set attributes are not overwritten with blank values.

[ajbenjam]

Which problem is this PR solving?

• This PR adds a unit test to prevent a regression in behavior that was present prior to commit 143a1f4 (refactor(resource-detector-gcp): migrate away from getEnv() #2705)

Short description of the changes

• Prior to commit 143a1f4 (refactor(resource-detector-gcp): migrate away from getEnv() #2705), the behavior was as follows:
  • OTEL_RESOURCE_ATTRIBUTES = 'k8s.namespace.name=my-namespace'
  • NAMESPACE is undefined
  • EnvDetector runs and adds 'k8s.namespace.name=my-namespace' to attributes list
  • GcpDetector runs and sets 'k8s.namespace.name=' (blank)

The GcpDetector should only set 'k8s.namespace.name' if the source environment variable is defined.

[linux-foundation-easycla]

The committers listed above are authorized under a signed CLA.

• ✅ login: ajbenjam / name: Adam Benjamin (ae16321)

[ajbenjam]

/easycla

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.31%. Comparing base (4b51e60) to head (22b3ad3).
Report is 17 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2755      +/-   ##
==========================================
- Coverage   91.53%   91.31%   -0.23%     
==========================================
  Files         171      166       -5     
  Lines        8172     7861     -311     
  Branches     1660     1619      -41     
==========================================
- Hits         7480     7178     -302     
+ Misses        692      683       -9     

see 7 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.