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

Unified Diff: mojo/shell/shell_apptest.cc

Issue 1049993002: Get mojo_shell building inside chromium checkout. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix presubmit Created 5 years, 9 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/shell/out_of_process_native_runner.cc ('k') | mojo/shell/shell_test_base.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/shell/shell_apptest.cc
diff --git a/mojo/shell/shell_apptest.cc b/mojo/shell/shell_apptest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1acdf7d1f4eaf7c56f0b3acb91be618e00209baa
--- /dev/null
+++ b/mojo/shell/shell_apptest.cc
@@ -0,0 +1,197 @@
+// Copyright 2015 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 "base/base_paths.h"
+#include "base/bind.h"
+#include "base/files/file_path.h"
+#include "base/files/file_util.h"
+#include "base/path_service.h"
+#include "base/run_loop.h"
+#include "base/strings/string_util.h"
+#include "base/strings/stringprintf.h"
+#include "mojo/common/data_pipe_utils.h"
+#include "mojo/public/cpp/application/application_impl.h"
+#include "mojo/public/cpp/application/application_test_base.h"
+#include "mojo/public/cpp/system/macros.h"
+#include "mojo/services/http_server/public/cpp/http_server_util.h"
+#include "mojo/services/http_server/public/interfaces/http_server.mojom.h"
+#include "mojo/services/http_server/public/interfaces/http_server_factory.mojom.h"
+#include "mojo/services/network/public/interfaces/net_address.mojom.h"
+#include "mojo/shell/kPingable.h"
+#include "mojo/shell/test/pingable.mojom.h"
+
+namespace mojo {
+namespace {
+
+std::string GetURL(uint16_t port, const std::string& path) {
+ return base::StringPrintf("http://127.0.0.1:%u/%s",
+ static_cast<unsigned>(port), path.c_str());
+}
+
+class GetHandler : public http_server::HttpHandler {
+ public:
+ GetHandler(InterfaceRequest<http_server::HttpHandler> request, uint16_t port)
+ : binding_(this, request.Pass()), port_(port) {
+ }
+ ~GetHandler() override {}
+
+ private:
+ // http_server::HttpHandler:
+ void HandleRequest(
+ http_server::HttpRequestPtr request,
+ const Callback<void(http_server::HttpResponsePtr)>& callback) override {
+ http_server::HttpResponsePtr response;
+ if (StartsWithASCII(request->relative_url, "/app", true)) {
+ response = http_server::CreateHttpResponse(
+ 200, std::string(kPingable.data, kPingable.size));
+ response->content_type = "application/octet-stream";
+ } else if (request->relative_url == "/redirect") {
+ response = http_server::HttpResponse::New();
+ response->status_code = 302;
+ response->custom_headers.insert("Location", GetURL(port_, "app"));
+ } else {
+ NOTREACHED();
+ }
+
+ callback.Run(response.Pass());
+ }
+
+ Binding<http_server::HttpHandler> binding_;
+ uint16_t port_;
+
+ MOJO_DISALLOW_COPY_AND_ASSIGN(GetHandler);
+};
+
+typedef test::ApplicationTestBase ShellAppTest;
+
+class ShellHTTPAppTest : public test::ApplicationTestBase {
+ public:
+ ShellHTTPAppTest() : ApplicationTestBase() {}
+ ~ShellHTTPAppTest() override {}
+
+ protected:
+ // ApplicationTestBase:
+ void SetUp() override {
+ ApplicationTestBase::SetUp();
+
+ application_impl()->ConnectToService("mojo:http_server",
+ &http_server_factory_);
+
+ NetAddressPtr local_address(NetAddress::New());
+ local_address->family = NET_ADDRESS_FAMILY_IPV4;
+ local_address->ipv4 = NetAddressIPv4::New();
+ local_address->ipv4->addr.resize(4);
+ local_address->ipv4->addr[0] = 127;
+ local_address->ipv4->addr[1] = 0;
+ local_address->ipv4->addr[2] = 0;
+ local_address->ipv4->addr[3] = 1;
+ local_address->ipv4->port = 0;
+ http_server_factory_->CreateHttpServer(GetProxy(&http_server_),
+ local_address.Pass());
+
+ http_server_->GetPort([this](uint16_t p) { port_ = p; });
+ EXPECT_TRUE(http_server_.WaitForIncomingMethodCall());
+
+ InterfacePtr<http_server::HttpHandler> http_handler;
+ handler_.reset(new GetHandler(GetProxy(&http_handler).Pass(), port_));
+ http_server_->SetHandler(".*", http_handler.Pass(),
+ [](bool result) { EXPECT_TRUE(result); });
+ EXPECT_TRUE(http_server_.WaitForIncomingMethodCall());
+ }
+
+ std::string GetURL(const std::string& path) {
+ return ::mojo::GetURL(port_, path);
+ }
+
+ http_server::HttpServerFactoryPtr http_server_factory_;
+ http_server::HttpServerPtr http_server_;
+ scoped_ptr<GetHandler> handler_;
+ uint16_t port_;
+
+ private:
+ MOJO_DISALLOW_COPY_AND_ASSIGN(ShellHTTPAppTest);
+};
+
+// Test that we can load apps over http.
+TEST_F(ShellHTTPAppTest, Http) {
+ InterfacePtr<Pingable> pingable;
+ application_impl()->ConnectToService(GetURL("app"), &pingable);
+ pingable->Ping("hello",
+ [this](const String& app_url, const String& connection_url,
+ const String& message) {
+ EXPECT_EQ(GetURL("app"), app_url);
+ EXPECT_EQ(GetURL("app"), connection_url);
+ EXPECT_EQ("hello", message);
+ base::MessageLoop::current()->Quit();
+ });
+ base::RunLoop().Run();
+}
+
+// Test that redirects work.
+// TODO(aa): Test that apps receive the correct URL parameters.
+TEST_F(ShellHTTPAppTest, Redirect) {
+ InterfacePtr<Pingable> pingable;
+ application_impl()->ConnectToService(GetURL("redirect"), &pingable);
+ pingable->Ping("hello",
+ [this](const String& app_url, const String& connection_url,
+ const String& message) {
+ EXPECT_EQ(GetURL("app"), app_url);
+ EXPECT_EQ(GetURL("app"), connection_url);
+ EXPECT_EQ("hello", message);
+ base::MessageLoop::current()->Quit();
+ });
+ base::RunLoop().Run();
+}
+
+// Test that querystring is not considered when resolving http applications.
+// TODO(aa|qsr): Fix this test on Linux ASAN http://crbug.com/463662
+#if defined(ADDRESS_SANITIZER)
+#define MAYBE_QueryHandling DISABLED_QueryHandling
+#else
+#define MAYBE_QueryHandling QueryHandling
+#endif // ADDRESS_SANITIZER
+TEST_F(ShellHTTPAppTest, MAYBE_QueryHandling) {
+ InterfacePtr<Pingable> pingable1;
+ InterfacePtr<Pingable> pingable2;
+ application_impl()->ConnectToService(GetURL("app?foo"), &pingable1);
+ application_impl()->ConnectToService(GetURL("app?bar"), &pingable2);
+
+ int num_responses = 0;
+ auto callback = [this, &num_responses](const String& app_url,
+ const String& connection_url,
+ const String& message) {
+ EXPECT_EQ(GetURL("app"), app_url);
+ EXPECT_EQ("hello", message);
+ ++num_responses;
+ if (num_responses == 1) {
+ EXPECT_EQ(GetURL("app?foo"), connection_url);
+ } else if (num_responses == 2) {
+ EXPECT_EQ(GetURL("app?bar"), connection_url);
+ base::MessageLoop::current()->Quit();
+ } else {
+ CHECK(false);
+ }
+ };
+ pingable1->Ping("hello", callback);
+ pingable2->Ping("hello", callback);
+ base::RunLoop().Run();
+}
+
+// mojo: URLs can have querystrings too
+TEST_F(ShellAppTest, MojoURLQueryHandling) {
+ InterfacePtr<Pingable> pingable;
+ application_impl()->ConnectToService("mojo:pingable_app?foo", &pingable);
+ auto callback = [this](const String& app_url, const String& connection_url,
+ const String& message) {
+ EXPECT_TRUE(EndsWith(app_url, "/pingable_app.mojo", true));
+ EXPECT_EQ(app_url.To<std::string>() + "?foo", connection_url);
+ EXPECT_EQ("hello", message);
+ base::MessageLoop::current()->Quit();
+ };
+ pingable->Ping("hello", callback);
+ base::RunLoop().Run();
+}
+
+} // namespace
+} // namespace mojo
« no previous file with comments | « mojo/shell/out_of_process_native_runner.cc ('k') | mojo/shell/shell_test_base.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698