Hi Chamila
Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>:
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.
yes, there seems to be an issue with multiException.
I've also seen that we still miss nodejs-nodejs tests, I've added them.
https://git-wip-us.apache.org/repos/asf?p=thrift.git;a=commitdiff;h=
d9b331d2662ff8bbd14f04800ce5de56b21ba01a
- 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?
yes, Please provide patches also for combinations that are broken.
We need full coverage of combinations and can fix the bugs in a second
step.
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?
yes, this was added earlier this year.
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?
Javascript test uses the javaserver from lib/js/test/src and phantomjs
it's another story.
thanks
-roger
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 <
ro...@bufferoverflow.ch>wrote:
Hi Chamila
Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>:
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 <
cdwijayarat...@gmail.com> wrote:
Patch attached at https://issues.apache.org/
jira/browse/THRIFT-2534.
On Fri, May 16, 2014 at 9:07 PM, Chamila Wijayarathna <
cdwijayarat...@gmail.com> wrote:
Sorry, status.html is attached here.
On Fri, May 16, 2014 at 9:06 PM, Chamila Wijayarathna <
cdwijayarat...@gmail.com> 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 <
cdwijayarat...@gmail.com> 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 <
ro...@bufferoverflow.ch
>wrote:
yes, just creat a sub Task for THRIFT-847
thanks
roger
Quoting Chamila Wijayarathna <cdwijayarat...@gmail.com>:
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 <
ro...@bufferoverflow.ch
>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 <cdwijayarat...@gmail.com>:
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 4edac7ff085e4bd28096fef3cf8234
e4991544ff)"
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.