search

ARMmbed / ci-test-shield

mbed CI Test Shield
Apache License 2.0
9 stars 36 forks source link

tests-api-businout : increase TIMEOUT #68

Closed jeromecoutant closed 7 years ago

jeromecoutant commented 7 years ago

Test with NUCLEO_L476RG-GCC_ARM : 30s is not long enough, it seems that 40s is OK

BlackstoneEngineering commented 7 years ago

@jeromecoutant any idea why 30s is not long enough on that target? The businout test should take almost no time at all.

jeromecoutant commented 7 years ago

Hi Check the code, there are many wait 1s...

ex:

volatile int x = 0x00;
while(x < 0xF){
    x = (x<<1) +1;
    ...
    wait(1);
    ...
    wait(1);
    ...
}

x = 0x00;
while(x < 0xF){
    x = (x<<1) +1;
    ...
    wait(1);
    ...
    wait(1);
    ...
}
jeromecoutant commented 7 years ago

Replacing these wait(1) by wait_ms(100) is also OK for me

jeromecoutant commented 7 years ago

Should I update PR with wait_ms(100) ?

BlackstoneEngineering commented 7 years ago

@jeromecoutant what boards have you tested this change on?

@mray19027 we should probably run this through the test farm to see if changing form 1s to .1s causes any other boards to fail.

mray190 commented 7 years ago

@BlackstoneEngineering We currently do not support running ci-test-shield tests on the CI. But that has been planned for a while and will work with @studavekar to see if we can get that integrated ASAP