From d99821753cba1e46f3d9550d9e75e7393b5cf98d Mon Sep 17 00:00:00 2001
From: cav71 <antonio.cavallo@luxor.tech>
Date: Mon, 14 Oct 2024 14:51:13 -0400
Subject: [PATCH] minor cleaups, switch to package namespace, reduce
 dependencies (#44)

* reduce dependencies to the minimum
* report has information in the short version (7 chars)
* Update README.md
* adds the new updatecheck command to the api.json
* adds 3.13 support
* update test
---
 .github/workflows/pull-python-luxos.yml | 2 +-
 .github/workflows/push-main.yml         | 2 +-
 Makefile                                | 2 +-
 pyproject.toml                          | 1 +
 src/luxos/api.json                      | 3 +++
 tests/test_luxos.py                     | 2 +-
 6 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/pull-python-luxos.yml b/.github/workflows/pull-python-luxos.yml
index a5d9107..a6daf75 100644
--- a/.github/workflows/pull-python-luxos.yml
+++ b/.github/workflows/pull-python-luxos.yml
@@ -15,7 +15,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        python-version: ["3.9", "3.10", "3.11", "3.12"]
+        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
     runs-on: ${{ matrix.os }}
 
     steps:
diff --git a/.github/workflows/push-main.yml b/.github/workflows/push-main.yml
index f9c9fa0..915f9ba 100644
--- a/.github/workflows/push-main.yml
+++ b/.github/workflows/push-main.yml
@@ -21,7 +21,7 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, macos-latest, windows-latest]
-        python-version: ["3.9", "3.10", "3.11", "3.12"]
+        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
     runs-on: ${{ matrix.os }}
 
     steps:
diff --git a/Makefile b/Makefile
index f5568d4..a05008e 100644
--- a/Makefile
+++ b/Makefile
@@ -32,7 +32,7 @@ lint:  ## Runs the linter (mypy) and report errors.
 
 
 .PHONY: test
-test: ## Run test suite
+test: ## Run test suite (make test MINER=IP.ADDRESS will run against a miner)
 	@pytest -vvs \
         --cov-report=html:build/coverage --cov-report=xml:build/coverage.xml \
         --junitxml=build/junit.xml --html=build/junit.html --self-contained-html \
diff --git a/pyproject.toml b/pyproject.toml
index 95930a0..2d997b2 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -24,6 +24,7 @@ classifiers = [
   "Programming Language :: Python :: 3.10",
   "Programming Language :: Python :: 3.11",
   "Programming Language :: Python :: 3.12",
+  "Programming Language :: Python :: 3.13",
 ]
 
 dependencies = [
diff --git a/src/luxos/api.json b/src/luxos/api.json
index d0696b1..79ed5be 100644
--- a/src/luxos/api.json
+++ b/src/luxos/api.json
@@ -200,6 +200,9 @@
   "uninstallluxos": {
     "logon_required": true
   },
+  "updatecheck" : {
+    "logon_required": false
+  },
   "updaterun": {
     "logon_required": true
   },
diff --git a/tests/test_luxos.py b/tests/test_luxos.py
index d08af7d..8ad9dc2 100644
--- a/tests/test_luxos.py
+++ b/tests/test_luxos.py
@@ -9,7 +9,7 @@ def test_import():
     assert (Path(api.__file__).parent / "api.json").exists()
 
     ### number of API tests hardcoded here
-    assert len(api.COMMANDS) == 72
+    assert len(api.COMMANDS) == 73
 
 
 def test_logon_required():