Skip to content

Commit

Permalink
updated code to reflect changes to api.json for many new commands
Browse files Browse the repository at this point in the history
updated luxos.pyz and health-checker.pyz using make.py to include new json
updated unit tests to reflect the new number of commands, 71
  • Loading branch information
cschwartz1234 committed Apr 4, 2024
1 parent 884a790 commit 185166f
Show file tree
Hide file tree
Showing 4 changed files with 48 additions and 5 deletions.
Binary file modified health-checker.pyz
Binary file not shown.
Binary file modified luxos.pyz
Binary file not shown.
50 changes: 46 additions & 4 deletions src/luxos/api.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,12 @@
"atmset": {
"logon_required": true
},
"autotunerget": {
"logon_required": false
},
"autotunerset": {
"logon_required": true
},
"check": {
"logon_required": false
},
Expand All @@ -35,12 +41,18 @@
"devs": {
"logon_required": false
},
"disableboard": {
"logon_required": true
},
"disablepool": {
"logon_required": false
},
"edevs": {
"logon_required": false
},
"enableboard": {
"logon_required": true
},
"enablepool": {
"logon_required": false
},
Expand Down Expand Up @@ -95,9 +107,18 @@
"logon": {
"logon_required": false
},
"logset": {
"logon_required": false
},
"netset": {
"logon_required": true
},
"poolopts": {
"logon_required": false
},
"pooloptsset": {
"logon_required": false
},
"pools": {
"logon_required": false
},
Expand All @@ -116,6 +137,9 @@
"profiles": {
"logon_required": false
},
"profilerestore": {
"logon_required": true
},
"profileset": {
"logon_required": true
},
Expand All @@ -128,12 +152,15 @@
"removegroup": {
"logon_required": false
},
"removepool": {
"logon_required": false
},
"resetminer": {
"logon_required": true
},
"resetconfig": {
"logon_required": true
},
"removepool": {
"logon_required": false
},
"session": {
"logon_required": false
},
Expand All @@ -155,6 +182,21 @@
"temps": {
"logon_required": false
},
"tempsensor": {
"logon_required": false
},
"tempsensorset": {
"logon_required": true
},
"tunerstatus": {
"logon_required": false
},
"tunerswitch": {
"logon_required": true
},
"uninstallluxos": {
"logon_required": true
},
"updaterun": {
"logon_required": true
},
Expand All @@ -170,4 +212,4 @@
"voltageset": {
"logon_required": true
}
}
}
3 changes: 2 additions & 1 deletion tests/test_luxos.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@ def test_import():
from luxos import api
assert (Path(api.__file__).parent / "api.json").exists()

assert len(api.COMMANDS) == 57
### number of API tests hardcoded here
assert len(api.COMMANDS) == 71


def test_logon_required():
Expand Down

0 comments on commit 185166f

Please sign in to comment.