Bug 54873

Summary:	let account storage plugins mainly deal in GVariants
Product:	Telepathy	Reporter:	Simon McVittie <smcv>
Component:	mission-control	Assignee:	Simon McVittie <smcv>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium	CC:	jonny.lamb
Version:	unspecified	Keywords:	patch
Hardware:	Other
OS:	All
URL:	http://cgit.freedesktop.org/~smcv/telepathy-mission-control/log/?h=mock-account-store-54873
Whiteboard:	r+
i915 platform:		i915 features:
Bug Depends on:	54872
Bug Blocks:	29563, 54780

Description Simon McVittie 2012-09-13 15:12:31 UTC

When we first introduced pluggable account storage, we should have written a mock accounts store D-Bus service in the regression tests, so that the tests could q.expect() the store storing the right things. This branch rectifies that omission, then uses the new plugin to test new infrastructure for passing GVariants (rather than keyfile-syntax strings) to account plugins.

The last couple of commits demonstrate that account storage plugins can alter Parameters at will, but that altering non-Parameters basically doesn't work (Bug #54780). That will be the next in the review queue.

This is part 3 of my account storage refactoring megabranch.

Comment 1 Will Thompson 2012-11-07 16:22:17 UTC

looks fine!

Comment 2 Simon McVittie 2013-02-13 14:37:33 UTC

Fixed in git for 5.15.0

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.