Skip to content

Cap def download - #3160

Open
varzac wants to merge 2 commits into
mainfrom
cap-def-download
Open

Cap def download#3160
varzac wants to merge 2 commits into
mainfrom
cap-def-download

Conversation

@varzac

@varzac varzac commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This adds support for downloading current capability definitions for use in running tests. This ensures that tests match the current hub environment.

@github-actions

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

All capability definitions downloaded successfully.

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

Test Results

   73 files    540 suites   0s ⏱️
3 241 tests 3 241 ✅ 0 💤 0 ❌
5 233 runs  5 233 ✅ 0 💤 0 ❌

Results for commit 9f75ddc.

♻️ This comment has been updated with latest results.

@github-actions

Copy link
Copy Markdown

zigbee-bed_coverage.xml

File Coverage
All files 98%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-bed/src/shus-mattress/can_handle.lua 83%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-bed/src/init.lua 92%
/home/runner/work/SmartThingsEdgeDrivers/SmartThingsEdgeDrivers/drivers/SmartThings/zigbee-bed/src/lazy_load_subdriver.lua 57%

Minimum allowed coverage is 90%

Generated by 🐒 cobertura-action against 9f75ddc

env:
CAPABILITY_PAT: ${{ secrets.CAPABILITY_DEFINITIONS_PAT }}
- name: Report capability download failures
if: github.event_name == 'pull_request' && hashFiles('capability_failures_comment.md') != '' && always()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

github.event_name == 'pull_request' is unnecessary as this workflow will only fire on PRs

import sys
import time
from pathlib import Path

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
from typing import List, Tuple, Set, Dict

Then all the subscripted type hints changed with it too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants