test: fix conformance tests for SQLAlchemy 2.0

[olavloite]

The conformance tests for SQLAlchemy 2.0 were failing due to a dependency conflict for OpenTelemetry. This change removes the use of OpenTelemetry entirely from the tests, as the version that is currently used by the Spanner client library triggers a deprecation warning when used with SQLAlchemy. That in itself is not a big problem, except that the SQLAlchemy tests verify that there are no warnings, and there are no reasonable ways to ignore these warnings, other than just getting rid of the OpenTelemetry usage.

See https://github.com/googleapis/python-spanner-sqlalchemy/actions/runs/22582034832/job/65569068380?pr=825 for an example of the build error that is being fixed.

[gemini-code-assist]

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

This pull request addresses a critical issue where conformance tests for SQLAlchemy 2.0 were failing due to a dependency conflict related to OpenTelemetry. The changes involve removing OpenTelemetry from the test suite to avoid deprecation warnings and ensure the tests pass reliably. This improves the overall stability and reliability of the testing process.

Highlights

• Dependency Conflict Resolution: The primary goal of this PR is to resolve a dependency conflict with OpenTelemetry that was causing conformance tests to fail in SQLAlchemy 2.0.
• OpenTelemetry Removal: To address the conflict, the PR removes the use of OpenTelemetry from the tests, as the current version used by the Spanner client library triggers a deprecation warning that cannot be easily ignored.
• Test Stability: By removing OpenTelemetry, the PR aims to stabilize the conformance tests and prevent failures due to deprecation warnings.

Changelog

• google/cloud/sqlalchemy_spanner/_opentelemetry_tracing.py
  • Refactored to remove direct tracer instantiation to avoid deprecation warnings.
• noxfile.py
  • Updated session install to remove tracing and force reinstall.
• test/conftest.py
  • Monkeypatched trace_call to avoid OpenTelemetry usage entirely.
• test/test_suite_20.py
  • Removed unused imports and skipped tests.

Activity

• The PR addresses a failing build error in the conformance tests for SQLAlchemy 2.0.
• The changes focus on removing OpenTelemetry to resolve dependency conflicts and deprecation warnings.
• The PR aims to stabilize the test suite and ensure reliable test execution.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

This pull request fixes failing conformance tests for SQLAlchemy 2.0 by removing OpenTelemetry usage from the test environment. This is achieved by not installing the tracing extras and by monkey-patching trace_call to a no-op function. The changes are logical and effectively address the described issue. I have a couple of suggestions to improve code maintainability.

[bhatt4982]

LGTM...