Crop urls in Data Saver feedback reports

components/data_reduction_proxy/core/common/data_reduction_proxy_event_store_unittest.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/data_reduction_proxy/core/common/data_reduction_proxy_event_store.h"
 
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/json/json_writer.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/strings/string_number_conversions.h"
 #include "base/time/time.h"
 #include "base/values.h"
 #include "components/data_reduction_proxy/core/browser/data_reduction_proxy_configurator.h"
 #include "components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.h"
 #include "components/data_reduction_proxy/core/common/data_reduction_proxy_params_test_utils.h"
 #include "net/base/host_port_pair.h"
 #include "net/base/net_errors.h"
 #include "net/http/http_status_code.h"
 #include "net/log/net_log.h"
 #include "net/log/test_net_log.h"
@@ skipping 199 matching lines @@
                                           net::HostPortPair("foo.com", 80)));
   http_proxies.push_back(net::ProxyServer(net::ProxyServer::SCHEME_HTTPS,
                                           net::HostPortPair("bar.com", 443)));
   https_proxies.push_back(net::ProxyServer(net::ProxyServer::SCHEME_HTTP,
                                            net::HostPortPair("baz.com", 80)));
   configurator.Enable(false, http_proxies, https_proxies);
   EXPECT_EQ("foo.com:80;https://bar.com:443",
             event_store()->GetHttpProxyList());
   EXPECT_EQ("baz.com:80", event_store()->GetHttpsProxyList());
 
+  configurator.Disable();
+  EXPECT_EQ(std::string(), event_store()->GetHttpProxyList());
+  EXPECT_EQ(std::string(), event_store()->GetHttpsProxyList());
+}
+
+TEST_F(DataReductionProxyEventStoreTest, TestFeedbackLastBypassEventFullURL) {
+  DataReductionProxyConfigurator configurator(net_log(), event_creator());
+  std::vector<net::ProxyServer> http_proxies;
+  std::vector<net::ProxyServer> https_proxies;
+  configurator.Enable(false, http_proxies, https_proxies);
+
   scoped_ptr<base::DictionaryValue> bypass_event(new base::DictionaryValue());
   scoped_ptr<base::DictionaryValue> bypass_params(new base::DictionaryValue());
   scoped_ptr<base::DictionaryValue> sanitized_event(
       new base::DictionaryValue());
 
-  bypass_event->SetString("time", "12/31/2014 23:58");
-  sanitized_event->SetString("bypass_time", "12/31/2014 23:58");
+  std::string time = net::NetLog::TickCountToString(base::TimeTicks::Now());
+  bypass_event->SetString("time", time);
+  sanitized_event->SetString("bypass_time", time);
   bypass_params->SetInteger("bypass_type", 4);
   sanitized_event->SetInteger("bypass_type", 4);
   bypass_params->SetString("bypass_duration_seconds", "40");
   sanitized_event->SetString("bypass_seconds", "40");
   bypass_params->SetString("url", "http://www.foo.com/bar?baz=1234");
   sanitized_event->SetString("url", "http://www.foo.com/bar");
 
   bypass_event->Set("params", bypass_params.Pass());
   std::string sanitized_output;
   base::JSONWriter::Write(*sanitized_event.get(), &sanitized_output);
   event_store()->AddAndSetLastBypassEvent(bypass_event.Pass(), 0);
   EXPECT_EQ(sanitized_output, event_store()->SanitizedLastBypassEvent());
+}
 
-  configurator.Disable();
-  EXPECT_EQ(std::string(), event_store()->GetHttpProxyList());
-  EXPECT_EQ(std::string(), event_store()->GetHttpsProxyList());
+TEST_F(DataReductionProxyEventStoreTest, TestFeedbackLastBypassEventHostOnly) {
+  DataReductionProxyConfigurator configurator(net_log(), event_creator());
+  std::vector<net::ProxyServer> http_proxies;
+  std::vector<net::ProxyServer> https_proxies;
+  configurator.Enable(false, http_proxies, https_proxies);
+
+  scoped_ptr<base::DictionaryValue> bypass_event(new base::DictionaryValue());
+  scoped_ptr<base::DictionaryValue> bypass_params(new base::DictionaryValue());
+  scoped_ptr<base::DictionaryValue> sanitized_event(
+      new base::DictionaryValue());
+
+  int64 delta_time =
+      (base::TimeTicks::Now() - base::TimeTicks()).InMilliseconds();
+  // Set bypass event time to be 5 minutes ago.
+  delta_time -= 300000;
+  std::string time = base::Int64ToString(delta_time);
+  bypass_event->SetString("time", time);
+  sanitized_event->SetString("bypass_time", time);
+  bypass_params->SetInteger("bypass_type", 4);
+  sanitized_event->SetInteger("bypass_type", 4);
+  bypass_params->SetString("bypass_duration_seconds", "40");
+  sanitized_event->SetString("bypass_seconds", "40");
+  bypass_params->SetString("url", "http://www.foo.com/bar?baz=1234");
+  sanitized_event->SetString("url", "www.foo.com");
+
+  bypass_event->Set("params", bypass_params.Pass());
+  std::string sanitized_output;
+  base::JSONWriter::Write(*sanitized_event.get(), &sanitized_output);
+  event_store()->AddAndSetLastBypassEvent(bypass_event.Pass(), 0);
+  EXPECT_EQ(sanitized_output, event_store()->SanitizedLastBypassEvent());
 }
 
 }  // namespace data_reduction_proxy