[med-svn] [Git][med-team/biomaj3-download][master] 3 commits: New upstream version 3.2.6

Olivier Sallou (@osallou) gitlab at salsa.debian.org
Mon Feb 14 07:57:14 GMT 2022



Olivier Sallou pushed to branch master at Debian Med / biomaj3-download


Commits:
9cba77fc by Olivier Sallou at 2022-02-14T07:51:37+00:00
New upstream version 3.2.6
- - - - -
cc920a4b by Olivier Sallou at 2022-02-14T07:51:37+00:00
Update upstream source from tag 'upstream/3.2.6'

Update to upstream version '3.2.6'
with Debian dir 8a250986e212d134572355a6f989f8ffeaa977a1
- - - - -
690c8e63 by Olivier Sallou at 2022-02-14T07:55:09+00:00
new upstream 3.2.6 version fixing #1005457

- - - - -


5 changed files:

- CHANGES.txt
- biomaj_download/download/interface.py
- debian/changelog
- setup.py
- tests/biomaj_tests.py


Changes:

=====================================
CHANGES.txt
=====================================
@@ -1,3 +1,9 @@
+3.2.6:
+  Disabled test with tenacity, compat.make_retry_state removed from lib
+
+3.2.5:
+  Fix regex handling for **/*
+
 3.2.4:
   #39 directhttp download issues
     biomaj sends file name instead of file dict, workaround this


=====================================
biomaj_download/download/interface.py
=====================================
@@ -3,7 +3,7 @@ import logging
 import datetime
 import time
 import re
-import copy
+# import copy
 
 import tenacity
 from simpleeval import simple_eval, ast
@@ -264,14 +264,15 @@ class DownloadInterface(object):
                 # Check that it is an instance of stop_base
                 if not isinstance(stop_cond, tenacity.stop.stop_base):
                     raise ValueError(stop_condition + " doesn't yield a stop condition")
+                # Disabled test, compat.make_retry_state removed from lib
                 # Test that this is a correct stop condition by calling it.
                 # We use a deepcopy to be sure to not alter the object (even
                 # if it seems that calling a wait policy doesn't modify it).
-                try:
-                    s = copy.deepcopy(stop_cond)
-                    s(tenacity.compat.make_retry_state(0, 0))
-                except Exception:
-                    raise ValueError(stop_condition + " doesn't yield a stop condition")
+                # try:
+                #    s = copy.deepcopy(stop_cond)
+                #    s(tenacity.compat.make_retry_state(0, 0))
+                # except Exception:
+                #    raise ValueError(stop_condition + " doesn't yield a stop condition")
             except Exception as e:
                 raise ValueError("Error while parsing stop condition: %s" % e)
         else:
@@ -290,14 +291,15 @@ class DownloadInterface(object):
                 # Check that it is an instance of wait_base
                 if not isinstance(wait_pol, tenacity.wait.wait_base):
                     raise ValueError(wait_policy + " doesn't yield a wait policy")
+                # Disabled test, compat.make_retry_state removed from lib
                 # Test that this is a correct wait policy by calling it.
                 # We use a deepcopy to be sure to not alter the object (even
                 # if it seems that calling a stop condition doesn't modify it).
-                try:
-                    w = copy.deepcopy(wait_pol)
-                    w(tenacity.compat.make_retry_state(0, 0))
-                except Exception:
-                    raise ValueError(wait_policy + " doesn't yield a wait policy")
+                # try:
+                #    w = copy.deepcopy(wait_pol)
+                #    w(tenacity.compat.make_retry_state(0, 0))
+                # except Exception:
+                #    raise ValueError(wait_policy + " doesn't yield a wait policy")
             except Exception as e:
                 raise ValueError("Error while parsing wait policy: %s" % e)
         else:
@@ -377,19 +379,13 @@ class DownloadInterface(object):
                             rfile['name'] = prefix + '/' + rfile['name']
                         self._append_file_to_download(rfile)
                         self.logger.debug('Download:File:MatchRegExp:' + rfile['name'])
-                    return
+                    # return
                 for direlt in dir_list:
                     subdir = direlt['name']
                     self.logger.debug('Download:File:Subdir:Check:' + subdir)
                     if pattern == '**/*':
                         (subfile_list, subdirs_list) = self.list(prefix + '/' + subdir + '/')
                         self.match([pattern], subfile_list, subdirs_list, prefix + '/' + subdir, True)
-                        for rfile in file_list:
-                            if pattern == '**/*' or re.match(pattern, rfile['name']):
-                                if prefix != '':
-                                    rfile['name'] = prefix + '/' + rfile['name']
-                                self._append_file_to_download(rfile)
-                                self.logger.debug('Download:File:MatchRegExp:' + rfile['name'])
                     else:
                         if re.match(subdirs_pattern[0], subdir):
                             self.logger.debug('Download:File:Subdir:Match:' + subdir)


=====================================
debian/changelog
=====================================
@@ -1,3 +1,10 @@
+biomaj3-download (3.2.6-1) unstable; urgency=medium
+
+  * New upstream release, fixing tenacity breaking changes and minor fixes 
+    (Closes: #1005457).
+
+ -- Olivier Sallou <osallou at debian.org>  Mon, 14 Feb 2022 07:52:05 +0000
+
 biomaj3-download (3.2.4-1) unstable; urgency=medium
 
   [ Michael R. Crusoe ]


=====================================
setup.py
=====================================
@@ -22,7 +22,7 @@ config = {
     'url': 'http://biomaj.genouest.org',
     'download_url': 'http://biomaj.genouest.org',
     'author_email': 'olivier.sallou at irisa.fr',
-    'version': '3.2.4',
+    'version': '3.2.6',
     'classifiers': [
         # How mature is this project? Common values are
         #   3 - Alpha


=====================================
tests/biomaj_tests.py
=====================================
@@ -190,8 +190,8 @@ class TestDownloadInterface(unittest.TestCase):
     # Test some garbage
     d = dict(stop_condition="stop_after_attempts")  # no param
     self.assertRaises(ValueError, downloader.set_options, d)
-    d = dict(stop_condition="1 & 1")  # not a stop_condition
-    self.assertRaises(ValueError, downloader.set_options, d)
+    #d = dict(stop_condition="1 & 1")  # not a stop_condition
+    #self.assertRaises(ValueError, downloader.set_options, d)
     d = dict(stop_condition="stop_after_attempts(5) & 1")  # not a stop_condition
     self.assertRaises(ValueError, downloader.set_options, d)
     # Test some garbage
@@ -199,8 +199,8 @@ class TestDownloadInterface(unittest.TestCase):
     self.assertRaises(ValueError, downloader.set_options, d)
     d = dict(wait_policy="I love python")  # not a wait_condition
     self.assertRaises(ValueError, downloader.set_options, d)
-    d = dict(wait_policy="wait_random(5) + 3")  # not a wait_condition
-    self.assertRaises(ValueError, downloader.set_options, d)
+    #d = dict(wait_policy="wait_random(5) + 3")  # not a wait_condition
+    #self.assertRaises(ValueError, downloader.set_options, d)
     # Test operators
     d = dict(stop_condition="stop_never | stop_after_attempt(5)",
              wait_policy="wait_none + wait_random(1, 2)")



View it on GitLab: https://salsa.debian.org/med-team/biomaj3-download/-/compare/2d9cd856e536694b7c588716accb97cd9bb1ed29...690c8e634be961e2343a725c28db265369d9a07a

-- 
View it on GitLab: https://salsa.debian.org/med-team/biomaj3-download/-/compare/2d9cd856e536694b7c588716accb97cd9bb1ed29...690c8e634be961e2343a725c28db265369d9a07a
You're receiving this email because of your account on salsa.debian.org.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/debian-med-commit/attachments/20220214/20206359/attachment-0001.htm>


More information about the debian-med-commit mailing list