Commit e9ad4df5 authored by Maksim Moskvitin's avatar Maksim Moskvitin Committed by Commit Bot

[Invalidations] Remove EqualsJSON matcher

Removed matcher is unused.

Bug: 1105388
Change-Id: Iafc5a2236873c1bc02ea1f1c7500647d714553de
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2494870
Commit-Queue: Maksim Moskvitin <mmoskvitin@google.com>
Commit-Queue: Marc Treib <treib@chromium.org>
Reviewed-by: default avatarMarc Treib <treib@chromium.org>
Auto-Submit: Maksim Moskvitin <mmoskvitin@google.com>
Cr-Commit-Position: refs/heads/master@{#820249}
parent 46c719d3
......@@ -5,7 +5,6 @@
#include "components/invalidation/impl/per_user_topic_subscription_request.h"
#include "base/bind.h"
#include "base/json/json_reader.h"
#include "base/memory/ptr_util.h"
#include "base/run_loop.h"
#include "base/strings/stringprintf.h"
......@@ -30,24 +29,6 @@ namespace {
using testing::_;
using testing::SaveArg;
MATCHER_P(EqualsJSON, json, "equals JSON") {
base::Optional<base::Value> expected = base::JSONReader::Read(json);
if (!expected) {
*result_listener << "INTERNAL ERROR: couldn't parse expected JSON";
return false;
}
base::JSONReader::ValueWithError actual =
base::JSONReader::ReadAndReturnValueWithError(arg);
if (!actual.value) {
*result_listener << "input:" << actual.error_line << ":"
<< actual.error_column << ": "
<< "parse error: " << actual.error_message;
return false;
}
return *expected == *actual.value;
}
network::mojom::URLResponseHeadPtr CreateHeadersForTest(int responce_code) {
auto head = network::mojom::URLResponseHead::New();
head->headers = new net::HttpResponseHeaders(base::StringPrintf(
......@@ -60,7 +41,8 @@ network::mojom::URLResponseHeadPtr CreateHeadersForTest(int responce_code) {
class PerUserTopicSubscriptionRequestTest : public testing::Test {
public:
PerUserTopicSubscriptionRequestTest() {}
PerUserTopicSubscriptionRequestTest() = default;
~PerUserTopicSubscriptionRequestTest() override = default;
GURL url(PerUserTopicSubscriptionRequest* request) {
return request->GetUrlForTesting();
......@@ -74,8 +56,6 @@ class PerUserTopicSubscriptionRequestTest : public testing::Test {
base::test::SingleThreadTaskEnvironment task_environment_;
data_decoder::test::InProcessDataDecoder in_process_data_decoder_;
network::TestURLLoaderFactory url_loader_factory_;
DISALLOW_COPY_AND_ASSIGN(PerUserTopicSubscriptionRequestTest);
};
TEST_F(PerUserTopicSubscriptionRequestTest,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment