henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 1 | /* |
kjellander | b24317b | 2016-02-10 15:54:43 | [diff] [blame] | 2 | * Copyright 2012 The WebRTC project authors. All Rights Reserved. |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 3 | * |
kjellander | b24317b | 2016-02-10 15:54:43 | [diff] [blame] | 4 | * Use of this source code is governed by a BSD-style license |
| 5 | * that can be found in the LICENSE file in the root of the source |
| 6 | * tree. An additional intellectual property rights grant can be found |
| 7 | * in the file PATENTS. All contributing project authors may |
| 8 | * be found in the AUTHORS file in the root of the source tree. |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 9 | */ |
| 10 | |
Steve Anton | 10542f2 | 2019-01-11 17:11:00 | [diff] [blame] | 11 | #include "pc/dtmf_sender.h" |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 12 | |
Yves Gerey | 3e70781 | 2018-11-28 15:47:49 | [diff] [blame] | 13 | #include <stddef.h> |
Jonas Olsson | a4d8737 | 2019-07-05 17:08:33 | [diff] [blame] | 14 | |
kwiberg | d1fe281 | 2016-04-27 13:47:29 | [diff] [blame] | 15 | #include <memory> |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 16 | #include <string> |
| 17 | #include <vector> |
| 18 | |
Steve Anton | 10542f2 | 2019-01-11 17:11:00 | [diff] [blame] | 19 | #include "rtc_base/fake_clock.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 04:47:31 | [diff] [blame] | 20 | #include "rtc_base/gunit.h" |
Steve Anton | 10542f2 | 2019-01-11 17:11:00 | [diff] [blame] | 21 | #include "rtc_base/time_utils.h" |
Yves Gerey | 3e70781 | 2018-11-28 15:47:49 | [diff] [blame] | 22 | #include "test/gtest.h" |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 23 | |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 24 | using webrtc::DtmfProviderInterface; |
| 25 | using webrtc::DtmfSender; |
| 26 | using webrtc::DtmfSenderObserverInterface; |
| 27 | |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 28 | // TODO(deadbeef): Even though this test now uses a fake clock, it has a |
| 29 | // generous 3-second timeout for every test case. The timeout could be tuned |
| 30 | // to each test based on the tones sent, instead. |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 31 | static const int kMaxWaitMs = 3000; |
| 32 | |
Magnus Jedvert | fc95084 | 2015-10-12 14:10:43 | [diff] [blame] | 33 | class FakeDtmfObserver : public DtmfSenderObserverInterface { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 34 | public: |
| 35 | FakeDtmfObserver() : completed_(false) {} |
| 36 | |
| 37 | // Implements DtmfSenderObserverInterface. |
kjellander@webrtc.org | 14665ff | 2015-03-04 12:58:35 | [diff] [blame] | 38 | void OnToneChange(const std::string& tone) override { |
Harald Alvestrand | d7b79af | 2018-09-06 07:33:56 | [diff] [blame] | 39 | tones_from_single_argument_callback_.push_back(tone); |
| 40 | if (tone.empty()) { |
| 41 | completed_ = true; |
| 42 | } |
| 43 | } |
| 44 | void OnToneChange(const std::string& tone, |
| 45 | const std::string& tone_buffer) override { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 46 | tones_.push_back(tone); |
Harald Alvestrand | d7b79af | 2018-09-06 07:33:56 | [diff] [blame] | 47 | tones_remaining_ = tone_buffer; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 48 | if (tone.empty()) { |
| 49 | completed_ = true; |
| 50 | } |
| 51 | } |
| 52 | |
| 53 | // getters |
Yves Gerey | 665174f | 2018-06-19 13:03:05 | [diff] [blame] | 54 | const std::vector<std::string>& tones() const { return tones_; } |
Harald Alvestrand | d7b79af | 2018-09-06 07:33:56 | [diff] [blame] | 55 | const std::vector<std::string>& tones_from_single_argument_callback() const { |
| 56 | return tones_from_single_argument_callback_; |
| 57 | } |
| 58 | const std::string tones_remaining() { return tones_remaining_; } |
Yves Gerey | 665174f | 2018-06-19 13:03:05 | [diff] [blame] | 59 | bool completed() const { return completed_; } |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 60 | |
| 61 | private: |
| 62 | std::vector<std::string> tones_; |
Harald Alvestrand | d7b79af | 2018-09-06 07:33:56 | [diff] [blame] | 63 | std::vector<std::string> tones_from_single_argument_callback_; |
| 64 | std::string tones_remaining_; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 65 | bool completed_; |
| 66 | }; |
| 67 | |
| 68 | class FakeDtmfProvider : public DtmfProviderInterface { |
| 69 | public: |
| 70 | struct DtmfInfo { |
| 71 | DtmfInfo(int code, int duration, int gap) |
Yves Gerey | 665174f | 2018-06-19 13:03:05 | [diff] [blame] | 72 | : code(code), duration(duration), gap(gap) {} |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 73 | int code; |
| 74 | int duration; |
| 75 | int gap; |
| 76 | }; |
| 77 | |
| 78 | FakeDtmfProvider() : last_insert_dtmf_call_(0) {} |
| 79 | |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 80 | // Implements DtmfProviderInterface. |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 81 | bool CanInsertDtmf() override { return can_insert_; } |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 82 | |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 83 | bool InsertDtmf(int code, int duration) override { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 84 | int gap = 0; |
buildbot@webrtc.org | d4e598d | 2014-07-29 17:36:52 | [diff] [blame] | 85 | // TODO(ronghuawu): Make the timer (basically the rtc::TimeNanos) |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 86 | // mockable and use a fake timer in the unit tests. |
| 87 | if (last_insert_dtmf_call_ > 0) { |
Honghai Zhang | 82d7862 | 2016-05-06 18:29:15 | [diff] [blame] | 88 | gap = static_cast<int>(rtc::TimeMillis() - last_insert_dtmf_call_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 89 | } |
Honghai Zhang | 82d7862 | 2016-05-06 18:29:15 | [diff] [blame] | 90 | last_insert_dtmf_call_ = rtc::TimeMillis(); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 91 | |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 92 | dtmf_info_queue_.push_back(DtmfInfo(code, duration, gap)); |
| 93 | return true; |
| 94 | } |
| 95 | |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 96 | // getter and setter |
| 97 | const std::vector<DtmfInfo>& dtmf_info_queue() const { |
| 98 | return dtmf_info_queue_; |
| 99 | } |
| 100 | |
| 101 | // helper functions |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 102 | void SetCanInsertDtmf(bool can_insert) { can_insert_ = can_insert; } |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 103 | |
| 104 | private: |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 105 | bool can_insert_ = false; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 106 | std::vector<DtmfInfo> dtmf_info_queue_; |
Peter Boström | 0c4e06b | 2015-10-07 10:23:21 | [diff] [blame] | 107 | int64_t last_insert_dtmf_call_; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 108 | }; |
| 109 | |
Mirko Bonadei | 6a489f2 | 2019-04-09 13:11:12 | [diff] [blame] | 110 | class DtmfSenderTest : public ::testing::Test { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 111 | protected: |
| 112 | DtmfSenderTest() |
Tomas Gunnarsson | e249d19 | 2021-04-26 09:46:54 | [diff] [blame] | 113 | : observer_(new FakeDtmfObserver()), provider_(new FakeDtmfProvider()) { |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 114 | provider_->SetCanInsertDtmf(true); |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 115 | dtmf_ = DtmfSender::Create(rtc::Thread::Current(), provider_.get()); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 116 | dtmf_->RegisterObserver(observer_.get()); |
| 117 | } |
| 118 | |
| 119 | ~DtmfSenderTest() { |
| 120 | if (dtmf_.get()) { |
| 121 | dtmf_->UnregisterObserver(); |
| 122 | } |
| 123 | } |
| 124 | |
Artem Titov | 880fa81 | 2021-07-30 20:30:23 | [diff] [blame] | 125 | // Constructs a list of DtmfInfo from `tones`, `duration` and |
| 126 | // `inter_tone_gap`. |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 127 | void GetDtmfInfoFromString( |
| 128 | const std::string& tones, |
| 129 | int duration, |
| 130 | int inter_tone_gap, |
| 131 | std::vector<FakeDtmfProvider::DtmfInfo>* dtmfs, |
| 132 | int comma_delay = webrtc::DtmfSender::kDtmfDefaultCommaDelayMs) { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 133 | // Init extra_delay as -inter_tone_gap - duration to ensure the first |
| 134 | // DtmfInfo's gap field will be 0. |
| 135 | int extra_delay = -1 * (inter_tone_gap + duration); |
| 136 | |
| 137 | std::string::const_iterator it = tones.begin(); |
| 138 | for (; it != tones.end(); ++it) { |
| 139 | char tone = *it; |
| 140 | int code = 0; |
| 141 | webrtc::GetDtmfCode(tone, &code); |
| 142 | if (tone == ',') { |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 143 | extra_delay = comma_delay; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 144 | } else { |
Yves Gerey | 665174f | 2018-06-19 13:03:05 | [diff] [blame] | 145 | dtmfs->push_back(FakeDtmfProvider::DtmfInfo( |
| 146 | code, duration, duration + inter_tone_gap + extra_delay)); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 147 | extra_delay = 0; |
| 148 | } |
| 149 | } |
| 150 | } |
| 151 | |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 152 | void VerifyExpectedState(const std::string& tones, |
Yves Gerey | 665174f | 2018-06-19 13:03:05 | [diff] [blame] | 153 | int duration, |
| 154 | int inter_tone_gap) { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 155 | EXPECT_EQ(tones, dtmf_->tones()); |
| 156 | EXPECT_EQ(duration, dtmf_->duration()); |
| 157 | EXPECT_EQ(inter_tone_gap, dtmf_->inter_tone_gap()); |
| 158 | } |
| 159 | |
| 160 | // Verify the provider got all the expected calls. |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 161 | void VerifyOnProvider( |
| 162 | const std::string& tones, |
| 163 | int duration, |
| 164 | int inter_tone_gap, |
| 165 | int comma_delay = webrtc::DtmfSender::kDtmfDefaultCommaDelayMs) { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 166 | std::vector<FakeDtmfProvider::DtmfInfo> dtmf_queue_ref; |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 167 | GetDtmfInfoFromString(tones, duration, inter_tone_gap, &dtmf_queue_ref, |
| 168 | comma_delay); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 169 | VerifyOnProvider(dtmf_queue_ref); |
| 170 | } |
| 171 | |
| 172 | void VerifyOnProvider( |
| 173 | const std::vector<FakeDtmfProvider::DtmfInfo>& dtmf_queue_ref) { |
| 174 | const std::vector<FakeDtmfProvider::DtmfInfo>& dtmf_queue = |
| 175 | provider_->dtmf_info_queue(); |
| 176 | ASSERT_EQ(dtmf_queue_ref.size(), dtmf_queue.size()); |
| 177 | std::vector<FakeDtmfProvider::DtmfInfo>::const_iterator it_ref = |
| 178 | dtmf_queue_ref.begin(); |
| 179 | std::vector<FakeDtmfProvider::DtmfInfo>::const_iterator it = |
| 180 | dtmf_queue.begin(); |
| 181 | while (it_ref != dtmf_queue_ref.end() && it != dtmf_queue.end()) { |
| 182 | EXPECT_EQ(it_ref->code, it->code); |
| 183 | EXPECT_EQ(it_ref->duration, it->duration); |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 184 | // Allow ~10ms error (can be small since we're using a fake clock). |
| 185 | EXPECT_GE(it_ref->gap, it->gap - 10); |
| 186 | EXPECT_LE(it_ref->gap, it->gap + 10); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 187 | ++it_ref; |
| 188 | ++it; |
| 189 | } |
| 190 | } |
| 191 | |
| 192 | // Verify the observer got all the expected callbacks. |
| 193 | void VerifyOnObserver(const std::string& tones_ref) { |
| 194 | const std::vector<std::string>& tones = observer_->tones(); |
| 195 | // The observer will get an empty string at the end. |
| 196 | EXPECT_EQ(tones_ref.size() + 1, tones.size()); |
Harald Alvestrand | d7b79af | 2018-09-06 07:33:56 | [diff] [blame] | 197 | EXPECT_EQ(observer_->tones(), |
| 198 | observer_->tones_from_single_argument_callback()); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 199 | EXPECT_TRUE(tones.back().empty()); |
Harald Alvestrand | d7b79af | 2018-09-06 07:33:56 | [diff] [blame] | 200 | EXPECT_TRUE(observer_->tones_remaining().empty()); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 201 | std::string::const_iterator it_ref = tones_ref.begin(); |
| 202 | std::vector<std::string>::const_iterator it = tones.begin(); |
| 203 | while (it_ref != tones_ref.end() && it != tones.end()) { |
| 204 | EXPECT_EQ(*it_ref, it->at(0)); |
| 205 | ++it_ref; |
| 206 | ++it; |
| 207 | } |
| 208 | } |
| 209 | |
Niels Möller | 83830f3 | 2022-05-20 07:12:57 | [diff] [blame] | 210 | rtc::AutoThread main_thread_; |
kwiberg | d1fe281 | 2016-04-27 13:47:29 | [diff] [blame] | 211 | std::unique_ptr<FakeDtmfObserver> observer_; |
| 212 | std::unique_ptr<FakeDtmfProvider> provider_; |
buildbot@webrtc.org | d4e598d | 2014-07-29 17:36:52 | [diff] [blame] | 213 | rtc::scoped_refptr<DtmfSender> dtmf_; |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 214 | rtc::ScopedFakeClock fake_clock_; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 215 | }; |
| 216 | |
| 217 | TEST_F(DtmfSenderTest, CanInsertDtmf) { |
| 218 | EXPECT_TRUE(dtmf_->CanInsertDtmf()); |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 219 | provider_->SetCanInsertDtmf(false); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 220 | EXPECT_FALSE(dtmf_->CanInsertDtmf()); |
| 221 | } |
| 222 | |
| 223 | TEST_F(DtmfSenderTest, InsertDtmf) { |
| 224 | std::string tones = "@1%a&*$"; |
| 225 | int duration = 100; |
| 226 | int inter_tone_gap = 50; |
| 227 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 228 | EXPECT_TRUE_SIMULATED_WAIT(observer_->completed(), kMaxWaitMs, fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 229 | |
| 230 | // The unrecognized characters should be ignored. |
| 231 | std::string known_tones = "1a*"; |
| 232 | VerifyOnProvider(known_tones, duration, inter_tone_gap); |
| 233 | VerifyOnObserver(known_tones); |
| 234 | } |
| 235 | |
| 236 | TEST_F(DtmfSenderTest, InsertDtmfTwice) { |
| 237 | std::string tones1 = "12"; |
| 238 | std::string tones2 = "ab"; |
| 239 | int duration = 100; |
| 240 | int inter_tone_gap = 50; |
| 241 | EXPECT_TRUE(dtmf_->InsertDtmf(tones1, duration, inter_tone_gap)); |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 242 | VerifyExpectedState(tones1, duration, inter_tone_gap); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 243 | // Wait until the first tone got sent. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 244 | EXPECT_TRUE_SIMULATED_WAIT(observer_->tones().size() == 1, kMaxWaitMs, |
| 245 | fake_clock_); |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 246 | VerifyExpectedState("2", duration, inter_tone_gap); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 247 | // Insert with another tone buffer. |
| 248 | EXPECT_TRUE(dtmf_->InsertDtmf(tones2, duration, inter_tone_gap)); |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 249 | VerifyExpectedState(tones2, duration, inter_tone_gap); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 250 | // Wait until it's completed. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 251 | EXPECT_TRUE_SIMULATED_WAIT(observer_->completed(), kMaxWaitMs, fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 252 | |
| 253 | std::vector<FakeDtmfProvider::DtmfInfo> dtmf_queue_ref; |
| 254 | GetDtmfInfoFromString("1", duration, inter_tone_gap, &dtmf_queue_ref); |
| 255 | GetDtmfInfoFromString("ab", duration, inter_tone_gap, &dtmf_queue_ref); |
| 256 | VerifyOnProvider(dtmf_queue_ref); |
| 257 | VerifyOnObserver("1ab"); |
| 258 | } |
| 259 | |
| 260 | TEST_F(DtmfSenderTest, InsertDtmfWhileProviderIsDeleted) { |
| 261 | std::string tones = "@1%a&*$"; |
| 262 | int duration = 100; |
| 263 | int inter_tone_gap = 50; |
| 264 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
| 265 | // Wait until the first tone got sent. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 266 | EXPECT_TRUE_SIMULATED_WAIT(observer_->tones().size() == 1, kMaxWaitMs, |
| 267 | fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 268 | // Delete provider. |
Fredrik Solenberg | da2afbd | 2022-08-03 10:07:51 | [diff] [blame] | 269 | dtmf_->OnDtmfProviderDestroyed(); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 270 | provider_.reset(); |
| 271 | // The queue should be discontinued so no more tone callbacks. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 272 | SIMULATED_WAIT(false, 200, fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 273 | EXPECT_EQ(1U, observer_->tones().size()); |
| 274 | } |
| 275 | |
| 276 | TEST_F(DtmfSenderTest, InsertDtmfWhileSenderIsDeleted) { |
| 277 | std::string tones = "@1%a&*$"; |
| 278 | int duration = 100; |
| 279 | int inter_tone_gap = 50; |
| 280 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
| 281 | // Wait until the first tone got sent. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 282 | EXPECT_TRUE_SIMULATED_WAIT(observer_->tones().size() == 1, kMaxWaitMs, |
| 283 | fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 284 | // Delete the sender. |
| 285 | dtmf_ = NULL; |
| 286 | // The queue should be discontinued so no more tone callbacks. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 287 | SIMULATED_WAIT(false, 200, fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 288 | EXPECT_EQ(1U, observer_->tones().size()); |
| 289 | } |
| 290 | |
| 291 | TEST_F(DtmfSenderTest, InsertEmptyTonesToCancelPreviousTask) { |
| 292 | std::string tones1 = "12"; |
| 293 | std::string tones2 = ""; |
| 294 | int duration = 100; |
| 295 | int inter_tone_gap = 50; |
| 296 | EXPECT_TRUE(dtmf_->InsertDtmf(tones1, duration, inter_tone_gap)); |
| 297 | // Wait until the first tone got sent. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 298 | EXPECT_TRUE_SIMULATED_WAIT(observer_->tones().size() == 1, kMaxWaitMs, |
| 299 | fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 300 | // Insert with another tone buffer. |
| 301 | EXPECT_TRUE(dtmf_->InsertDtmf(tones2, duration, inter_tone_gap)); |
| 302 | // Wait until it's completed. |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 303 | EXPECT_TRUE_SIMULATED_WAIT(observer_->completed(), kMaxWaitMs, fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 304 | |
| 305 | std::vector<FakeDtmfProvider::DtmfInfo> dtmf_queue_ref; |
| 306 | GetDtmfInfoFromString("1", duration, inter_tone_gap, &dtmf_queue_ref); |
| 307 | VerifyOnProvider(dtmf_queue_ref); |
| 308 | VerifyOnObserver("1"); |
| 309 | } |
| 310 | |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 311 | TEST_F(DtmfSenderTest, InsertDtmfWithDefaultCommaDelay) { |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 312 | std::string tones = "3,4"; |
| 313 | int duration = 100; |
| 314 | int inter_tone_gap = 50; |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 315 | int default_comma_delay = webrtc::DtmfSender::kDtmfDefaultCommaDelayMs; |
| 316 | EXPECT_EQ(dtmf_->comma_delay(), default_comma_delay); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 317 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
deadbeef | e7fc7d5 | 2016-10-28 20:53:08 | [diff] [blame] | 318 | EXPECT_TRUE_SIMULATED_WAIT(observer_->completed(), kMaxWaitMs, fake_clock_); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 319 | |
| 320 | VerifyOnProvider(tones, duration, inter_tone_gap); |
| 321 | VerifyOnObserver(tones); |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 322 | EXPECT_EQ(dtmf_->comma_delay(), default_comma_delay); |
| 323 | } |
| 324 | |
| 325 | TEST_F(DtmfSenderTest, InsertDtmfWithNonDefaultCommaDelay) { |
| 326 | std::string tones = "3,4"; |
| 327 | int duration = 100; |
| 328 | int inter_tone_gap = 50; |
| 329 | int default_comma_delay = webrtc::DtmfSender::kDtmfDefaultCommaDelayMs; |
| 330 | int comma_delay = 500; |
| 331 | EXPECT_EQ(dtmf_->comma_delay(), default_comma_delay); |
| 332 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap, comma_delay)); |
| 333 | EXPECT_TRUE_SIMULATED_WAIT(observer_->completed(), kMaxWaitMs, fake_clock_); |
| 334 | |
| 335 | VerifyOnProvider(tones, duration, inter_tone_gap, comma_delay); |
| 336 | VerifyOnObserver(tones); |
| 337 | EXPECT_EQ(dtmf_->comma_delay(), comma_delay); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 338 | } |
| 339 | |
| 340 | TEST_F(DtmfSenderTest, TryInsertDtmfWhenItDoesNotWork) { |
| 341 | std::string tones = "3,4"; |
| 342 | int duration = 100; |
| 343 | int inter_tone_gap = 50; |
deadbeef | 20cb0c1 | 2017-02-02 04:27:00 | [diff] [blame] | 344 | provider_->SetCanInsertDtmf(false); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 345 | EXPECT_FALSE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
| 346 | } |
| 347 | |
| 348 | TEST_F(DtmfSenderTest, InsertDtmfWithInvalidDurationOrGap) { |
| 349 | std::string tones = "3,4"; |
dminor | 588101c | 2017-03-28 18:18:32 | [diff] [blame] | 350 | int duration = 40; |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 351 | int inter_tone_gap = 50; |
| 352 | |
| 353 | EXPECT_FALSE(dtmf_->InsertDtmf(tones, 6001, inter_tone_gap)); |
dminor | 588101c | 2017-03-28 18:18:32 | [diff] [blame] | 354 | EXPECT_FALSE(dtmf_->InsertDtmf(tones, 39, inter_tone_gap)); |
Harald Alvestrand | 52e5852 | 2018-02-20 07:15:36 | [diff] [blame] | 355 | EXPECT_FALSE(dtmf_->InsertDtmf(tones, duration, 29)); |
Aaron Alaniz | 529d886 | 2020-01-21 03:09:47 | [diff] [blame] | 356 | EXPECT_FALSE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap, 29)); |
henrike@webrtc.org | 28e2075 | 2013-07-10 00:45:36 | [diff] [blame] | 357 | |
| 358 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
| 359 | } |
Harald Alvestrand | 1e0c804 | 2018-03-06 09:51:27 | [diff] [blame] | 360 | |
| 361 | TEST_F(DtmfSenderTest, InsertDtmfSendsAfterWait) { |
| 362 | std::string tones = "ABC"; |
| 363 | int duration = 100; |
| 364 | int inter_tone_gap = 50; |
| 365 | EXPECT_TRUE(dtmf_->InsertDtmf(tones, duration, inter_tone_gap)); |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 366 | VerifyExpectedState("ABC", duration, inter_tone_gap); |
Harald Alvestrand | 1e0c804 | 2018-03-06 09:51:27 | [diff] [blame] | 367 | // Wait until the first tone got sent. |
| 368 | EXPECT_TRUE_SIMULATED_WAIT(observer_->tones().size() == 1, kMaxWaitMs, |
| 369 | fake_clock_); |
Steve Anton | b983bae | 2018-06-20 18:16:53 | [diff] [blame] | 370 | VerifyExpectedState("BC", duration, inter_tone_gap); |
Harald Alvestrand | 1e0c804 | 2018-03-06 09:51:27 | [diff] [blame] | 371 | } |