Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(153)

Side by Side Diff: components/copresence/rpc/rpc_handler_unittest.cc

Issue 461803003: Stop playing/recording when not needed. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/copresence/rpc/rpc_handler.h" 5 #include "components/copresence/rpc/rpc_handler.h"
6 6
7 #include <map> 7 #include <map>
8 #include <string> 8 #include <string>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 public: 55 public:
56 FakeDirectiveHandler() {} 56 FakeDirectiveHandler() {}
57 virtual ~FakeDirectiveHandler() {} 57 virtual ~FakeDirectiveHandler() {}
58 58
59 const std::vector<Directive>& added_directives() const { 59 const std::vector<Directive>& added_directives() const {
60 return added_directives_; 60 return added_directives_;
61 } 61 }
62 62
63 virtual void Initialize( 63 virtual void Initialize(
64 const AudioRecorder::DecodeSamplesCallback& decode_cb, 64 const AudioRecorder::DecodeSamplesCallback& decode_cb,
65 const AudioDirectiveList::EncodeTokenCallback& encode_cb) OVERRIDE {} 65 const AudioDirectiveHandler::EncodeTokenCallback& encode_cb) OVERRIDE {}
66 66
67 virtual void AddDirective(const Directive& directive) OVERRIDE { 67 virtual void AddDirective(const Directive& directive) OVERRIDE {
68 added_directives_.push_back(directive); 68 added_directives_.push_back(directive);
69 } 69 }
70 70
71 virtual void RemoveDirectives(const std::string& op_id) OVERRIDE { 71 virtual void RemoveDirectives(const std::string& op_id) OVERRIDE {
72 // TODO(ckehoe): Add a parallel implementation when prod has one. 72 // TODO(ckehoe): Add a parallel implementation when prod has one.
73 } 73 }
74 74
75 private: 75 private:
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
242 StatusCallback()); 242 StatusCallback());
243 EXPECT_EQ(RpcHandler::kReportRequestRpcName, rpc_name_); 243 EXPECT_EQ(RpcHandler::kReportRequestRpcName, rpc_name_);
244 ReportRequest* report = static_cast<ReportRequest*>(request_proto_.get()); 244 ReportRequest* report = static_cast<ReportRequest*>(request_proto_.get());
245 EXPECT_TRUE(report->header().has_framework_version()); 245 EXPECT_TRUE(report->header().has_framework_version());
246 EXPECT_EQ("CreateRequestHeader App ID", 246 EXPECT_EQ("CreateRequestHeader App ID",
247 report->header().client_version().client()); 247 report->header().client_version().client());
248 EXPECT_EQ("CreateRequestHeader Device ID", 248 EXPECT_EQ("CreateRequestHeader Device ID",
249 report->header().registered_device_id()); 249 report->header().registered_device_id());
250 } 250 }
251 251
252 TEST_F(RpcHandlerTest, ReportTokens) { 252 // TODO(ckehoe): Renable these after https://codereview.chromium.org/453203002/
253 std::vector<FullToken> test_tokens; 253 // lands.
254 test_tokens.push_back(FullToken("token 1", false)); 254 #define MAYBE_ReportTokens DISABLED_ReportTokens
255 test_tokens.push_back(FullToken("token 2", true)); 255
256 test_tokens.push_back(FullToken("token 3", false)); 256 TEST_F(RpcHandlerTest, MAYBE_ReportTokens) {
257 std::vector<AudioToken> test_tokens;
258 test_tokens.push_back(AudioToken("token 1", false));
259 test_tokens.push_back(AudioToken("token 2", true));
260 test_tokens.push_back(AudioToken("token 3", false));
257 AddInvalidToken("token 2"); 261 AddInvalidToken("token 2");
258 262
259 rpc_handler_.ReportTokens(test_tokens); 263 rpc_handler_.ReportTokens(test_tokens);
260 EXPECT_EQ(RpcHandler::kReportRequestRpcName, rpc_name_); 264 EXPECT_EQ(RpcHandler::kReportRequestRpcName, rpc_name_);
261 ReportRequest* report = static_cast<ReportRequest*>(request_proto_.get()); 265 ReportRequest* report = static_cast<ReportRequest*>(request_proto_.get());
262 google::protobuf::RepeatedPtrField<TokenObservation> tokens_sent = 266 google::protobuf::RepeatedPtrField<TokenObservation> tokens_sent =
263 report->update_signals_request().token_observation(); 267 report->update_signals_request().token_observation();
264 ASSERT_EQ(2, tokens_sent.size()); 268 ASSERT_EQ(2, tokens_sent.size());
265 EXPECT_EQ("token 1", tokens_sent.Get(0).token_id()); 269 EXPECT_EQ("token 1", tokens_sent.Get(0).token_id());
266 EXPECT_EQ("token 3", tokens_sent.Get(1).token_id()); 270 EXPECT_EQ("token 3", tokens_sent.Get(1).token_id());
(...skipping 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
321 messages_by_subscription_["Subscription 2"][1].payload()); 325 messages_by_subscription_["Subscription 2"][1].payload());
322 326
323 ASSERT_EQ(2U, directive_handler->added_directives().size()); 327 ASSERT_EQ(2U, directive_handler->added_directives().size());
324 EXPECT_EQ("Subscription 1", 328 EXPECT_EQ("Subscription 1",
325 directive_handler->added_directives()[0].subscription_id()); 329 directive_handler->added_directives()[0].subscription_id());
326 EXPECT_EQ("Subscription 2", 330 EXPECT_EQ("Subscription 2",
327 directive_handler->added_directives()[1].subscription_id()); 331 directive_handler->added_directives()[1].subscription_id());
328 } 332 }
329 333
330 } // namespace copresence 334 } // namespace copresence
OLDNEW
« components/copresence/rpc/rpc_handler.cc ('K') | « components/copresence/rpc/rpc_handler.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698