diff options
author | Rick Pasetto <rick@lindenlab.com> | 2009-10-09 18:56:36 -0700 |
---|---|---|
committer | Rick Pasetto <rick@lindenlab.com> | 2009-10-09 18:56:36 -0700 |
commit | d4b2897700c66354413af42ab055bd1aaa47f91c (patch) | |
tree | c496bd8785b94bb60bca33ea59758c5929341451 /indra/llmessage/tests | |
parent | 8d1f3f735194775b754011de1f6000ccb6d1039e (diff) |
Unit tests for LLMediaDataClient
This required a bit of refactoring of LLMediaDataClient:
- Created LLMediaDataClientObject ABC, which now has a
concrete impl in LLVOVolume
- Created unit test with 6 tests (for now), testing
- LLObjectMediaDataClient::fetchMedia()
- LLObjectMediaDataClient::updateMedia()
- LLObjectMediaNavigateClient::navigate()
- queue ordering
- retries
- nav bounce back
- Also ensures that ref counting works properly (this is important, because
ownership is tricky with smart pointers put into queues, peeled off
into timers that fire and auto destruct, and HTTP responders that also
auto-destruct)
- Had to fix LLCurl::Responder's stub, which was not initializing
the ref count to 0, causing the ref counting tests to fail
(boy, that was hard to find!).
Reviewed by Callum
Diffstat (limited to 'indra/llmessage/tests')
-rw-r--r-- | indra/llmessage/tests/llcurl_stub.cpp | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/indra/llmessage/tests/llcurl_stub.cpp b/indra/llmessage/tests/llcurl_stub.cpp index 5dc5932fde..e6a5ad9946 100644 --- a/indra/llmessage/tests/llcurl_stub.cpp +++ b/indra/llmessage/tests/llcurl_stub.cpp @@ -22,6 +22,7 @@ #include "linden_common.h" LLCurl::Responder::Responder() + : mReferenceCount(0) { } |