You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Andrew de Andrade (JIRA)" <ji...@apache.org> on 2015/02/02 19:57:34 UTC
[jira] [Commented] (THRIFT-2969) nodejs: DRY up library tests
[ https://issues.apache.org/jira/browse/THRIFT-2969?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14301663#comment-14301663 ]
Andrew de Andrade commented on THRIFT-2969:
-------------------------------------------
Also, you may want to consider applying patch 0008 after 0001 but before 0002. The reason for this is that patch 0001 changed the interface of lib/nodejs/test/client.js and lib/nodejs/test/server.js. Those changes in 0001 shouldn't break the cross tests because, in 0001, I use 'tcp' as the default server type if the {{--type}} flag isn't set and all the cross tests use only the basic tcp server/client. That said, patch 0008 adds features to the main test/test.sh file so that cross tests can be tested across all the server types (tcp, multiplex, http and websocket), but only tcp is enabled for now.
> nodejs: DRY up library tests
> ----------------------------
>
> Key: THRIFT-2969
> URL: https://issues.apache.org/jira/browse/THRIFT-2969
> Project: Thrift
> Issue Type: Improvement
> Components: Node.js - Library
> Reporter: Andrew de Andrade
> Attachments: 0001-Consolidated-nodejs-server-client-test-code.patch, 0002-DRY-up-nodejs-test-drivers.patch, 0003-DRY-up-nodejs-test-handlers.patch, 0004-DRY-nodejs-handlers-more.patch, 0005-Clean-up-test-driver-code-some-more.patch, 0006-Dry-up-nodejs-testAll.sh.-Cover-all-configurations.patch, 0007-nodejs-include-websockets-in-for-each-testAll.sh-loo.patch, 0008-Add-nodejs-type-to-the-cross-tests.-tcp-only-for-now.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)