test: fix failing grpc agent otel tests

[santigimeno]

Make sure we take service.version into account once the fix for appVersion was introduced.
Make sure we use the common checkResource() function in the tracing test too.

Summary by CodeRabbit

• Tests

  • Refactored tests to use centralized resource validation that now accepts agent identifier and configuration.

• Chores

  • Extended telemetry/resource attributes to include service.version (mapped from application version).

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

Refactors tests to use a shared checkResource(resource, agentId, config) from test/common/nsolid-grpc-agent, retrieves agent config via TestClient.config(), and updates the shared validator to expect service.version derived from config.appVersion.

Changes

Cohort / File(s)	Summary
Test update for shared resource validation test/agents/test-grpc-tracing.mjs	Imports checkResource from test/common/nsolid-grpc-agent, removes local checkResource, calls checkResource(resource, agentId, config) throughout, and obtains config via TestClient.config().
Shared resource validation enhancement test/common/nsolid-grpc-agent/index.js	checkResource now includes service.version in expected attributes, mapped from config.appVersion.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• test: fix failing grpc agent otel tests #421: Same code-level change — tests switched to the shared checkResource(resource, agentId, config) and shared validator extended to expect service.version.

Suggested reviewers

• RafaelGSS

Poem

🐇 I hopped through tests and shared my tune,
One check to find each resource's sign —
Agent id, config, version soon,
All bundled neat in one small line.
Carrots coded, trails align.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'test: fix failing grpc agent otel tests' clearly identifies the main change: fixing failing gRPC agent OpenTelemetry tests, which directly aligns with the PR objectives.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch santi/fix_grpc_resource_tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}