tests: add gsettings interface spread test #1640

Closed
wants to merge 7 commits into
from

Conversation

Projects
None yet
4 participants
Contributor

fgimenez commented Aug 5, 2016

No description provided.

+ snap interfaces | grep -Pzq "$CONNECTED_PATTERN"
+
+ echo "Then the snap is able to write the gsettings user configuration"
+ sudo -i -u test /bin/sh -c 'gsettings-consumer.write /home/test/.config/dconf/user gsettings_content'
@niemeyer

niemeyer Aug 6, 2016

Contributor

This doesn't feel like a very realistic gsettings client. It's a lot of logic just to ensure we have these two path lines on the apparmor profile, and would blindly continue passing even if everything else is broken and gsettings doesn't actually work.

@fgimenez

fgimenez Aug 9, 2016

Contributor

@niemeyer indeed very good catch, I've prepared a test snap with a gsettings client, will wait for it to be uploaded to the store and push the changes to use it here.

Thanks,

@niemeyer niemeyer added the Reviewed label Aug 6, 2016

fgimenez added some commits Aug 8, 2016

Contributor

fgimenez commented Aug 9, 2016

@niemeyer done, the test is throwing an error No schemas installed but no apparmor related problems in dmesg, running with sudo strace -f /snap/bin/test-snapd-gsettings-consumer.get:

[pid 14815] open("/usr/local/share/glib-2.0/schemas/gschemas.compiled", O_RDONLY) = -1 ENOENT (No such file or directory)
[pid 14815] futex(0x7f3691d99428, FUTEX_WAKE_PRIVATE, 2147483647) = 0
[pid 14815] write(2, "No schemas installed\n", 21No schemas installed

@jdstrand could you please have a look?

Thanks!

@niemeyer niemeyer added the Decaying label Aug 26, 2016

Contributor

zyga commented Sep 12, 2016

Can we please merge master into this to see what is the current status?

Contributor

fgimenez commented Sep 12, 2016

@zyga done thx for having a look :)

Contributor

niemeyer commented Sep 20, 2016

Tests still fail, and related to the new test. This has been up for more than a month without it working. @fgimenez, please reopen the PR when there's something we can merge.

@niemeyer niemeyer closed this Sep 20, 2016

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