Rename addon_base_id to asset_base_id in the addon testing scripts, add few tweaks

Author: agajdosi

.github/workflows/webhook_test_addon.yml renamed to .github/workflows/webhook_process_addon.yml

@@ -1,10 +1,10 @@
-name: Test Addon
-run-name: Test addon | ${{ github.event.inputs.addon_base_id || github.event.client_payload.addon_base_id }} | ${{ github.event.inputs.verification_status || github.event.client_payload.verification_status }} | ${{ github.event.inputs.is_private || github.event.client_payload.is_private }}
+name: Process Addon
+run-name: Process Add-on | ${{ github.event.inputs.asset_base_id || github.event.client_payload.asset_base_id }} | ${{ github.event.inputs.verification_status || github.event.client_payload.verification_status }} | ${{ github.event.inputs.is_private || github.event.client_payload.is_private }}
 
 on:
   workflow_dispatch:
     inputs:
-      addon_base_id:
+      asset_base_id:
         description: 'Addon Base ID'
         required: true
       verification_status:
@@ -20,7 +20,7 @@ env:
   BLENDERKIT_API_KEY: '${{ secrets.BLENDERKIT_API_KEY }}'
   TEXTYBOT_API_KEY: '${{ secrets.TEXTYBOT_API_KEY }}' # for commenting
   BLENDER_PATH: /home/headless/blender/blender
-  ADDON_BASE_ID: ${{ github.event.inputs.addon_base_id || github.event.client_payload.addon_base_id }}
+  ASSET_BASE_ID: ${{ github.event.inputs.asset_base_id || github.event.client_payload.asset_base_id }}
 
 jobs:
   test_matrix:

test_addon.py

@@ -60,15 +60,15 @@ def blender_validate_extension():
 if __name__ == '__main__':
   BLENDER_PATH = os.environ.get('BLENDER_PATH','')
   API_KEY = os.environ.get('BLENDERKIT_API_KEY', '')
-  ADDON_BASE_ID = os.environ.get('ADDON_BASE_ID', '')
+  ASSET_BASE_ID = os.environ.get('ASSET_BASE_ID', '')
 
-  params = {'asset_base_id': ADDON_BASE_ID, 'asset_type': 'addon'}
+  params = {'asset_base_id': ASSET_BASE_ID, 'asset_type': 'addon'}
   addons = search.get_search_without_bullshit(params, api_key=API_KEY)
   if len(addons) == 0:
     raise Exception("Addon not found in the database")
 
   for i, asset in enumerate(addons): # One result is expected, but for transparency..
-    print(f"{i+1}. {asset['assetType']}: {asset['name']}")
+    print(f"{i+1}. {asset['assetType']}: {asset['name']} ({asset['url']})")
 
   # We just take 1st result
   test_ok, test_results = test_addon(addons[0], API_KEY, binary_path=BLENDER_PATH)

test_addon_report.py

@@ -56,7 +56,8 @@ def generate_comment(results: OrderedDict[str, dict]) -> str:
 
 api_nice.create_comment(
   comment=comment,
-  asset_base_id=environ.get('ADDON_BASE_ID', ''),
+  asset_base_id=environ.get('ASSET_BASE_ID', ''),
   api_key=environ.get('TEXTYBOT_API_KEY', environ.get('BLENDERKIT_API_KEY', '')), # prefer KEY for account of specialized commenting bot
   server_url=environ.get('BLENDERKIT_SERVER', '')
 )
+print(f"comment uploaded")