Potential fix for code scanning alert no. 2: Variable defined multiple times

[zliang-akamai]

Potential fix for https://github.com/linode/py-metadata/security/code-scanning/2

In general, when a variable or function is defined multiple times without the earlier definition being used, you should either remove the earlier definition or rename one of them so all intended code paths are reachable. For pytest tests, having two functions with the same name in the same module means only the second definition is discoverable and runnable; the first is effectively lost.

Here, the best fix without changing existing functionality is to give the synchronous and asynchronous tests distinct names, so both can run. We keep the bodies exactly the same and only adjust the first function name to something unique, e.g., test_get_ssh_keys_sync. This preserves the synchronous test, preserves the async test and its marker, and resolves the CodeQL complaint that the first assignment to test_get_ssh_keys is unnecessary.

Concretely, in test/integration/test_ssh.py, change line 8 from def test_get_ssh_keys(client: MetadataClient): to def test_get_ssh_keys_sync(client: MetadataClient):. No imports, methods, or other definitions are needed beyond this rename.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

[vshanthe]

Valid fix 👍