Skip to content

Commit 20181d0

Browse files
committed
fixup! tests: add desktop-file and common-id checks to desktop-file-ids test
Signed-off-by: Oliver Calder <oliver.calder@canonical.com>
1 parent 37cd765 commit 20181d0

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

tests/main/desktop-file-ids/task.yaml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ prepare: |
1717
cat > "$TESTSLIB"/snaps/test-snapd-desktop-file-ids/meta/gui/'bad. ,?**[]{}^\$#.desktop' << EOF
1818
[Desktop Entry]
1919
Name=test-snapd-desktop-file-ids.cmd
20-
Comment=A desktop file for test-snapd-desktop.cmd
20+
Comment=A desktop file for test-snapd-desktop-file-ids.cmd
2121
Exec=test-snapd-desktop-file-ids.cmd
2222
Terminal=true
2323
Type=Application
@@ -32,7 +32,7 @@ restore: |
3232
tests.session -u test restore
3333
3434
execute: |
35-
files="$DIR/org.example.desktop $DIR/org.example.Foo.desktop $DIR/org.example.Fallback.desktop $DIR/test-snapd-desktop-file-ids_foo.desktop $DIR/test-snapd-desktop-file-ids_bar.desktop $DIR/test-snapd-desktop-file-ids_org.example.Bar.desktop $DIR/test-snapd-desktop-file-ids_bad._____________.desktop"
35+
files="$DIR/org.example.desktop $DIR/org.example.Foo.desktop $DIR/org.example.Fallback.desktop $DIR/test-snapd-desktop-file-ids_arbitrary.desktop $DIR/test-snapd-desktop-file-ids_foo.desktop $DIR/test-snapd-desktop-file-ids_bar.desktop $DIR/test-snapd-desktop-file-ids_org.example.Bar.desktop $DIR/test-snapd-desktop-file-ids_bad._____________.desktop"
3636
3737
# Connect desktop-legacy interface to test generated desktop allow/deny rules
3838
# only allows access to the snap's desktop files
@@ -53,12 +53,13 @@ execute: |
5353
5454
echo "Just because a desktop file ID is declared does not mean it is created"
5555
test ! -f "$DIR/org.example.NonexistentFile.desktop"
56+
test ! -f "$DIR/org.example.MyCommonID.desktop"
5657
5758
echo "Check that common ID and desktop file ID is correct"
5859
RESULT="$(snap debug api /v2/snaps/test-snapd-desktop-file-ids)"
5960
echo "$RESULT" | MATCH '"status-code": 200'
6061
APPS="$(echo "$RESULT" | gojq .result.apps)"
61-
# Should have common-id but no file id
62+
# Should have both common-id and desktop-file
6263
echo "$APPS" | gojq '.[] | select(.name == "cmd")."common-id"' | MATCH "org.example.Foo"
6364
echo "$APPS" | gojq '.[] | select(.name == "cmd")."desktop-file"' | MATCH "org.example.Foo.desktop"
6465

0 commit comments

Comments
 (0)