Drop Cursor.status property

diff --git a/psycopg/psycopg/cursor.py b/psycopg/psycopg/cursor.py
index 17807790ccff71e1c36b87f12d6da46612eff3bb..f1f9274d7bc81d32f655e8e52b5e7ebe2bb81926 100644 (file)
--- a/psycopg/psycopg/cursor.py
+++ b/psycopg/psycopg/cursor.py
@@ -98,12 +98,6 @@ class BaseCursor(Generic[ConnectionType, Row]):
         """`True` if the cursor is closed."""
         return self._closed
 
-    @property
-    def status(self) -> Optional[pq.ExecStatus]:
-        # TODO: do we want this?
-        res = self.pgresult
-        return pq.ExecStatus(res.status) if res else None
-
     @property
     def query(self) -> Optional[bytes]:
         """The last query sent to the server, if available."""

diff --git a/tests/test_cursor.py b/tests/test_cursor.py
index 2962afa7d0c6a8a2127a5915750710257c3cd915..d1875aeec3ea72ed738c86fdd7b15ac57072d29c 100644 (file)
--- a/tests/test_cursor.py
+++ b/tests/test_cursor.py
@@ -42,17 +42,6 @@ def test_weakref(conn):
     assert w() is None
 
 
-def test_status(conn):
-    cur = conn.cursor()
-    assert cur.status is None
-    cur.execute("reset all")
-    assert cur.status == cur.ExecStatus.COMMAND_OK
-    cur.execute("select 1")
-    assert cur.status == cur.ExecStatus.TUPLES_OK
-    cur.close()
-    assert cur.status == cur.ExecStatus.TUPLES_OK
-
-
 def test_execute_many_results(conn):
     cur = conn.cursor()
     assert cur.nextset() is None
@@ -84,7 +73,7 @@ def test_execute_sequence(conn):
 def test_execute_empty_query(conn, query):
     cur = conn.cursor()
     cur.execute(query)
-    assert cur.status == cur.ExecStatus.EMPTY_QUERY
+    assert cur.pgresult.status == cur.ExecStatus.EMPTY_QUERY
     with pytest.raises(psycopg.ProgrammingError):
         cur.fetchone()
 

diff --git a/tests/test_cursor_async.py b/tests/test_cursor_async.py
index 0c61a2fa5df8f9e553a2a5db3f7cbd71bef8ae53..bbc81bdd781f9ce04e67690b95323000d92d15d8 100644 (file)
--- a/tests/test_cursor_async.py
+++ b/tests/test_cursor_async.py
@@ -42,17 +42,6 @@ async def test_weakref(aconn):
     assert w() is None
 
 
-async def test_status(aconn):
-    cur = aconn.cursor()
-    assert cur.status is None
-    await cur.execute("reset all")
-    assert cur.status == cur.ExecStatus.COMMAND_OK
-    await cur.execute("select 1")
-    assert cur.status == cur.ExecStatus.TUPLES_OK
-    await cur.close()
-    assert cur.status == cur.ExecStatus.TUPLES_OK
-
-
 async def test_execute_many_results(aconn):
     cur = aconn.cursor()
     assert cur.nextset() is None
@@ -87,7 +76,7 @@ async def test_execute_sequence(aconn):
 async def test_execute_empty_query(aconn, query):
     cur = aconn.cursor()
     await cur.execute(query)
-    assert cur.status == cur.ExecStatus.EMPTY_QUERY
+    assert cur.pgresult.status == cur.ExecStatus.EMPTY_QUERY
     with pytest.raises(psycopg.ProgrammingError):
         await cur.fetchone()