Fix for "Uncaught TypeError: Cannot read property '0' of undefined".

[ankush-garg]

Hello,

A call to client.describe() fails for 'https://webservices.netsuite.com/wsdl/v2013_2_0/netsuite.wsdl' wsdl. The following mocha test can be used to reproduce the issue:

var soap = require('..');
var assert = require('assert');

describe('NetSuite Service Manager', function() {

it('should describe the wsdl', function(done) {
this.timeout(0);
soap.createClient('https://webservices.netsuite.com/wsdl/v2013_2_0/netsuite.wsdl', function(err, client) {
assert.ok(client);
console.log(client.describe());
done();
});
});

});

I made the required changes to fix this issue in wsdl.js. Please review.

Thanks,
Ankush

[jsdevel]

@ankush-garg Can you include the wsdl you're using under test/wsdl... and add a test case to test/server-test.js?

@aheckmann we really need to get #234 merged in to keep whitespace etc. consistent in our tests.

[ankush-garg]

@jsdevel The wsdl I am using has about 46 xsd files associated with it. Do you want to me drop all of them in the test/wsdl folder? Its given here - https://system.netsuite.com/download/WSDL_v2013_2_0.zip Just want to check before dropping the stuff there as it might create too much noise in the folder. Or, I can create another sub folder for this wsdl. Please let me know what works best.

[jsdevel]

@ankush-garg ideally you could find what the issue is, then make a simple wsdl that recreates the issue. I'm a fan of keeping the commits small wherever possible.

[ankush-garg]

@jsdevel Good idea. I updated the Wsdl and committed the code. Please review.

[jsdevel]

@ankush-garg looks great! Thanks for adding this.