Permalink
Browse files

Merge pull request #50 from tonyskn/DescribeBug

Fix bug causing client.describe() to yield incomplete results
  • Loading branch information...
2 parents 1479c53 + ac32582 commit 1f70a949f832515ce32b97b67d51153a106092a4 @milewise milewise committed May 8, 2012
Showing with 11 additions and 2 deletions.
  1. +0 −1 lib/wsdl.js
  2. +11 −1 test/server-test.js
View
1 lib/wsdl.js
@@ -508,7 +508,6 @@ PortTypeElement.prototype.description = function(definitions) {
return methods;
}
OperationElement.prototype.description = function(definitions) {
- return;
var inputDesc = this.input.description(definitions);
var outputDesc = this.output.description(definitions);
return {
View
12 test/server-test.js
@@ -71,6 +71,16 @@ module.exports = {
})
},
+ 'should return complete client description': function(done) {
+ soap.createClient('http://localhost:15099/stockquote?wsdl', function(err, client) {
+ assert.ok(!err);
+ var description = client.describe(),
+ expected = { input: { tickerSymbol: "string" }, output:{ price: "float" } };
+ assert.deepEqual(expected , description.StockQuoteService.StockQuotePort.GetLastTradePrice );
+ done();
+ });
+ },
+
'should return correct results': function(done) {
soap.createClient('http://localhost:15099/stockquote?wsdl', function(err, client) {
assert.ok(!err);
@@ -84,4 +94,4 @@ module.exports = {
},
'WSDL Parser (strict)': wsdlStrictTests,
'WSDL Parser (non-strict)': wsdlNonStrictTests
-}
+}

0 comments on commit 1f70a94

Please sign in to comment.