krb5 commit: Allow small errors in t_renew.py tests

Greg Hudson <[email protected]>
Newsgroups gmane.comp.encryption.kerberos.cvs
Message-ID <[email protected]>
https://github.com/krb5/krb5/commit/4cfb64e362443be36a4cb92fb8f5d1d93280fb3f
commit 4cfb64e362443be36a4cb92fb8f5d1d93280fb3f
Author: Greg Hudson <[email protected]>
Date:   Thu Aug 31 14:11:43 2017 -0400

    Allow small errors in t_renew.py tests
    
    ticket: 8609

 src/tests/t_renew.py |    8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/src/tests/t_renew.py b/src/tests/t_renew.py
index aa58ece..034190c 100755
--- a/src/tests/t_renew.py
+++ b/src/tests/t_renew.py
@@ -31,9 +31,11 @@ def test(testname, life, rlife, exp_life, exp_rlife, env=None):
     endtime = times[1]
     rtime = times[2] if len(times) >= 3 else None
 
-    # Check the ticket lifetime against expectations.
+    # Check the ticket lifetime against expectations.  If the lifetime
+    # was determined by the request, there may be a small error
+    # because KDC requests contain an end time rather than a lifetime.
     life = (endtime - starttime).seconds
-    if life != exp_life:
+    if abs(life - exp_life) > 5:
         fail('%s: expected life %d, got %d' % (testname, exp_life, life))
 
     # Check the ticket renewable lifetime against expectations.
@@ -43,7 +45,7 @@ def test(testname, life, rlife, exp_life, exp_rlife, env=None):
         fail('%s: ticket is renewable but has no renew_till' % testname)
     if rtime is not None:
         rlife = (rtime - starttime).seconds
-        if rlife != exp_rlife:
+        if abs(rlife - exp_rlife) > 5:
             fail('%s: expected rlife %d, got %d' (testname, exp_rlife, rlife))
 
 # Get renewable tickets.
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.