Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: test/ConsoleJsObject.cpp

Issue 10213003: Make JsEngine::Evaluate() return a wrapper for v8::Value to accessdifferent variable types easily (Closed)
Patch Set: Created April 12, 2013, 2:55 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
1 #include <AdblockPlus.h> 1 #include <AdblockPlus.h>
2 #include <gtest/gtest.h> 2 #include <gtest/gtest.h>
3 3
4 class MockErrorCallback : public AdblockPlus::ErrorCallback 4 class MockErrorCallback : public AdblockPlus::ErrorCallback
5 { 5 {
6 public: 6 public:
7 std::string lastMessage; 7 std::string lastMessage;
8 8
9 void operator()(const std::string& message) 9 void operator()(const std::string& message)
10 { 10 {
11 lastMessage = message; 11 lastMessage = message;
12 } 12 }
13 }; 13 };
14 14
15 TEST(ConsoleJsObjectTest, ErrorInvokesErrorCallback) 15 TEST(ConsoleJsObjectTest, ErrorInvokesErrorCallback)
16 { 16 {
17 MockErrorCallback errorCallback; 17 MockErrorCallback errorCallback;
18 AdblockPlus::JsEngine jsEngine(0, 0, &errorCallback); 18 AdblockPlus::JsEngine jsEngine(0, 0, &errorCallback);
19 jsEngine.Evaluate("console.error('foo')"); 19 jsEngine.Evaluate("console.error('foo')");
20 ASSERT_EQ("foo", errorCallback.lastMessage); 20 ASSERT_EQ("foo", errorCallback.lastMessage);
21 } 21 }
22 22
23 TEST(ConsoleJsObjectTest, ErrorWithMultipleArguments) 23 TEST(ConsoleJsObjectTest, ErrorWithMultipleArguments)
24 { 24 {
25 MockErrorCallback errorCallback; 25 MockErrorCallback errorCallback;
26 AdblockPlus::JsEngine jsEngine(0, &errorCallback); 26 AdblockPlus::JsEngine jsEngine(0, 0, &errorCallback);
27 jsEngine.Evaluate("console.error('foo', 'bar')"); 27 jsEngine.Evaluate("console.error('foo', 'bar')");
28 ASSERT_EQ("foobar", errorCallback.lastMessage); 28 ASSERT_EQ("foobar", errorCallback.lastMessage);
29 } 29 }
30 30
31 TEST(ConsoleJsObjectTest, TraceDoesNothing) 31 TEST(ConsoleJsObjectTest, TraceDoesNothing)
32 { 32 {
33 AdblockPlus::JsEngine jsEngine(0, 0, 0); 33 AdblockPlus::JsEngine jsEngine(0, 0, 0);
34 jsEngine.Evaluate("console.trace()"); 34 jsEngine.Evaluate("console.trace()");
35 } 35 }
OLDNEW
« include/AdblockPlus/JsValue.h ('K') | « src/JsValue.cpp ('k') | test/GlobalJsObject.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld