Deserialization autogenerated client and related tests for basic data ty... #1164
Conversation
done(); | ||
}); | ||
|
||
test('GetBoolNullable with a null elemet should deserialize into null', function (done) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
refer to bug #1165
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spelling - elemet
Great job, Amar! Some issues noted. We should consider whether to move the visual studio generation architecture here in the long term - we should really be consuming the spec and generating whenever there are changes made to hydra or the node library. |
suite('array-datatype-deserialization-tests', function () { | ||
var baseUri = 'http://helloworld'; | ||
var subscriptionId = "c9cbd920-c00c-427c-852b-8aaf38badaeb"; | ||
var pemfile = process.env.AZURE_CERTIFICATE_PEM_FILE; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really need an actual pem file? None of these go out over the wire, can we just mock/fake this?
test('GetPrimitiveSequence with valid primitive values should deserialize properly', function (done) { | ||
nock("http://helloworld") | ||
.get("/GetPrimitiveSequence") | ||
.reply(200, "<LIST><STRING>Hello world</STRING><STRING>Hurray!!</STRING></LIST>"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use lowercase/camelcase here, not all caps.
@amarzavery coudl you apply the review comments so that we can merge this in ? Thanks, |
@amarzavery : ping ? |
...pes