diff mbox

st_destroy_session: remove invalid testcase

Message ID 1380918492-8413-1-git-send-email-tigran.mkrtchyan@desy.de (mailing list archive)
State New, archived
Headers show

Commit Message

Mkrtchyan, Tigran Oct. 4, 2013, 8:28 p.m. UTC
we cant expect NFS4ERR_NOT_ONLY_OP if spec
enforces NFS4ERR_OP_NOT_IN_SESSION

Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>
---
 nfs4.1/server41tests/st_destroy_session.py | 3 ---
 1 file changed, 3 deletions(-)

Comments

J. Bruce Fields Oct. 7, 2013, 3:37 p.m. UTC | #1
On Fri, Oct 04, 2013 at 10:28:12PM +0200, Tigran Mkrtchyan wrote:
> we cant expect NFS4ERR_NOT_ONLY_OP if spec
> enforces NFS4ERR_OP_NOT_IN_SESSION

I guess we could just add that to the list of allowable error returns,
but... yeah, OK, let's just remove that test.  Applied.

--b.

> 
> Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>
> ---
>  nfs4.1/server41tests/st_destroy_session.py | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/nfs4.1/server41tests/st_destroy_session.py b/nfs4.1/server41tests/st_destroy_session.py
> index 1feb671..f4f0762 100644
> --- a/nfs4.1/server41tests/st_destroy_session.py
> +++ b/nfs4.1/server41tests/st_destroy_session.py
> @@ -164,6 +164,3 @@ def testDestoryNotSoleOps(t, env):
>      sid = sess1.sessionid
>      res = c.c.compound([op.destroy_session(sess1.sessionid), op.putrootfh()])
>      check(res, NFS4ERR_NOT_ONLY_OP)
> -
> -    res = c.c.compound([op.putrootfh(), op.destroy_session(sess1.sessionid)])
> -    check(res, NFS4ERR_NOT_ONLY_OP)
> -- 
> 1.8.3.1
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-nfs" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/nfs4.1/server41tests/st_destroy_session.py b/nfs4.1/server41tests/st_destroy_session.py
index 1feb671..f4f0762 100644
--- a/nfs4.1/server41tests/st_destroy_session.py
+++ b/nfs4.1/server41tests/st_destroy_session.py
@@ -164,6 +164,3 @@  def testDestoryNotSoleOps(t, env):
     sid = sess1.sessionid
     res = c.c.compound([op.destroy_session(sess1.sessionid), op.putrootfh()])
     check(res, NFS4ERR_NOT_ONLY_OP)
-
-    res = c.c.compound([op.putrootfh(), op.destroy_session(sess1.sessionid)])
-    check(res, NFS4ERR_NOT_ONLY_OP)