Handle no compatibility requirements case

Author: MaxBQb

README.md

@@ -33,7 +33,7 @@ python -m github_release_downloader -u OwnerName -n RepoName -r ~1.1 -m .*\.exe
 ```
 
 ## Features
-1. Downloads only compatible releases
+1. Downloads compatible releases (or latest if no requirements set)
 2. Filters assets using regex
 3. Has optional download_callback
 4. CLI tool can be used in CI

src/github_release_downloader/core.py

@@ -43,7 +43,7 @@ def main():
     args = get_args()
     check_and_download_updates(
         GitHubRepo(args.user, args.repo_name, args.token),
-        SimpleSpec(args.require),
+        SimpleSpec(args.require) if args.require else None,
         assets_mask=re.compile(args.mask),
         current_version=Version(args.current_version) if args.current_version else None,
         downloads_dir=Path(args.output_dir)
@@ -52,7 +52,7 @@ def main():
 
 def check_and_download_updates(
     repo: GitHubRepo,
-    compatibility_spec: SimpleSpec,
+    compatibility_spec: SimpleSpec = None,
     current_version: Version = None,
     assets_mask=re.compile('.*'),
     downloads_dir=Path(),
@@ -66,14 +66,18 @@ def check_and_download_updates(
         if current_version is None:
             current_version = Version("0.0.0")
     AuthSession.init(repo)
-    logging.info(f"Compatibility requirement: '{compatibility_spec}'")
-
-    versions = list(sorted(compatibility_spec.filter(get_available_versions(repo)))[-10:])
-    if not versions:
-        logging.warning(f"No newer compatible versions available.")
+    if compatibility_spec is None:
+        logging.info(f"No compatibility requirements set")
+        download_version = get_latest_version(repo)
+    else:
+        logging.info(f"Compatibility requirement: '{compatibility_spec}'")
+        download_version = get_compatible_version(repo, compatibility_spec)
+
+    if download_version is None:
+        compatible = " compatible" if compatibility_spec is not None else ""
+        logging.warning(f"No newer{compatible} versions available.")
         return
-    logging.info(f"Available versions: {tuple(map(str, versions))}")
-    download_version = versions[-1]
+
     if is_already_installed(download_version, current_version, compatibility_spec):
         return
     tag_name = getattr(download_version, '_origin_tag_name', str(download_version))
@@ -86,6 +90,14 @@ def check_and_download_updates(
     cache.version = download_version
 
 
+def get_compatible_version(repo: GitHubRepo, compatibility_spec: SimpleSpec):
+    versions = sorted(compatibility_spec.filter(get_available_versions(repo)))[-10:]
+    if not versions:
+        return
+    logging.info(f"Available versions: {tuple(map(str, versions))}")
+    return versions[-1]
+
+
 def download_assets(
     assets: typing.Iterable[ReleaseAsset],
     out_dir=Path(),
@@ -147,6 +159,22 @@ def get_available_versions(repo: GitHubRepo, process_tag: Callable[[str], Versio
             break
 
 
+def get_latest_version(repo: GitHubRepo, process_tag: Callable[[str], Version] = None):
+    if process_tag is None:
+        process_tag = parse_tag
+    logging.info(f"Searching for latest release in 'https://github.com/{repo.user}/{repo.repo}/'...")
+    request_url = f"https://api.github.com/repos/{repo.user}/{repo.repo}/releases/latest"
+    data = json.loads(requests.get(request_url, headers=AuthSession.header).text)
+    if 'message' in data:
+        return
+    tag_name = data.get("tag_name")
+    if tag_name is None:
+        return
+    version = process_tag(tag_name)
+    version._origin_tag_name = tag_name
+    return version
+
+
 def parse_tag(tag_name: str):
     return Version(tag_name.lstrip("v").strip())
 

tests.py

@@ -1,3 +1,4 @@
+import logging
 import os
 import re
 import shutil
@@ -15,15 +16,20 @@ class DownloadRelease(unittest.TestCase):
     @classmethod
     def setUpClass(cls):
         cls.repo = GitHubRepo("MBQbUtils", "BulkStartStop", os.environ.get("GITHUB_TOKEN", ""))
-        cls.spec = SimpleSpec("~1")
+        cls.spec = SimpleSpec("~1.0")
         cls.out_dir = Path("test/out")
+        logging.basicConfig(
+            format="[%(asctime)s][%(levelname)s] %(message)s",
+            datefmt='%H:%M:%S',
+            level=logging.INFO
+        )
 
     def tearDown(self):
         shutil.rmtree(self.out_dir, True)
         for path in glob("repo-*-*.cache"):
             Path(path).unlink()
 
-    def test_run_and_stop(self):
+    def test_download_compatible_exe(self):
         check_and_download_updates(
             self.repo,
             self.spec,
@@ -35,6 +41,17 @@ def test_run_and_stop(self):
         file_actual = Path(files[0])
         self.assertTrue(file_actual.samefile(file_expected))
 
+    def test_download_latest_zip(self):
+        check_and_download_updates(
+            self.repo,
+            assets_mask=re.compile(".*\\.zip"),
+            downloads_dir=self.out_dir
+        )
+        files = glob(str(self.out_dir.joinpath("*.zip")))
+        file_expected = Path(self.out_dir).joinpath("BulkStartStop_portable.zip")
+        file_actual = Path(files[0])
+        self.assertTrue(file_actual.samefile(file_expected))
+
 
 if __name__ == '__main__':
     unittest.main()