Mir

Merge lp:~alan-griffiths/mir/fix-1218436 into lp:mir

Proposed by Alan Griffiths
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 1408
Proposed branch: lp:~alan-griffiths/mir/fix-1218436
Merge into: lp:mir
Diff against target: 11 lines (+2/-0)
1 file modified
tests/acceptance-tests/test_client_authorization.cpp (+2/-0)
To merge this branch: bzr merge lp:~alan-griffiths/mir/fix-1218436
Reviewer Review Type Date Requested Status
Alberto Aguirre (community) Approve
Alexandros Frantzis (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+206947@code.launchpad.net

Commit message

tests: Add expectation to validate second SessionAuthorizer method is called wit correct parameter.

Description of the change

tests: Add expectation to validate second SessionAuthorizer method is called wit correct parameter.

This silences the GMOCK WARNING associated with ClientPidTestFixture.session_authorizer_receives_pid_of_connecting_clients

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alexandros Frantzis (afrantzis) wrote :

OK.

review: Approve
Revision history for this message
Alberto Aguirre (albaguirre) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/acceptance-tests/test_client_authorization.cpp'
2--- tests/acceptance-tests/test_client_authorization.cpp 2014-01-13 06:12:33 +0000
3+++ tests/acceptance-tests/test_client_authorization.cpp 2014-02-18 14:38:00 +0000
4@@ -116,6 +116,8 @@
5
6 EXPECT_CALL(mock_authorizer, connection_is_allowed(client_pid)).Times(1)
7 .WillOnce(Return(true));
8+ EXPECT_CALL(mock_authorizer, configure_display_is_allowed(client_pid)).Times(1)
9+ .WillOnce(Return(false));
10 connect_sync.try_signal_ready_for();
11 }
12

Subscribers

People subscribed via source and target branches