qtorganizer5-eds

Merge lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836 into lp://qastaging/qtorganizer5-eds

  1. fix-1347836
  2. Merge into trunk
Proposed by Renato Araujo Oliveira Filho
Status: Superseded
Proposed branch: lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836
Merge into: lp://qastaging/qtorganizer5-eds
Diff against target: 705 lines (+320/-59)
13 files modified
CMakeLists.txt (+1/-0)
organizer/qorganizer-eds-engine.cpp (+56/-16)
organizer/qorganizer-eds-engine.h (+3/-1)
organizer/qorganizer-eds-requestdata.cpp (+16/-4)
organizer/qorganizer-eds-requestdata.h (+1/-1)
organizer/qorganizer-eds-source-registry.cpp (+16/-2)
organizer/qorganizer-eds-source-registry.h (+4/-1)
tests/unittest/CMakeLists.txt (+0/-1)
tests/unittest/collections-test.cpp (+15/-2)
tests/unittest/eds-base-test.cpp (+61/-0)
tests/unittest/eds-base-test.h (+3/-0)
tests/unittest/event-test.cpp (+102/-5)
tests/unittest/run-eds-test.sh (+42/-26)
To merge this branch: bzr merge lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu Phablet Team Pending
Review via email: mp+239038@code.qastaging.launchpad.net

This proposal has been superseded by a proposal from 2015-04-09.

Commit message

Set extra metadata in collection (read-only property).

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:66
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/184/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-amd64-ci/42/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-armhf-ci/42/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-i386-ci/42/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/184/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836 updated
67. By Renato Araujo Oliveira Filho

Fixed wrong function call.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:67
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/186/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-amd64-ci/44
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-armhf-ci/44/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-i386-ci/44/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/186/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:67
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/187/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-amd64-ci/45/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-armhf-ci/45
        deb: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-armhf-ci/45/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-i386-ci/45

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/187/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Kunal Parmar (pkunal-parmar) wrote :

I am not getting readonly true for birth day calendar

qml: Personal - collectin-readonly : false
qml: Birthdays & Anniversaries - collectin-readonly : false

Where creating event on Birthdays calendar fails as its readonly

ail to create items: Cannot create calendar object: Permission denied

lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836 updated
68. By Renato Araujo Oliveira Filho

Update read-only collection property after connect with client.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:68
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/188/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-amd64-ci/46/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-armhf-ci/46
        deb: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-armhf-ci/46/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-utopic-i386-ci/46

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/188/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836 updated
69. By Renato Araujo Oliveira Filho

Trunk merged.

70. By Renato Araujo Oliveira Filho

Update readonly conllections based on client information.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:70
http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-ci/204/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-amd64-ci/11/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-armhf-ci/11/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/qtorganizer5-eds-vivid-i386-ci/11/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/qtorganizer5-eds-ci/204/rebuild

review: Needs Fixing (continuous-integration)
lp://qastaging/~renatofilho/qtorganizer5-eds/fix-1347836 updated
71. By Renato Araujo Oliveira Filho

Merged: lp:~renatofilho/qtorganizer5-eds/fix-1440878

72. By Renato Araujo Oliveira Filho

Update collection test.

Instead of compare all the colection data to check if it exists, I am using only the collection id in case of the other properties changed during the process.

73. By Renato Araujo Oliveira Filho

Fix collection test.

Wait for collection to be writable before try to remove id.

74. By Renato Araujo Oliveira Filho

Do not try to compare secs on unit test.

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches