canonical / canonical/matter-snap-testing

Make slot optional in SnapConnect

Open
#31 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Go
Stars
2
Forks
1
PR merge metrics
No merged PRs in 30d

Description

https://github.com/canonical/matter-snap-testing/blob/abae29ac5e865f0c5208350bdab63cecb3bdcc5a/utils/snap.go#L80

The slot side can be omitted for system slots.

Without this possibility, the caller needs to pass an empty string:
```go
utils.SnapConnect(t, "chip-tool:avahi-observe", "")
```
or explicitly set the slot:
```go
utils.SnapConnect(t, chipToolSnap+":avahi-observe", ":avahi-observe")
```

Contributor guide

No contributing guide indexed for this repository

Research direction

The relevant entry point is utils/snap.go around line 80; inspect SnapConnect's current argument handling and its callers. Make the slot side optional while preserving existing forms, then verify the examples in the issue no longer require an empty string.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
testing-qa
Issue type
Feature
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
58/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.