Hi Roger,
Upto now I did following on nodejs tests we discussed earlier.
- Ran nodejs-nodejs, nodejs-java, java-nodejs tests manually
- Ran nodejs-cpp, cpp-nodejs, nodejs-nodejs tests manually with compact
protocol
I came up with following observations.
- We can add nodejs-nodejs, nodejs-java, java-nodejs tests to test.sh in
the same way we have implemented nodejs-cpp, cpp-nodejs tests. I only
checked them manually, I couldn't ran them through test.sh since I couldn't
figure out how to make 'node' command work on test.sh. Also nodejs client
exists after 3 seconds, so I couldn't check if all tests give success or
failure, is there any way to extend this limit? Node server seems to fail
with multiException[1] tests in existing cpp tests, so I expect same output
with other tests which use nodejs server. But I couldn't check it since I'm
having my node client exists after 3 seconds.
- Compact protocol seems to work ok with nodejs-nodejs, but when cpp
involves it gives some errors. So we need to look at what are the issues
there.
I would be very grateful if you can provide any help with problems I have
mentioned above.
Should I create patch with this progress or shall we create patch after we
solve these problems?
I also tried to run tests with 'ssl', since those options are available in
node client and server at lib/nodejs/test, but I came up with errors there.
Do we have 'ssl' for nodejs?
I also looked at existing JavaScript test on cross language test suite, but
it seems to be very different from others. Can you give me some information
on how it works?
1. testMultiException(success, test 3)
events.js:72
throw er; // Unhandled 'error' event
^
Error: read ECONNRESET
at errnoException (net.js:901:11)
at TCP.onread (net.js:556:19)
Thank You!
On Thu, May 22, 2014 at 12:40 AM, Roger Meier <[email protected]>wrote:
> Hi Chamila
>
>
> Quoting Chamila Wijayarathna <[email protected]>:
>
> Hi Roger,
>>
>> I think now we have a good automated test reporting? Do I have do to any
>> modification on this?
>>
> I'm fine with this for the moment.
> Everybody is now able to browse the tested combinations and results.
>
>
> If not what should I do next? Do you have any suggestions on tasks to
>> implement next.
>> Since in cross language test suite, it only tests nodejs with cpp, I think
>> nodejs-nodejs, nodejs-java and java-nodejs tests are suitable as next
>> task.
>>
> yes, this makes sense. Do as many combinations as possible and add
> language by language.
>
> please check also that TestClient and TestServers follow the parameter
> signatures
> and return value definitions: http://thrift.apache.org/test/
>
> What do you think?
>>
> yes proceed with this.
>
> -roger
>
>
> Thank You!
>>
>>
>> On Fri, May 16, 2014 at 9:17 PM, Chamila Wijayarathna <
>> [email protected]> wrote:
>>
>> Patch attached at https://issues.apache.org/jira/browse/THRIFT-2534.
>>>
>>>
>>> On Fri, May 16, 2014 at 9:07 PM, Chamila Wijayarathna <
>>> [email protected]> wrote:
>>>
>>> Sorry, status.html is attached here.
>>>>
>>>>
>>>> On Fri, May 16, 2014 at 9:06 PM, Chamila Wijayarathna <
>>>> [email protected]> wrote:
>>>>
>>>> Hi Roger,
>>>>> I generated status.html I have attached with this. Please let me know
>>>>> what else should be added here.
>>>>> Thank You!
>>>>>
>>>>>
>>>>> On Fri, May 16, 2014 at 1:19 PM, Chamila Wijayarathna <
>>>>> [email protected]> wrote:
>>>>>
>>>>> Created a sub task and added patch.
>>>>>> https://issues.apache.org/jira/browse/THRIFT-2534
>>>>>>
>>>>>>
>>>>>> On Fri, May 16, 2014 at 1:16 PM, Roger Meier <[email protected]
>>>>>> >wrote:
>>>>>>
>>>>>> yes, just creat a sub Task for THRIFT-847
>>>>>>>
>>>>>>>
>>>>>>> thanks
>>>>>>> roger
>>>>>>>
>>>>>>> Quoting Chamila Wijayarathna <[email protected]>:
>>>>>>>
>>>>>>> Hi Roger,
>>>>>>>
>>>>>>>> I'll start working on that.
>>>>>>>> Shall I create a Jira for this and add patch for current stuff
>>>>>>>> there?
>>>>>>>> Than You.
>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> On Thu, May 15, 2014 at 3:52 AM, Roger Meier <
>>>>>>>> [email protected]
>>>>>>>> >wrote:
>>>>>>>>
>>>>>>>> Hi Chamila
>>>>>>>>
>>>>>>>>>
>>>>>>>>> generating this as an additional result out of test.sh is good
>>>>>>>>> first
>>>>>>>>> step.
>>>>>>>>>
>>>>>>>>> attached is a template for the second step:
>>>>>>>>> git apply test_status.diff
>>>>>>>>>
>>>>>>>>> it uses jquery datatables, a simple and useful table control
>>>>>>>>> we can add filter functions to the table, etc.
>>>>>>>>>
>>>>>>>>> you can generate status.html
>>>>>>>>> or generate status.json and load the table within status.html from
>>>>>>>>> status.json
>>>>>>>>> (e.g. use https://datatables.net/manual/data#Objects as
>>>>>>>>> initialization
>>>>>>>>> method)
>>>>>>>>>
>>>>>>>>> thanks
>>>>>>>>> roger
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Quoting Chamila Wijayarathna <[email protected]>:
>>>>>>>>>
>>>>>>>>> Hi Roger,
>>>>>>>>>
>>>>>>>>> As we discussed earlier I have changed test.sh to write results of
>>>>>>>>>> cross
>>>>>>>>>> language test run to a status.md file. I have attached a sample
>>>>>>>>>> status.mdgenerated.
>>>>>>>>>> I observed that in commit "THRIFT-847 Test Framework harmonization
>>>>>>>>>> across
>>>>>>>>>> all languages (commit 4edac7ff085e4bd28096fef3cf8234e4991544ff)"
>>>>>>>>>> you have
>>>>>>>>>> wrote results of failed test to an error.log file. Are we planning
>>>>>>>>>> to keep
>>>>>>>>>> both status.md and error.log or only one from them?
>>>>>>>>>> What do you think about the format of status.md? I am only
>>>>>>>>>> writing
>>>>>>>>>> success/failure, no any description in failure. Is there any thing
>>>>>>>>>> else
>>>>>>>>>> needed to be added or is this enough for now?
>>>>>>>>>> Thank You!
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> *Chamila Dilshan Wijayarathna,*
>>>>>>>>>>
>>>>>>>>>> SMIEEE, SMIESL,
>>>>>>>>>> Undergraduate,
>>>>>>>>>> Department of Computer Science and Engineering,
>>>>>>>>>> University of Moratuwa.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>> --
>>>>>>>> *Chamila Dilshan Wijayarathna,*
>>>>>>>> SMIEEE, SMIESL,
>>>>>>>> Undergraduate,
>>>>>>>> Department of Computer Science and Engineering,
>>>>>>>> University of Moratuwa.
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>>>>> --
>>>>>> *Chamila Dilshan Wijayarathna,*
>>>>>>
>>>>>> SMIEEE, SMIESL,
>>>>>> Undergraduate,
>>>>>> Department of Computer Science and Engineering,
>>>>>> University of Moratuwa.
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> *Chamila Dilshan Wijayarathna,*
>>>>> SMIEEE, SMIESL,
>>>>> Undergraduate,
>>>>> Department of Computer Science and Engineering,
>>>>> University of Moratuwa.
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>> *Chamila Dilshan Wijayarathna,*
>>>> SMIEEE, SMIESL,
>>>> Undergraduate,
>>>> Department of Computer Science and Engineering,
>>>> University of Moratuwa.
>>>>
>>>>
>>>
>>>
>>> --
>>> *Chamila Dilshan Wijayarathna,*
>>> SMIEEE, SMIESL,
>>> Undergraduate,
>>> Department of Computer Science and Engineering,
>>> University of Moratuwa.
>>>
>>>
>>
>>
>> --
>> *Chamila Dilshan Wijayarathna,*
>> SMIEEE, SMIESL,
>> Undergraduate,
>> Department of Computer Science and Engineering,
>> University of Moratuwa.
>>
>
>
--
*Chamila Dilshan Wijayarathna,*
SMIEEE, SMIESL,
Undergraduate,
Department of Computer Science and Engineering,
University of Moratuwa.
