Package libkvkontakte
Closed, ResolvedPublic

Description

Optional dep for Digikam

https://github.com/KDE/libkvkontakte

Stuck on tests. All of the create type tests, it does seem odd to me that a test would access a remote api like that..
QDEBUG : TestAlbums::initTestCase() Starting request QUrl("https://api.vk.com/method/photos.createAlbum?title=__album for unit testing of libkvkontakte %231&description=Description for album for unit testing of libkvkontakte %231&privacy=3&comment_privacy=3&access_token=832b99a9fdf7b3c4390e3e884b3dd93610ab4cb59d3effcc1c099eb7aec69c797dee5392fd2243c69719e")
QDEBUG : TestAlbums::initTestCase() Got data: "{\"error\":{\"error_code\":15,\"error_msg\":\"Access denied: no access to call this method\",\"request_params\":[{\"key\":\"oauth\",\"value\":\"1\"},{\"key\":\"method\",\"value\":\"photos.createAlbum\"},{\"key\":\"title\",\"value\":\"
album for unit testing of libkvkontakte #1\"},{\"key\":\"description\",\"value\":\"Description for __album for unit testing of libkvkontakte #1\"},{\"key\":\"privacy\",\"value\":\"3\"},{\"key\":\"comment_privacy\",\"value\":\"3\"}]}}"
QWARN : TestAlbums::initTestCase() An error of type 15 occurred: "Access denied: no access to call this method"

jriddell moved this task from Ready To Do to Review on the Neon board.Jul 25 2017, 11:57 AM

This has now been released. @jriddell you reviewed during release process, can you accept this as done?

stable build not happy
https://build.neon.kde.org/job/xenial_stable_kde-extras_libkvkontakte/
it's because there's no stable branch. our tooling should be clever enough to detect this and use the master branch in these cases but for a workaround remove the locale/ dir and merge into unstable and add it back there

release not working
https://build.neon.kde.org/job/xenial_release_kde-extras_libkvkontakte_src/1/console

releaes-lts not working
https://build.neon.kde.org/job/xenial_release-lts_kde-extras_libkvkontakte/

These issues have now been resolved. Need another review :)

acc test fails, we don't use it in neon so just remove it I guess

otherwise all good

disabled acc test, feel free to close this

jriddell closed this task as Resolved.Sep 5 2017, 12:49 PM