Unittesting: Update mutex stub

[lorjala]

Description

Fix linker error by updating rtos::Mutex::lock() in UNITTESTS/stubs/Mutex_stub.cpp.

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Breaking change

[0xc0170]

Fix looks good - to update to a new method but this should not cause a linker error (overloaded method). Please explain

[lorjala]

@0xc0170 Since unit tests use UNITTESTS/stubs/Mutex_stub.cpp to implement rtos/Mutex.h and rtos::Mutex::lock() was not implemented, a test failed with:

Undefined symbols for architecture x86_64:

  "rtos::Mutex::lock()", referenced from:

      InternetSocket::open(NetworkStack*)       in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      InternetSocket::close()      in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      InternetSocket::bind(SocketAddress const&)  in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      InternetSocket::set_timeout(int)    in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      InternetSocket::setsockopt(int, int, void const*, unsigned int) in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      InternetSocket::getsockopt(int, int, void*, unsigned int*) in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      InternetSocket::sigio(mbed::Callback<void ()>)    in libfeatures_netsocket_UDPSocket.MbedOS.a(InternetSocket.cpp.o)

      ...

ld: symbol(s) not found for architecture x86_64

collect2: error: ld returned 1 exit status

[0xc0170]

stub should implement both (void and uint32 arguments) methods, this is just one?

[lorjala]

Now it implements both.

[0xc0170]

/morph build

[cmonr]

@0xc0170 This should be @ARMmbed/mbed-os-test since the code change is to the unittest stub.

[mbed-ci]

Build : SUCCESS

Build number : 3012
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7988/

Triggering tests

/morph test
/morph export-build
/morph mbed2-build

[mbed-ci]

Exporter Build : FAILURE

Build number : 2624
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7988/

[cmonr]

Multiply defined asm symbols...
/morph export-build

[mbed-ci]

Test : FAILURE

Build number : 2779
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/7988/2779

[mbed-ci]

Exporter Build : FAILURE

Build number : 2626
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7988/

[cmonr]

Test failure does not appear to be related to PR.
/morph test.

Another ASM Symbol multiply defined error. Curious that this is the second time (@ARMmbed/mbed-os-test)
/morph export-build

[cmonr]

Whoops, added a period.
/morph test

[mbed-ci]

Test : SUCCESS

Build number : 2782
Test logs :http://mbed-os-logs.s3-website-us-west-1.amazonaws.com/?prefix=logs/7988/2782

[mbed-ci]

Exporter Build : SUCCESS

Build number : 2628
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/exporter/7988/