Author: jake
Date: Wed Jul 28 22:31:12 2010
New Revision: 980253
URL: http://svn.apache.org/viewvc?rev=980253&view=rev
Log:
added test case and fix for records contained in arrays
Modified:
incubator/thrift/trunk/lib/js/test/src/test/TestHandler.java
incubator/thrift/trunk/lib/js/test/test.html
incubator/thrift/trunk/lib/js/thrift.js
Modified: incubator/thrift/trunk/lib/js/test/src/test/TestHandler.java
URL:
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/js/test/src/test/TestHandler.java?rev=980253&r1=980252&r2=980253&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/js/test/src/test/TestHandler.java (original)
+++ incubator/thrift/trunk/lib/js/test/src/test/TestHandler.java Wed Jul 28
22:31:12 2010
@@ -48,7 +48,7 @@ public class TestHandler implements Thri
}
public void testException(String arg) throws Xception, TException {
- throw new Xception(1,"server test exception");
+ throw new Xception(1,arg);
}
public int testI32(int thing) throws TException {
Modified: incubator/thrift/trunk/lib/js/test/test.html
URL:
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/js/test/test.html?rev=980253&r1=980252&r2=980253&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/js/test/test.html (original)
+++ incubator/thrift/trunk/lib/js/test/test.html Wed Jul 28 22:31:12 2010
@@ -1,95 +1,142 @@
-<!--
- Licensed to the Apache Software Foundation (ASF) under one
- or more contributor license agreements. See the NOTICE file
- distributed with this work for additional information
- regarding copyright ownership. The ASF licenses this file
- to you under the Apache License, Version 2.0 (the
- "License"); you may not use this file except in compliance
- with the License. You may obtain a copy of the License at
-
- http://www.apache.org/licenses/LICENSE-2.0
-
- Unless required by applicable law or agreed to in writing,
- software distributed under the License is distributed on an
- "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- KIND, either express or implied. See the License for the
- specific language governing permissions and limitations
- under the License.
--->
-
-<html>
+<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd";>
+<html xmlns="http://www.w3.org/1999/xhtml"; xml:lang="en" lang="en">
<head>
- <title>Thrift Javascript Bindings - Example</title>
-
- <script src="/thrift.js" type="text/javascript"></script>
- <script src="gen-js/ThriftTest_types.js" type="text/javascript"></script>
- <script src="gen-js/ThriftTest.js" type="text/javascript"></script>
+ <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+ <title>Thrift Javascript Bindings: Unit Test</title>
- <!-- for async example -->
- <script type="text/javascript"
src="http://ajax.googleapis.com/ajax/libs/jquery/1.3.2/jquery.min.js";></script>
+ <script src="/thrift.js" type="text/javascript"
charset="utf-8"></script>
+ <script src="gen-js/ThriftTest_types.js" type="text/javascript"
charset="utf-8"></script>
+ <script src="gen-js/ThriftTest.js" type="text/javascript"
charset="utf-8"></script>
+
+ <!-- jQuery -->
+ <script type="text/javascript"
src="http://ajax.googleapis.com/ajax/libs/jquery/1.4.2/jquery.min.js";
charset="utf-8"></script>
+
+ <!-- QUnit Test framework-->
+ <script type="text/javascript"
src="http://github.com/jquery/qunit/raw/master/qunit/qunit.js";
charset="utf-8"></script>
+ <link rel="stylesheet"
href="http://github.com/jquery/qunit/raw/master/qunit/qunit.css";
type="text/css" media="screen" />
+
+ <script type="text/javascript" charset="utf-8">
+ //<![CDATA[
+ $(document).ready(function(){
+ var transport = new Thrift.Transport("/service")
+ var protocol = new Thrift.Protocol(transport)
+ var client = new ThriftTest.ThriftTestClient(protocol)
+
+
+ module("Base Types");
+
+ test("String", function() {
+ equals(client.testString("works"), "works");
+ });
+ test("String UTF-8", function() {
+ var languagesUtf8 = "Afrikaans, Alemannisch, Aragonés, اÙعربÙØ©,
Ù
صرÙ, Asturianu, Aymar aru, AzÉrbaycan, ÐаÑҡоÑÑ, Boarisch,
ŽemaitÄÅ¡ka, ÐелаÑÑÑкаÑ, ÐелаÑÑÑкаÑ
(ÑаÑаÑкевÑÑа), ÐÑлгаÑÑки, Bamanankan, বাà¦à¦²à¦¾,
Brezhoneg, Bosanski, Català , Mìng-dÄ̤ng-ngá¹³Ì, ÐоÑ
Ñийн, Cebuano,
á£á³á©, Äesky, СловѣÌнÑÑÐºÑ / â°â°â°â°â°¡â°â° â°â°â°,
ЧÓваÑла, Cymraeg, Dansk, Zazaki, ÞÞ¨ÞÞ¬ÞÞ¨ÞÞ¦ÞÞ°, Îλληνικά,
Emilià n e rumagnòl, English, Esperanto, Español, Eesti, Euskara, ÙارسÛ,
Suomi, Võro, Føroyskt, Français, Arpetan, Furlan, Frysk, Gaeilge, è´èª,
GÃ id
hlig, Galego, Avañe'ẽ, àªà«àªàª°àª¾àª¤à«, Gaelg, ×¢×ר×ת,
हिनà¥à¤¦à¥, Fiji Hindi, Hrvatski, Kreyòl ayisyen, Magyar,
ÕÕ¡ÕµÕ¥ÖÕ¥Õ¶, Interlingua, Bahasa Indonesia, Ilokano, Ido, Ãslenska,
Italiano, æ¥æ¬èª, Lojban, Basa Jawa, á¥áá áá£áá, Kongo,
Kalaallisut, à²à²¨à³à²¨à²¡, íêµì´, ÐÑаÑаÑай-ÐалкÑаÑ,
Ripoarisch, Kurdî, Ðоми, Kernewek, ÐÑÑгÑзÑа, Latina, Ladino,
Lëtzebuergesch, Limburgs, Lingála, ລາວ, Lietuvių, Latviešu, Basa
Banyumasan, Malagasy, ÐакедонÑки, മലയാളà´,
मराठà¥, Bahasa Melayu, Ù
ازÙرÙÙÛ, Nnapulitano, Nedersaksisch,
नà¥à¤ªà¤¾à¤² à¤à¤¾à¤·à¤¾, Nederlands, âªNors
k (nynorsk)â¬, âªNorsk (bokmÃ¥l)â¬, Nouormand, Diné bizaad, Occitan,
ÐÑонаÑ, Papiamentu, Deitsch, Norfuk / Pitkern, Polski, Ù¾ÙجابÛ,
Ù¾ÚتÙ, Português, Runa Simi, Rumantsch, Romani, RomânÄ, Ð ÑÑÑкий,
СаÑ
а ÑÑла, Sardu, Sicilianu, Scots, Sámegiella, Simple English,
SlovenÄina, SlovenÅ¡Äina, СÑпÑки / Srpski, Seeltersk, Svenska,
Kiswahili, தமிழà¯, à°¤à±à°²à±à°à±, Тоҷикӣ, à¹à¸à¸¢,
Türkmençe, Tagalog, Türkçe, ТаÑаÑÑа/Tatarça, УкÑаÑнÑÑка,
اردÙ, Tiếng Viá»t, Volapük, Walon, Winaray, å´è¯, isiXhosa,
××Ö´××ש, Yorùbá, Zeêuws, ä¸æ, Bân-lâm-gú, ç²µèª";
+ equals(client.testString(languagesUtf8), languagesUtf8);
+ });
+ test("Double", function() {
+ equals(client.testDouble(3.14), 3.14);
+ });
+ test("Byte", function() {
+ equals(client.testByte(0x01), 0x01);
+ });
+ test("I32", function() {
+ equals(client.testI32(Math.pow(2,30)), Math.pow(2,30));
+ });
+ test("I64", function() {
+ equals(client.testI64(Math.pow(2,60)), Math.pow(2,60));
+ });
+
+
+ module("Structured Types");
+
+ var insanity = new ThriftTest.Insanity()
+ insanity.userMap[ThriftTest.Numberz.ONE] = 1
+ insanity.userMap[ThriftTest.Numberz.TWO] = 2
+
+ var xtr = new ThriftTest.Xtruct()
+ xtr.string_thing = 'worked'
+ insanity.xtructs.push(xtr)
+
+ var xtr2= new ThriftTest.Xtruct2()
+ xtr2.struct_thing = xtr
+
+ var list = [1,2,3]
+
+ test("Struct", function() {
+ equals(client.testStruct(xtr).string_thing, "worked");
+ });
+ test("Nest", function() {
+ equals(client.testNest(xtr2).struct_thing.string_thing, "worked");
+ });
+ test("Map", function() {
+ equals(client.testMap(insanity.userMap)[ThriftTest.Numberz.ONE], 1);
+ });
+ test("List", function() {
+ same(client.testList(list), list);
+ });
+ test("Set", function() {
+ same(client.testSet(list), list);
+ });
+ test("Enum", function() {
+ equals(client.testEnum(ThriftTest.Numberz.ONE), ThriftTest.Numberz.ONE);
+ });
+
+
+ module("Exception");
+
+ test("Xception", function() {
+ expect(2);
+ try{
+ client.testException("Xception");
+ }catch(e){
+ equals(e.errorCode, 1);
+ equals(e.message, "Xception");
+ }
+ });
+
+ test("ApplicationException", function() {
+ expect(1);
+ try{
+ client.testException("ApplicationException");
+ }catch(e){
+ equals(e.message, "ApplicationException");
+ }
+ });
+
+ test("no Exception", function() {
+ expect(1);
+ try{
+ client.testException("no Exception");
+ }catch(e){
+ equals(e.message, "no Exception");
+ }
+ });
+
+
+ module("Insanity");
+
+ test("testInsanity", function() {
+ var res = client.testInsanity(insanity);
+ equals(res["1"]["1"].xtructs[0].string_thing, "worked");
+ });
+
+ });
+ //]]>
+ </script>
</head>
-<body id="body">
-
-<script language="javascript">
-
- //create client
- var transport = new Thrift.Transport("/service")
- var protocol = new Thrift.Protocol(transport)
- var client = new ThriftTest.ThriftTestClient(protocol)
+<body>
- //create insanity obj
- var insanity = new ThriftTest.Insanity()
- insanity.userMap[ThriftTest.Numberz.ONE] = 1
- insanity.userMap[ThriftTest.Numberz.TWO] = 2
-
- var xtr = new ThriftTest.Xtruct()
- xtr.string_thing = 'worked'
- insanity.xtructs.push(xtr)
-
- var xtr2= new ThriftTest.Xtruct2()
- xtr2.struct_thing = xtr
-
- var list = [1,2,3]
-
- //run tests synchronously
-
- document.write("<h2><u>Thrift Javascript Bindings</u></h2>")
- document.write("<h2>Synchronous Example</h2>")
- document.write("client.testString() => "+(client.testString("works") ==
"works")+"<br/>")
- document.write("client.testString(utf-8) => "+(client.testString("ae")
== "ae")+"<br/>")
- document.write("client.testDouble() => "+(client.testDouble(3.14) ==
3.14)+"<br/>")
- document.write("client.testByte() => "+(client.testByte(0x01) ==
0x01)+"<br/>")
- document.write("client.testI32() => "+(client.testI32(Math.pow(2,30)) ==
Math.pow(2,30))+"<br/>")
- document.write("client.testI64() => "+(client.testI64(Math.pow(2,60)) ==
Math.pow(2,60))+"<br/>")
- document.write("client.testStruct() =>
"+(client.testStruct(xtr).string_thing == "worked")+"<br/>")
- document.write("client.testNest() =>
"+(client.testNest(xtr2).struct_thing.string_thing == "worked")+"<br/>")
- document.write("client.testMap() =>
"+(client.testMap(insanity.userMap)[ThriftTest.Numberz.ONE] == 1)+"<br/>")
- document.write("client.testList() => "+(client.testList(list).length == 3
&& client.testList(list)[1] == 2)+"<br/>")
- document.write("client.testSet() => "+(client.testSet(list).length ==
3)+"<br/>")
- document.write("client.testEnum() =>
"+(client.testEnum(ThriftTest.Numberz.ONE) == ThriftTest.Numberz.ONE)+"<br/>")
-
- document.write("client.testException() => ")
- try{
- client.testException("go")
- document.write("false<br/>")
- }catch(e){
- document.write("true<br/>")
- }
-
- document.write("client.testInsanity() => ")
- var res = client.testInsanity(insanity)
-
- document.write((res["1"]["1"].xtructs[0].string_thing == "worked")+"<br/>")
+ <script type="text/javascript" charset="utf-8">
+ //<![CDATA[
//////////////////////////////////
//Run same tests asynchronously
-
+/*
var transport = new Thrift.Transport()
var protocol = new Thrift.Protocol(transport)
var client = new ThriftTest.ThriftTestClient(protocol)
- document.write("<h2>Asynchronous Example</h2>")
+ document.write("<h2>Asynchronous Example<\/h2>")
jQuery.ajax({
url: "/service",
data: client.send_testI32(Math.pow(2,30)),
@@ -125,10 +172,18 @@
}
})
+*/
-
-
-</script>
-
+ //]]>
+ </script>
+ <h1 id="qunit-header">Thrift Javascript Bindings: Unit Test (<a
href="https://svn.apache.org/repos/asf/incubator/thrift/trunk/test/ThriftTest.thrift";>ThriftTest.thrift</a>)</h1>
+ <h2 id="qunit-banner"></h2>
+ <h2 id="qunit-userAgent"></h2>
+ <ol id="qunit-tests"><li><!-- get valid xhtml strict--></li></ol>
+ <p>
+ <a href="http://validator.w3.org/check/referer";><img
+ src="http://www.w3.org/Icons/valid-xhtml10";
+ alt="Valid XHTML 1.0!" height="31" width="88" /></a>
+ </p>
</body>
</html>
Modified: incubator/thrift/trunk/lib/js/thrift.js
URL:
http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/js/thrift.js?rev=980253&r1=980252&r2=980253&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/js/thrift.js (original)
+++ incubator/thrift/trunk/lib/js/thrift.js Wed Jul 28 22:31:12 2010
@@ -54,7 +54,7 @@ Thrift.TException.prototype = {
}
-Thrift.TApplicationException = {
+Thrift.TApplicationExceptionType = {
"UNKNOWN" : 0,
"UNKNOWN_METHOD" : 1,
"INVALID_MESSAGE_TYPE" : 2,
@@ -401,7 +401,9 @@ Thrift.Protocol.prototype = {
var p = this.tpos.pop()
while( this.tstack.length > p+1 ){
- this.tstack[p].push(this.tstack.pop())
+ var tmpVal = this.tstack[p+1]
+ this.tstack.splice(p+1, 1)
+ this.tstack[p].push(tmpVal)
}
this.tstack[p] = '['+this.tstack[p].join(",")+']';
@@ -416,7 +418,9 @@ Thrift.Protocol.prototype = {
var p = this.tpos.pop()
while( this.tstack.length > p+1 ){
- this.tstack[p].push(this.tstack.pop())
+ var tmpVal = this.tstack[p+1]
+ this.tstack.splice(p+1, 1)
+ this.tstack[p].push(tmpVal)
}
this.tstack[p] = '['+this.tstack[p].join(",")+']';
@@ -486,13 +490,19 @@ Thrift.Protocol.prototype = {
},
readStructBegin : function(name){
- var r = {};
- r["fname"] = '';
-
- return r;
+ var r = {}
+ r["fname"] = ''
+
+ //incase this is an array of structs
+ if(this.rstack[this.rstack.length-1] instanceof Array)
+ this.rstack.push(this.rstack[this.rstack.length-1].shift())
+
+ return r
},
readStructEnd : function(){
+ if(this.rstack[this.rstack.length-2] instanceof Array)
+ this.rstack.pop()
},
readFieldBegin : function(){
@@ -578,8 +588,7 @@ Thrift.Protocol.prototype = {
this.rpos.push(this.rstack.length);
this.rstack.push(list)
-
-
+
return r;
},
@@ -616,19 +625,24 @@ Thrift.Protocol.prototype = {
},
- readI32 : function(){
- var f = this.rstack[this.rstack.length-1]
+ readI32 : function(f){
+ if(f == undefined)
+ f = this.rstack[this.rstack.length-1]
+
var r = {}
if(f instanceof Array){
- r["value"] = f.pop()
-
+ if(f.length == 0)
+ r["value"] = undefined
+ else
+ r["value"] = f.shift()
+
}else if(f instanceof Object){
for(var i in f){
if(i == null) continue
this.rstack.push(f[i])
delete f[i]
-
+
r["value"] = i
break
}
@@ -636,7 +650,6 @@ Thrift.Protocol.prototype = {
r["value"] = f
}
-
return r
},
