forked from pool/python-paramiko
Matej Cepl
e9bca5f938
update to 2.11.0 OBS-URL: https://build.opensuse.org/request/show/979399 OBS-URL: https://build.opensuse.org/package/show/devel:languages:python/python-paramiko?expand=0&rev=105
68 lines
2.6 KiB
Diff
68 lines
2.6 KiB
Diff
From 5844aa0270d3ad8feab4bf1023e35aa4fc255b6c Mon Sep 17 00:00:00 2001
|
|
From: =?UTF-8?q?Micha=C5=82=20G=C3=B3rny?= <mgorny@gentoo.org>
|
|
Date: Thu, 16 Apr 2020 09:22:59 +0200
|
|
Subject: [PATCH] Replace pytest-relaxed with plain pytest.raises
|
|
|
|
There is really no technical reason to bring pytest-relaxed to call
|
|
@raises as a decorator while plain pytest works just fine. Plus,
|
|
pytest.raises() is used in test_sftp already.
|
|
|
|
pytest-relaxed causes humongous breakage to other packages
|
|
on the system. It has been banned from Gentoo for this reason.
|
|
---
|
|
dev-requirements.txt | 1 - (removed from patch=
|
|
setup.cfg | 3 ---
|
|
tests/test_client.py | 20 ++++++++++----------
|
|
3 files changed, 10 insertions(+), 14 deletions(-)
|
|
|
|
Index: paramiko-2.11.0/tests/test_client.py
|
|
===================================================================
|
|
--- paramiko-2.11.0.orig/tests/test_client.py 2022-05-17 03:04:13.000000000 +0200
|
|
+++ paramiko-2.11.0/tests/test_client.py 2022-05-26 22:57:20.216831045 +0200
|
|
@@ -33,7 +33,7 @@
|
|
import weakref
|
|
from tempfile import mkstemp
|
|
|
|
-from pytest_relaxed import raises
|
|
+import pytest
|
|
from mock import patch, Mock
|
|
|
|
import paramiko
|
|
@@ -733,11 +733,11 @@
|
|
|
|
# TODO: more granular exception pending #387; should be signaling "no auth
|
|
# methods available" because no key and no password
|
|
- @raises(SSHException)
|
|
@requires_sha1_signing
|
|
def test_passphrase_kwarg_not_used_for_password_auth(self):
|
|
- # Using the "right" password in the "wrong" field shouldn't work.
|
|
- self._test_connection(passphrase="pygmalion")
|
|
+ with pytest.raises(SSHException):
|
|
+ # Using the "right" password in the "wrong" field shouldn't work.
|
|
+ self._test_connection(passphrase="pygmalion")
|
|
|
|
@requires_sha1_signing
|
|
def test_passphrase_kwarg_used_for_key_passphrase(self):
|
|
@@ -757,15 +757,15 @@
|
|
password="television",
|
|
)
|
|
|
|
- @raises(AuthenticationException) # TODO: more granular
|
|
@requires_sha1_signing
|
|
def test_password_kwarg_not_used_for_passphrase_when_passphrase_kwarg_given( # noqa
|
|
self
|
|
):
|
|
# Sanity: if we're given both fields, the password field is NOT used as
|
|
# a passphrase.
|
|
- self._test_connection(
|
|
- key_filename=_support("test_rsa_password.key"),
|
|
- password="television",
|
|
- passphrase="wat? lol no",
|
|
- )
|
|
+ with pytest.raises(AuthenticationException):
|
|
+ self._test_connection(
|
|
+ key_filename=_support("test_rsa_password.key"),
|
|
+ password="television",
|
|
+ passphrase="wat? lol no",
|
|
+ )
|