15140
35%
master: 57%

Ran 09 Feb 2018 02:54PM UTC

Jobs 1

Files 481

Run time 19s

Badge

Embed ▾

pending completion

Build # 15140

Build Type

push

travis-ci

Committed by

adbridge

Commit Message

Correct test `socket_sigio`

A call to

    `TCPSocket::recv(void *data, nsapi_size_t size)`

returns, following the mbed documentation, the number of received bytes on
success, and a negative error code on failure.

So in case of success, the return value depends on both the value of parameter
`size` but also on the amount of data already available. This means, that the
value returned can be lower than or equal to the `size` of the `data` buffer
passed as argument to the call.

Therefore, in the cases of `test_tcp_hello_world()` & `find_substring()`
(i.e. test `socket_sigio`), the calls to `TCPSocket::recv()` might return from
one byte up to `sizeof(buffer) - 1` (i.e. 511) bytes for each single call,
while the tests expect to receive the whole response string with a single call.

This commit applies a fix to this situation by implementing a receive loop
which exits once there is no data anymore available to be read from the socket.

Run Details

25583 of 76136 relevant lines covered (33.6%)

0.34 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
2	15140.2 (NAME=tools)	09 Feb 2018 02:54PM UTC	0	33.6	Travis Job 15140.2

Source Files on build 15140

Detailed source file information is not available for this build.

ARMmbed / mbed-os / 15140
35%
master: 57%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 15140

ARMmbed / mbed-os / 15140 35% master: 57%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 15140

ARMmbed / mbed-os / 15140
35%
master: 57%

README BADGES
x