strepsirrhini/office-gobmx
9
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Also check number of invocations in the test

Browse source 
Change-Id: I4180f14aa633bf0f3f45178c1fd02b52b784f7e3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111960
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
This commit is contained in:
Mike Kaganski 2021-03-04 15:04:12 +03:00 committed by Mike Kaganski
parent 247963e53c
commit b3bfcb2805
1 changed files with 10 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
sal/qa/systools/test_retry_if_failed.cxx
View file

@ -20,27 +20,31 @@ class test_retry_if_failed : public CppUnit::TestFixture
public:
void test_success()
{
unsigned counter = 0;
const DWORD nTicksBefore = GetTickCount();
HRESULT hr = sal::systools::RetryIfFailed(10, 200, Tester(5));
HRESULT hr = sal::systools::RetryIfFailed(10, 200, Tester(5, counter));
const DWORD nTicksAfter = GetTickCount();
const DWORD nTicksElapsed = nTicksAfter > nTicksBefore ? nTicksAfter - nTicksBefore
: std::numeric_limits<DWORD>::max()
- nTicksBefore + nTicksAfter;
CPPUNIT_ASSERT(SUCCEEDED(hr));
// 5 attempts, 4 sleeps by ~200 ms
CPPUNIT_ASSERT_EQUAL(5U, counter);
CPPUNIT_ASSERT_GREATER(DWORD(800 - ClockRes), nTicksElapsed);
}
void test_failure()
{
unsigned counter = 0;
const DWORD nTicksBefore = GetTickCount();
HRESULT hr = sal::systools::RetryIfFailed(10, 100, Tester(15));
HRESULT hr = sal::systools::RetryIfFailed(10, 100, Tester(15, counter));
const DWORD nTicksAfter = GetTickCount();
const DWORD nTicksElapsed = nTicksAfter > nTicksBefore ? nTicksAfter - nTicksBefore
: std::numeric_limits<DWORD>::max()
- nTicksBefore + nTicksAfter;
CPPUNIT_ASSERT(FAILED(hr));
// 1 + 10 attempts, 10 sleeps by ~100 ms
CPPUNIT_ASSERT_EQUAL(11U, counter);
CPPUNIT_ASSERT_GREATER(DWORD(1000 - ClockRes), nTicksElapsed);
}
@ -52,18 +56,16 @@ public:
private:
struct Tester
{
Tester(unsigned triesBeforeSuccess)
Tester(unsigned triesBeforeSuccess, unsigned& counter)
: m_nTriesBeforeSuccess(triesBeforeSuccess)
, m_rCounter(counter)
{
}
HRESULT operator()()
{
return ++m_nTriesAttempted >= m_nTriesBeforeSuccess ? S_OK : E_FAIL;
}
HRESULT operator()() { return ++m_rCounter >= m_nTriesBeforeSuccess ? S_OK : E_FAIL; }
unsigned m_nTriesBeforeSuccess;
unsigned m_nTriesAttempted = 0;
unsigned& m_rCounter;
};
};