summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
-rwxr-xr-xandroid_webview/tools/webview_licenses.py2
-rw-r--r--gin/converter.cc8
-rw-r--r--gin/converter.h2
-rw-r--r--gin/runner.cc5
-rw-r--r--gin/runner.h2
-rw-r--r--gin/runner_unittest.cc2
-rw-r--r--gin/shell/gin_main.cc13
-rw-r--r--gin/test/expect.js289
-rw-r--r--gin/test/file_runner.cc9
-rw-r--r--gin/test/gtest.cc12
-rw-r--r--gin/try_catch.cc32
-rw-r--r--gin/try_catch.h2
-rw-r--r--mojo/apps/js/main.cc3
-rw-r--r--mojo/apps/js/mojo_runner_delegate.cc2
-rw-r--r--mojo/public/bindings/js/codec_unittests.js99
-rw-r--r--mojo/public/bindings/js/core_unittests.js31
-rw-r--r--mojo/public/bindings/sample/sample_test.js58
17 files changed, 424 insertions, 147 deletions
diff --git a/android_webview/tools/webview_licenses.py b/android_webview/tools/webview_licenses.py
index 98653c0..9702465 100755
--- a/android_webview/tools/webview_licenses.py
+++ b/android_webview/tools/webview_licenses.py
@@ -105,6 +105,8 @@ def _CheckLicenseHeaders(excluded_dirs_list, whitelisted_files):
excluded_dirs_list.append('chrome/tools/test/reference_build')
# This is tests directory, doesn't exist in the snapshot
excluded_dirs_list.append('content/test/data')
+ # This is a tests directory that doesn't exist in the shipped product.
+ excluded_dirs_list.append('gin/test')
# This is a test output directory
excluded_dirs_list.append('data/dom_perf')
# Histogram tools, doesn't exist in the snapshot
diff --git a/gin/converter.cc b/gin/converter.cc
index e60e3a0..614e515 100644
--- a/gin/converter.cc
+++ b/gin/converter.cc
@@ -171,5 +171,13 @@ v8::Handle<v8::String> StringToSymbol(v8::Isolate* isolate,
static_cast<uint32_t>(val.length()));
}
+std::string V8ToString(v8::Handle<v8::Value> value) {
+ if (value.IsEmpty())
+ return std::string();
+ std::string result;
+ if (!ConvertFromV8(value, &result))
+ return std::string();
+ return result;
+}
} // namespace gin
diff --git a/gin/converter.h b/gin/converter.h
index 2279319..c22c072 100644
--- a/gin/converter.h
+++ b/gin/converter.h
@@ -162,6 +162,8 @@ bool ConvertFromV8(v8::Handle<v8::Value> input, T* result) {
return Converter<T>::FromV8(input, result);
}
+std::string V8ToString(v8::Handle<v8::Value> value);
+
} // namespace gin
#endif // GIN_CONVERTER_H_
diff --git a/gin/runner.cc b/gin/runner.cc
index 0cd7269..37e16a0 100644
--- a/gin/runner.cc
+++ b/gin/runner.cc
@@ -58,8 +58,9 @@ Runner::Runner(RunnerDelegate* delegate, Isolate* isolate)
Runner::~Runner() {
}
-void Runner::Run(const std::string& script) {
- Run(Script::New(StringToV8(isolate(), script)));
+void Runner::Run(const std::string& source, const std::string& resource_name) {
+ Run(Script::New(StringToV8(isolate(), source),
+ StringToV8(isolate(), resource_name)));
}
void Runner::Run(v8::Handle<Script> script) {
diff --git a/gin/runner.h b/gin/runner.h
index 614b60d..21e656f 100644
--- a/gin/runner.h
+++ b/gin/runner.h
@@ -33,7 +33,7 @@ class Runner : public ContextHolder {
Runner(RunnerDelegate* delegate, v8::Isolate* isolate);
~Runner();
- void Run(const std::string& script);
+ void Run(const std::string& source, const std::string& resource_name);
void Run(v8::Handle<v8::Script> script);
v8::Handle<v8::Value> Call(v8::Handle<v8::Function> function,
diff --git a/gin/runner_unittest.cc b/gin/runner_unittest.cc
index 9c3c30e..9493cf1 100644
--- a/gin/runner_unittest.cc
+++ b/gin/runner_unittest.cc
@@ -25,7 +25,7 @@ TEST(RunnerTest, Run) {
Isolate* isolate = instance.isolate();
Runner runner(&delegate, isolate);
Runner::Scope scope(&runner);
- runner.Run(source);
+ runner.Run(source, "test_data.js");
std::string result;
EXPECT_TRUE(Converter<std::string>::FromV8(
diff --git a/gin/shell/gin_main.cc b/gin/shell/gin_main.cc
index 9d081a3..24fa6df 100644
--- a/gin/shell/gin_main.cc
+++ b/gin/shell/gin_main.cc
@@ -24,11 +24,11 @@ std::string Load(const base::FilePath& path) {
return source;
}
-void Run(base::WeakPtr<Runner> runner, const std::string& source) {
+void Run(base::WeakPtr<Runner> runner, const base::FilePath& path) {
if (!runner)
return;
Runner::Scope scope(runner.get());
- runner->Run(source);
+ runner->Run(Load(path), path.AsUTF8Unsafe());
}
std::vector<base::FilePath> GetModuleSearchPaths() {
@@ -46,7 +46,7 @@ class ShellRunnerDelegate : public ModuleRunnerDelegate {
virtual void UnhandledException(Runner* runner,
TryCatch& try_catch) OVERRIDE {
ModuleRunnerDelegate::UnhandledException(runner, try_catch);
- LOG(ERROR) << try_catch.GetPrettyMessage();
+ LOG(ERROR) << try_catch.GetStackTrace();
}
private:
@@ -68,11 +68,16 @@ int main(int argc, char** argv) {
gin::ShellRunnerDelegate delegate;
gin::Runner runner(&delegate, instance.isolate());
+ {
+ gin::Runner::Scope scope(&runner);
+ v8::V8::SetCaptureStackTraceForUncaughtExceptions(true);
+ }
+
CommandLine::StringVector args = CommandLine::ForCurrentProcess()->GetArgs();
for (CommandLine::StringVector::const_iterator it = args.begin();
it != args.end(); ++it) {
base::MessageLoop::current()->PostTask(FROM_HERE, base::Bind(
- gin::Run, runner.GetWeakPtr(), gin::Load(base::FilePath(*it))));
+ gin::Run, runner.GetWeakPtr(), base::FilePath(*it)));
}
message_loop.RunUntilIdle();
diff --git a/gin/test/expect.js b/gin/test/expect.js
new file mode 100644
index 0000000..4154456
--- /dev/null
+++ b/gin/test/expect.js
@@ -0,0 +1,289 @@
+// Copyright 2013 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.
+
+define(["console"], function(console) {
+ // Equality function based on isEqual in
+ // Underscore.js 1.5.2
+ // http://underscorejs.org
+ // (c) 2009-2013 Jeremy Ashkenas,
+ // DocumentCloud,
+ // and Investigative Reporters & Editors
+ // Underscore may be freely distributed under the MIT license.
+ //
+ function has(obj, key) {
+ return obj.hasOwnProperty(key);
+ }
+ function isFunction(obj) {
+ return typeof obj === 'function';
+ }
+ function isArrayBufferClass(className) {
+ return className == '[object ArrayBuffer]' ||
+ className.match(/\[object \w+\d+(Clamped)?Array\]/);
+ }
+ // Internal recursive comparison function for `isEqual`.
+ function eq(a, b, aStack, bStack) {
+ // Identical objects are equal. `0 === -0`, but they aren't identical.
+ // See the Harmony `egal` proposal:
+ // http://wiki.ecmascript.org/doku.php?id=harmony:egal.
+ if (a === b)
+ return a !== 0 || 1 / a == 1 / b;
+ // A strict comparison is necessary because `null == undefined`.
+ if (a == null || b == null)
+ return a === b;
+ // Compare `[[Class]]` names.
+ var className = toString.call(a);
+ if (className != toString.call(b))
+ return false;
+ switch (className) {
+ // Strings, numbers, dates, and booleans are compared by value.
+ case '[object String]':
+ // Primitives and their corresponding object wrappers are equivalent;
+ // thus, `"5"` is equivalent to `new String("5")`.
+ return a == String(b);
+ case '[object Number]':
+ // `NaN`s are equivalent, but non-reflexive. An `egal` comparison is
+ // performed for other numeric values.
+ return a != +a ? b != +b : (a == 0 ? 1 / a == 1 / b : a == +b);
+ case '[object Date]':
+ case '[object Boolean]':
+ // Coerce dates and booleans to numeric primitive values. Dates are
+ // compared by their millisecond representations. Note that invalid
+ // dates with millisecond representations of `NaN` are not equivalent.
+ return +a == +b;
+ // RegExps are compared by their source patterns and flags.
+ case '[object RegExp]':
+ return a.source == b.source &&
+ a.global == b.global &&
+ a.multiline == b.multiline &&
+ a.ignoreCase == b.ignoreCase;
+ }
+ if (typeof a != 'object' || typeof b != 'object')
+ return false;
+ // Assume equality for cyclic structures. The algorithm for detecting
+ // cyclic structures is adapted from ES 5.1 section 15.12.3, abstract
+ // operation `JO`.
+ var length = aStack.length;
+ while (length--) {
+ // Linear search. Performance is inversely proportional to the number of
+ // unique nested structures.
+ if (aStack[length] == a)
+ return bStack[length] == b;
+ }
+ // Objects with different constructors are not equivalent, but `Object`s
+ // from different frames are.
+ var aCtor = a.constructor, bCtor = b.constructor;
+ if (aCtor !== bCtor && !(isFunction(aCtor) && (aCtor instanceof aCtor) &&
+ isFunction(bCtor) && (bCtor instanceof bCtor))
+ && ('constructor' in a && 'constructor' in b)) {
+ return false;
+ }
+ // Add the first object to the stack of traversed objects.
+ aStack.push(a);
+ bStack.push(b);
+ var size = 0, result = true;
+ // Recursively compare objects and arrays.
+ if (className == '[object Array]' || isArrayBufferClass(className)) {
+ // Compare array lengths to determine if a deep comparison is necessary.
+ size = a.length;
+ result = size == b.length;
+ if (result) {
+ // Deep compare the contents, ignoring non-numeric properties.
+ while (size--) {
+ if (!(result = eq(a[size], b[size], aStack, bStack)))
+ break;
+ }
+ }
+ } else {
+ // Deep compare objects.
+ for (var key in a) {
+ if (has(a, key)) {
+ // Count the expected number of properties.
+ size++;
+ // Deep compare each member.
+ if (!(result = has(b, key) && eq(a[key], b[key], aStack, bStack)))
+ break;
+ }
+ }
+ // Ensure that both objects contain the same number of properties.
+ if (result) {
+ for (key in b) {
+ if (has(b, key) && !(size--))
+ break;
+ }
+ result = !size;
+ }
+ }
+ // Remove the first object from the stack of traversed objects.
+ aStack.pop();
+ bStack.pop();
+ return result;
+ };
+
+ function describe(subjects) {
+ var descriptions = [];
+ Object.getOwnPropertyNames(subjects).forEach(function(name) {
+ if (name === "Description")
+ descriptions.push(subjects[name]);
+ else
+ descriptions.push(name + ": " + JSON.stringify(subjects[name]));
+ });
+ return descriptions.join(" ");
+ }
+
+ var predicates = {};
+
+ predicates.toBe = function(actual, expected) {
+ return {
+ "result": actual === expected,
+ "message": describe({
+ "Actual": actual,
+ "Expected": expected,
+ }),
+ };
+ };
+
+ predicates.toEqual = function(actual, expected) {
+ return {
+ "result": eq(actual, expected, [], []),
+ "message": describe({
+ "Actual": actual,
+ "Expected": expected,
+ }),
+ };
+ };
+
+ predicates.toBeDefined = function(actual) {
+ return {
+ "result": typeof actual !== "undefined",
+ "message": describe({
+ "Actual": actual,
+ "Description": "Expected a defined value",
+ }),
+ };
+ };
+
+ predicates.toBeUndefined = function(actual) {
+ // Recall: undefined is just a global variable. :)
+ return {
+ "result": typeof actual === "undefined",
+ "message": describe({
+ "Actual": actual,
+ "Description": "Expected an undefined value",
+ }),
+ };
+ };
+
+ predicates.toBeNull = function(actual) {
+ // Recall: typeof null === "object".
+ return {
+ "result": actual === null,
+ "message": describe({
+ "Actual": actual,
+ "Expected": null,
+ }),
+ };
+ };
+
+ predicates.toBeTruthy = function(actual) {
+ return {
+ "result": !!actual,
+ "message": describe({
+ "Actual": actual,
+ "Description": "Expected a truthy value",
+ }),
+ };
+ };
+
+ predicates.toBeFalsy = function(actual) {
+ return {
+ "result": !!!actual,
+ "message": describe({
+ "Actual": actual,
+ "Description": "Expected a falsy value",
+ }),
+ };
+ };
+
+ predicates.toContain = function(actual, element) {
+ return {
+ "result": (function () {
+ for (var i = 0; i < actual.length; ++i) {
+ if (eq(actual[i], element, [], []))
+ return true;
+ }
+ return false;
+ })(),
+ "message": describe({
+ "Actual": actual,
+ "Element": element,
+ }),
+ };
+ };
+
+ predicates.toBeLessThan = function(actual, reference) {
+ return {
+ "result": actual < reference,
+ "message": describe({
+ "Actual": actual,
+ "Reference": reference,
+ }),
+ };
+ };
+
+ predicates.toBeGreaterThan = function(actual, reference) {
+ return {
+ "result": actual > reference,
+ "message": describe({
+ "Actual": actual,
+ "Reference": reference,
+ }),
+ };
+ };
+
+ predicates.toThrow = function(actual) {
+ return {
+ "result": (function () {
+ if (!isFunction(actual))
+ throw new TypeError;
+ try {
+ actual();
+ } catch (ex) {
+ return true;
+ }
+ return false;
+ })(),
+ "message": "Expected function to throw",
+ };
+ }
+
+ function negate(predicate) {
+ return function() {
+ var outcome = predicate.apply(null, arguments);
+ outcome.result = !outcome.result;
+ return outcome;
+ }
+ }
+
+ function check(predicate) {
+ return function() {
+ var outcome = predicate.apply(null, arguments);
+ if (outcome.result)
+ return;
+ throw outcome.message;
+ };
+ }
+
+ function Condition(actual) {
+ this.not = {};
+ Object.getOwnPropertyNames(predicates).forEach(function(name) {
+ var bound = predicates[name].bind(null, actual);
+ this[name] = check(bound);
+ this.not[name] = check(negate(bound));
+ }, this);
+ }
+
+ return function(actual) {
+ return new Condition(actual);
+ };
+});
diff --git a/gin/test/file_runner.cc b/gin/test/file_runner.cc
index 0c60c4b..2bad0b2 100644
--- a/gin/test/file_runner.cc
+++ b/gin/test/file_runner.cc
@@ -41,7 +41,7 @@ FileRunnerDelegate::~FileRunnerDelegate() {
void FileRunnerDelegate::UnhandledException(Runner* runner,
TryCatch& try_catch) {
ModuleRunnerDelegate::UnhandledException(runner, try_catch);
- EXPECT_FALSE(try_catch.HasCaught()) << try_catch.GetPrettyMessage();
+ FAIL() << try_catch.GetStackTrace();
}
void RunTestFromFile(const base::FilePath& path, FileRunnerDelegate* delegate) {
@@ -55,15 +55,14 @@ void RunTestFromFile(const base::FilePath& path, FileRunnerDelegate* delegate) {
gin::Runner runner(delegate, instance.isolate());
{
gin::Runner::Scope scope(&runner);
- runner.Run(source);
+ v8::V8::SetCaptureStackTraceForUncaughtExceptions(true);
+ runner.Run(source, path.AsUTF8Unsafe());
message_loop.RunUntilIdle();
v8::Handle<v8::Value> result = runner.context()->Global()->Get(
StringToSymbol(runner.isolate(), "result"));
- std::string result_string;
- ASSERT_TRUE(ConvertFromV8(result, &result_string));
- EXPECT_EQ("PASS", result_string);
+ EXPECT_EQ("PASS", V8ToString(result));
}
}
diff --git a/gin/test/gtest.cc b/gin/test/gtest.cc
index 422d3ec..254c735 100644
--- a/gin/test/gtest.cc
+++ b/gin/test/gtest.cc
@@ -18,6 +18,16 @@ namespace gin {
namespace {
+void Fail(const v8::FunctionCallbackInfo<v8::Value>& info) {
+ Arguments args(info);
+
+ std::string description;
+ if (!args.GetNext(&description))
+ return args.ThrowError();
+
+ FAIL() << description;
+}
+
void ExpectTrue(bool condition, const std::string& description) {
EXPECT_TRUE(condition) << description;
}
@@ -48,6 +58,8 @@ v8::Local<v8::ObjectTemplate> GTest::GetTemplate(v8::Isolate* isolate) {
data->GetObjectTemplate(&g_wrapper_info);
if (templ.IsEmpty()) {
templ = v8::ObjectTemplate::New();
+ templ->Set(StringToSymbol(isolate, "fail"),
+ v8::FunctionTemplate::New(Fail));
templ->Set(StringToSymbol(isolate, "expectTrue"),
CreateFunctionTempate(isolate, base::Bind(ExpectTrue)));
templ->Set(StringToSymbol(isolate, "expectFalse"),
diff --git a/gin/try_catch.cc b/gin/try_catch.cc
index 302d8bd..89a969f 100644
--- a/gin/try_catch.cc
+++ b/gin/try_catch.cc
@@ -4,6 +4,9 @@
#include "gin/try_catch.h"
+#include <sstream>
+
+#include "base/logging.h"
#include "gin/converter.h"
namespace gin {
@@ -18,15 +21,26 @@ bool TryCatch::HasCaught() {
return try_catch_.HasCaught();
}
-std::string TryCatch::GetPrettyMessage() {
- std::string info;
- ConvertFromV8(try_catch_.Message()->Get(), &info);
-
- std::string sounce_line;
- if (ConvertFromV8(try_catch_.Message()->GetSourceLine(), &sounce_line))
- info += "\n" + sounce_line;
-
- return info;
+std::string TryCatch::GetStackTrace() {
+ std::stringstream ss;
+ v8::Handle<v8::Message> message = try_catch_.Message();
+ ss << V8ToString(message->Get()) << std::endl
+ << V8ToString(message->GetSourceLine()) << std::endl;
+
+ v8::Handle<v8::StackTrace> trace = message->GetStackTrace();
+ if (trace.IsEmpty())
+ return ss.str();
+
+ int len = trace->GetFrameCount();
+ for (int i = 0; i < len; ++i) {
+ v8::Handle<v8::StackFrame> frame = trace->GetFrame(i);
+ ss << V8ToString(frame->GetScriptName()) << ":"
+ << frame->GetLineNumber() << ":"
+ << frame->GetColumn() << ": "
+ << V8ToString(frame->GetFunctionName())
+ << std::endl;
+ }
+ return ss.str();
}
} // namespace gin
diff --git a/gin/try_catch.h b/gin/try_catch.h
index 8026b3b..43f68ac 100644
--- a/gin/try_catch.h
+++ b/gin/try_catch.h
@@ -18,7 +18,7 @@ class TryCatch {
~TryCatch();
bool HasCaught();
- std::string GetPrettyMessage();
+ std::string GetStackTrace();
private:
v8::TryCatch try_catch_;
diff --git a/mojo/apps/js/main.cc b/mojo/apps/js/main.cc
index 0706182..5a6842a 100644
--- a/mojo/apps/js/main.cc
+++ b/mojo/apps/js/main.cc
@@ -32,7 +32,8 @@ void RunMojoJS(MojoHandle pipe) {
{
gin::Runner::Scope scope(&runner);
- runner.Run("define(['mojo/apps/js/main'], function(main) {});");
+ runner.Run("define(['mojo/apps/js/main'], function(main) {});",
+ "mojo.js");
}
base::MessageLoop::current()->Run();
diff --git a/mojo/apps/js/mojo_runner_delegate.cc b/mojo/apps/js/mojo_runner_delegate.cc
index ba085e5..610ede4 100644
--- a/mojo/apps/js/mojo_runner_delegate.cc
+++ b/mojo/apps/js/mojo_runner_delegate.cc
@@ -43,7 +43,7 @@ MojoRunnerDelegate::~MojoRunnerDelegate() {
void MojoRunnerDelegate::UnhandledException(gin::Runner* runner,
gin::TryCatch& try_catch) {
gin::ModuleRunnerDelegate::UnhandledException(runner, try_catch);
- LOG(ERROR) << try_catch.GetPrettyMessage();
+ LOG(ERROR) << try_catch.GetStackTrace();
}
} // namespace apps
diff --git a/mojo/public/bindings/js/codec_unittests.js b/mojo/public/bindings/js/codec_unittests.js
index 5624787..01ab143 100644
--- a/mojo/public/bindings/js/codec_unittests.js
+++ b/mojo/public/bindings/js/codec_unittests.js
@@ -3,31 +3,15 @@
// found in the LICENSE file.
define([
- "gtest",
+ "gin/test/expect",
"mojo/public/bindings/js/codec",
"mojom/sample_service",
- ], function(gtest, codec, sample) {
+ ], function(expect, codec, sample) {
testBar();
testFoo();
testAlign();
this.result = "PASS";
- function barMatches(bar, expected) {
- gtest.expectEqual(bar.alpha, expected.alpha,
- "bar.alpha is " + bar.alpha);
- gtest.expectEqual(bar.beta, expected.beta,
- "bar.beta is " + bar.beta);
- gtest.expectEqual(bar.gamma, expected.gamma,
- "bar.gamma is " + bar.gamma);
- }
-
- function equalsArray(actual, expected, tag) {
- for (var i = 0; i < expected.length; ++i) {
- gtest.expectEqual(actual[i], expected[i],
- tag + "[" + i + "] is " + actual[i]);
- }
- }
-
function testBar() {
var bar = new sample.Bar();
bar.alpha = 1;
@@ -43,7 +27,7 @@ define([
var message = builder.finish();
- var expectedMemory = [
+ var expectedMemory = new Uint8Array([
24, 0, 0, 0,
42, 0, 0, 0,
@@ -52,25 +36,21 @@ define([
1, 2, 3, 0,
0, 0, 0, 0,
- ];
+ ]);
- for (var i = 0; i < expectedMemory.length; ++i) {
- gtest.expectEqual(message.memory[i], expectedMemory[i],
- "message.memory[" + i + "] is " + message.memory[i]);
- }
+ expect(message.memory).toEqual(expectedMemory);
var reader = new codec.MessageReader(message);
- gtest.expectEqual(reader.payloadSize, payloadSize,
- "reader.payloadSize is " + reader.payloadSize);
- gtest.expectEqual(reader.messageName, messageName,
- "reader.messageName is " + reader.messageName);
+ expect(reader.payloadSize).toBe(payloadSize);
+ expect(reader.messageName).toBe(messageName);
var bar2 = reader.decodeStruct(sample.Bar);
- barMatches(bar2, bar);
- gtest.expectFalse("extraProperty" in bar2,
- "extraProperty appeared in bar2")
+ expect(bar2.alpha).toBe(bar.alpha);
+ expect(bar2.beta).toBe(bar.beta);
+ expect(bar2.gamma).toBe(bar.gamma);
+ expect("extraProperty" in bar2).toBeFalsy();
}
function testFoo() {
@@ -109,58 +89,45 @@ define([
var message = builder.finish();
- var expectedMemory = [
+ var expectedMemory = new Uint8Array([
/* 0: */ 240, 0, 0, 0, 31, 0, 0, 0,
/* 8: */ 64, 0, 0, 0, 10, 0, 0, 0,
/* 16: */ 0xD5, 0xB4, 0x12, 0x02, 0x93, 0x6E, 0x01, 0,
/* 24: */ 5, 0, 0, 0, 0, 0, 0, 0,
/* 32: */ 40, 0, 0, 0, 0, 0, 0, 0,
- // TODO(abarth): Test more of the message's raw memory.
- ];
+ ]);
+ // TODO(abarth): Test more of the message's raw memory.
+ var actualMemory = new Uint8Array(message.memory.buffer,
+ 0, expectedMemory.length);
- equalsArray(message.memory, expectedMemory, "message.memory");
+ expect(actualMemory).toEqual(expectedMemory);
var expectedHandles = [
23423782, 32549823, 98320423, 38502383, 92834093,
];
- equalsArray(message.handles, expectedHandles, "message.handles");
+ expect(message.handles).toEqual(expectedHandles);
var reader = new codec.MessageReader(message);
- gtest.expectEqual(reader.payloadSize, payloadSize,
- "reader.payloadSize is " + reader.payloadSize);
- gtest.expectEqual(reader.messageName, messageName,
- "reader.messageName is " + reader.messageName);
+ expect(reader.payloadSize).toBe(payloadSize);
+ expect(reader.messageName).toBe(messageName);
var foo2 = reader.decodeStruct(sample.Foo);
- gtest.expectEqual
-
- gtest.expectEqual(foo2.x, foo.x,
- "foo2.x is " + foo2.x);
- gtest.expectEqual(foo2.y, foo.y,
- "foo2.y is " + foo2.y);
+ expect(foo2.x).toBe(foo.x);
+ expect(foo2.y).toBe(foo.y);
- gtest.expectEqual(foo2.a, foo.a & 1 ? true : false,
- "foo2.a is " + foo2.a);
- gtest.expectEqual(foo2.b, foo.b & 1 ? true : false,
- "foo2.b is " + foo2.b);
- gtest.expectEqual(foo2.c, foo.c & 1 ? true : false,
- "foo2.c is " + foo2.c);
+ expect(foo2.a).toBe(foo.a & 1 ? true : false);
+ expect(foo2.b).toBe(foo.b & 1 ? true : false);
+ expect(foo2.c).toBe(foo.c & 1 ? true : false);
- barMatches(foo2.bar, foo.bar);
- equalsArray(foo2.data, foo.data, "foo2.data");
+ expect(foo2.bar).toEqual(foo.bar);
+ expect(foo2.data).toEqual(foo.data);
- var actualExtraBarsJSON = JSON.stringify(foo2.extra_bars);
- var expectedExtraBarsJSON = JSON.stringify(foo.extra_bars);
- gtest.expectEqual(actualExtraBarsJSON, expectedExtraBarsJSON,
- actualExtraBarsJSON);
-
- gtest.expectEqual(foo2.name, foo.name,
- "foo2.name is " + foo2.name);
-
- equalsArray(foo2.files, foo.files, "foo2.files");
+ expect(foo2.extra_bars).toEqual(foo.extra_bars);
+ expect(foo2.name).toBe(foo.name);
+ expect(foo2.files).toEqual(foo.files);
}
function testAlign() {
@@ -187,9 +154,7 @@ define([
24, // 19
24, // 20
];
- for (var i = 0; i < aligned.length; ++i) {
- gtest.expectEqual(codec.align(i), aligned[i],
- "codec.align(" + i + ") is " + codec.align(i));
- }
+ for (var i = 0; i < aligned.length; ++i)
+ expect(codec.align(i)).toBe(aligned[i]);
}
});
diff --git a/mojo/public/bindings/js/core_unittests.js b/mojo/public/bindings/js/core_unittests.js
index 5cddc06..bc3b55e 100644
--- a/mojo/public/bindings/js/core_unittests.js
+++ b/mojo/public/bindings/js/core_unittests.js
@@ -3,9 +3,9 @@
// found in the LICENSE file.
define([
- "gtest",
+ "gin/test/expect",
"mojo/public/bindings/js/core",
- ], function(gtest, core) {
+ ], function(expect, core) {
runWithPipe(testNop);
runWithPipe(testReadAndWriteMessage);
this.result = "PASS";
@@ -15,13 +15,8 @@ define([
test(pipe);
- var result0 = core.close(pipe.handle0);
- gtest.expectEqual(result0, core.RESULT_OK,
- "result0 is " + result0);
-
- var result1 = core.close(pipe.handle1);
- gtest.expectEqual(result1, core.RESULT_OK,
- "result1 is " + result1);
+ expect(core.close(pipe.handle0)).toBe(core.RESULT_OK);
+ expect(core.close(pipe.handle1)).toBe(core.RESULT_OK);
}
function testNop(pipe) {
@@ -37,25 +32,19 @@ define([
pipe.handle0, senderData, [],
core.WRITE_MESSAGE_FLAG_NONE);
- gtest.expectEqual(result, core.RESULT_OK,
- "writeMessage returned RESULT_OK: " + result);
+ expect(result).toBe(core.RESULT_OK);
var receiverData = new Uint8Array(50);
var read = core.readMessage(
pipe.handle1, core.READ_MESSAGE_FLAG_NONE)
- gtest.expectEqual(read.result, core.RESULT_OK,
- "read.result is " + read.result);
- gtest.expectEqual(read.buffer.byteLength, 42,
- "read.buffer.byteLength is " + read.buffer.byteLength);
- gtest.expectEqual(read.handles.length, 0,
- "read.handles.length is " + read.handles.length);
+ expect(read.result).toBe(core.RESULT_OK);
+ expect(read.buffer.byteLength).toBe(42);
+ expect(read.handles.length).toBe(0);
var memory = new Uint8Array(read.buffer);
- for (var i = 0; i < memory.length; ++i) {
- gtest.expectEqual(memory[i], (i * i) & 0xFF,
- "memory[" + i + "] is " + memory[i]);
- }
+ for (var i = 0; i < memory.length; ++i)
+ expect(memory[i]).toBe((i * i) & 0xFF);
}
});
diff --git a/mojo/public/bindings/sample/sample_test.js b/mojo/public/bindings/sample/sample_test.js
index 66fe158..54cf822 100644
--- a/mojo/public/bindings/sample/sample_test.js
+++ b/mojo/public/bindings/sample/sample_test.js
@@ -5,13 +5,13 @@
define([
"console",
"mojo/public/bindings/js/test/hexdump",
- "gtest",
+ "gin/test/expect",
// TODO(abarth): We shouldn't need to depend on codec, but there seems to
// be a bug in the module loading system whereby this test doesn't run if
// we don't import codec here.
"mojo/public/bindings/js/codec",
"mojom/sample_service"
- ], function(console, hexdump, gtest, codec, sample) {
+ ], function(console, hexdump, expect, codec, sample) {
var global = this;
@@ -59,41 +59,31 @@ define([
// Check that the given |Foo| is identical to the one made by |MakeFoo()|.
function checkFoo(foo) {
- gtest.expectEqual(foo.name, "foopy", "foo.name is " + foo.name);
- gtest.expectEqual(foo.x, 1, "foo.x is " + foo.x);
- gtest.expectEqual(foo.y, 2, "foo.y is " + foo.y);
- gtest.expectFalse(foo.a, "foo.a is " + foo.a);
- gtest.expectTrue(foo.b, "foo.b is " + foo.b);
- gtest.expectFalse(foo.c, "foo.c is " + foo.c);
- gtest.expectEqual(foo.bar.alpha, 20, "foo.bar.alpha is " + foo.bar.alpha);
- gtest.expectEqual(foo.bar.beta, 40, "foo.bar.beta is " + foo.bar.beta);
- gtest.expectEqual(foo.bar.gamma, 60, "foo.bar.gamma is " + foo.bar.gamma);
-
- gtest.expectEqual(foo.extra_bars.length, 3,
- "foo.extra_bars.length is " + foo.extra_bars.length);
+ expect(foo.name).toBe("foopy");
+ expect(foo.x).toBe(1);
+ expect(foo.y).toBe(2);
+ expect(foo.a).toBeFalsy();
+ expect(foo.b).toBeTruthy();
+ expect(foo.c).toBeFalsy();
+ expect(foo.bar.alpha).toBe(20);
+ expect(foo.bar.beta).toBe(40);
+ expect(foo.bar.gamma).toBe(60);
+
+ expect(foo.extra_bars.length).toBe(3);
for (var i = 0; i < foo.extra_bars.length; ++i) {
var base = i * 100;
- gtest.expectEqual(foo.extra_bars[i].alpha, base,
- "foo.extra_bars[" + i + "].alpha is " + foo.extra_bars[i].alpha);
- gtest.expectEqual(foo.extra_bars[i].beta, base + 20,
- "foo.extra_bars[" + i + "].beta is " + foo.extra_bars[i].beta);
- gtest.expectEqual(foo.extra_bars[i].gamma, base + 40,
- "foo.extra_bars[" + i + "].gamma is " + foo.extra_bars[i].gamma);
+ expect(foo.extra_bars[i].alpha).toBe(base);
+ expect(foo.extra_bars[i].beta).toBe(base + 20);
+ expect(foo.extra_bars[i].gamma).toBe(base + 40);
}
- gtest.expectEqual(foo.data.length, 10,
- "foo.data.length is " + foo.data.length);
- for (var i = 0; i < foo.data.length; ++i) {
- gtest.expectEqual(foo.data[i], foo.data.length - i,
- "foo.data[" + i + "] is " + foo.data[i]);
- }
+ expect(foo.data.length).toBe(10);
+ for (var i = 0; i < foo.data.length; ++i)
+ expect(foo.data[i]).toBe(foo.data.length - i);
- gtest.expectEqual(foo.files.length, 4,
- "foo.files.length " + foo.files.length);
- for (var i = 0; i < foo.files.length; ++i) {
- gtest.expectEqual(foo.files[i], 0xFFFF - i,
- "foo.files[" + i + "] is " + foo.files[i]);
- }
+ expect(foo.files.length).toBe(4);
+ for (var i = 0; i < foo.files.length; ++i)
+ expect(foo.files[i]).toBe(0xFFFF - i);
}
function ServiceImpl() {
@@ -103,8 +93,8 @@ define([
ServiceImpl.prototype.frobinate = function(foo, baz, port) {
checkFoo(foo);
- gtest.expectTrue(baz, "baz is " + baz);
- gtest.expectEqual(port, 10, "port is " + port);
+ expect(baz).toBeTruthy();
+ expect(port).toBe(10);
global.result = "PASS";
};
generated by cgit v1.1 at 2024-10-23 01:10:04 +0000