diff options
Diffstat (limited to 'test')
| -rw-r--r-- | test/BNodeTest.cc | 16 | ||||
| -rw-r--r-- | test/DHTAnnouncePeerMessageTest.cc | 4 | ||||
| -rw-r--r-- | test/DHTBucketTest.cc | 22 | ||||
| -rw-r--r-- | test/DHTFindNodeMessageTest.cc | 4 | ||||
| -rw-r--r-- | test/DHTGetPeersMessageTest.cc | 10 | ||||
| -rw-r--r-- | test/DHTIDCloserTest.cc | 10 | ||||
| -rw-r--r-- | test/DHTMessageFactoryImplTest.cc | 64 | ||||
| -rw-r--r-- | test/DHTPingMessageTest.cc | 4 | ||||
| -rw-r--r-- | test/DefaultBtRequestFactoryTest.cc | 12 | ||||
| -rw-r--r-- | test/DefaultPeerStorageTest.cc | 10 | ||||
| -rw-r--r-- | test/SequentialPickerTest.cc | 4 | ||||
| -rw-r--r-- | test/SingletonHolderTest.cc | 2 |
12 files changed, 83 insertions, 79 deletions
diff --git a/test/BNodeTest.cc b/test/BNodeTest.cc index f5ddac6e..730225b1 100644 --- a/test/BNodeTest.cc +++ b/test/BNodeTest.cc @@ -68,7 +68,7 @@ void BNodeTest::testFindBucketFor() { BNode b(bucket5); - CPPUNIT_ASSERT(bucket5 == BNode::findBucketFor(&b, localNodeID)); + CPPUNIT_ASSERT(*bucket5 == *BNode::findBucketFor(&b, localNodeID)); } { BNode b(bucket1); @@ -97,7 +97,7 @@ void BNodeTest::testFindBucketFor() bp4->setLeft(bp3); bp4->setRight(b2); - CPPUNIT_ASSERT(bucket5 == BNode::findBucketFor(bp4, localNode->getID())); + CPPUNIT_ASSERT(*bucket5 == *BNode::findBucketFor(bp4, localNode->getID())); delete bp4; } @@ -220,7 +220,7 @@ void BNodeTest::testEnumerateBucket() std::vector<SharedHandle<DHTBucket> > buckets; BNode::enumerateBucket(buckets, &b); CPPUNIT_ASSERT_EQUAL((size_t)1, buckets.size()); - CPPUNIT_ASSERT(bucket1 == buckets[0]); + CPPUNIT_ASSERT(*bucket1 == *buckets[0]); } { BNode* b1 = new BNode(bucket1); @@ -248,11 +248,11 @@ void BNodeTest::testEnumerateBucket() std::vector<SharedHandle<DHTBucket> > buckets; BNode::enumerateBucket(buckets, bp4); CPPUNIT_ASSERT_EQUAL((size_t)5, buckets.size()); - CPPUNIT_ASSERT(bucket1 == buckets[0]); - CPPUNIT_ASSERT(bucket3 == buckets[1]); - CPPUNIT_ASSERT(bucket5 == buckets[2]); - CPPUNIT_ASSERT(bucket4 == buckets[3]); - CPPUNIT_ASSERT(bucket2 == buckets[4]); + CPPUNIT_ASSERT(*bucket1 == *buckets[0]); + CPPUNIT_ASSERT(*bucket3 == *buckets[1]); + CPPUNIT_ASSERT(*bucket5 == *buckets[2]); + CPPUNIT_ASSERT(*bucket4 == *buckets[3]); + CPPUNIT_ASSERT(*bucket2 == *buckets[4]); delete bp4; } diff --git a/test/DHTAnnouncePeerMessageTest.cc b/test/DHTAnnouncePeerMessageTest.cc index 1a8c8149..20bc28db 100644 --- a/test/DHTAnnouncePeerMessageTest.cc +++ b/test/DHTAnnouncePeerMessageTest.cc @@ -111,8 +111,8 @@ void DHTAnnouncePeerMessageTest::testDoReceivedAction() SharedHandle<MockDHTResponseMessage> m (dynamic_pointer_cast<MockDHTResponseMessage> (dispatcher.messageQueue_[0].message_)); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("announce_peer"), m->getMessageType()); CPPUNIT_ASSERT_EQUAL(transactionID, m->getTransactionID()); std::vector<SharedHandle<Peer> > peers; diff --git a/test/DHTBucketTest.cc b/test/DHTBucketTest.cc index 14da5016..11adca59 100644 --- a/test/DHTBucketTest.cc +++ b/test/DHTBucketTest.cc @@ -319,7 +319,7 @@ void DHTBucketTest::testAddNode() // nodes[0] is located at the tail of the bucket(least recent seen) nodes[0]->markBad(); CPPUNIT_ASSERT(bucket.addNode(newNode)); - CPPUNIT_ASSERT(bucket.getNodes().back() == newNode); + CPPUNIT_ASSERT(*bucket.getNodes().back() == *newNode); } void DHTBucketTest::testMoveToHead() @@ -337,7 +337,7 @@ void DHTBucketTest::testMoveToHead() CPPUNIT_ASSERT(bucket.addNode(nodes[i])); } bucket.moveToHead(nodes[DHTBucket::K-1]); - CPPUNIT_ASSERT(bucket.getNodes().front() == nodes[DHTBucket::K-1]); + CPPUNIT_ASSERT(*bucket.getNodes().front() == *nodes[DHTBucket::K-1]); } void DHTBucketTest::testMoveToTail() @@ -355,7 +355,7 @@ void DHTBucketTest::testMoveToTail() CPPUNIT_ASSERT(bucket.addNode(nodes[i])); } bucket.moveToTail(nodes[0]); - CPPUNIT_ASSERT(bucket.getNodes().back() == nodes[0]); + CPPUNIT_ASSERT(*bucket.getNodes().back() == *nodes[0]); } void DHTBucketTest::testGetGoodNodes() @@ -400,12 +400,12 @@ void DHTBucketTest::testCacheNode() bucket.cacheNode(n1); bucket.cacheNode(n2); CPPUNIT_ASSERT_EQUAL((size_t)2, bucket.getCachedNodes().size()); - CPPUNIT_ASSERT(n2 == bucket.getCachedNodes()[0]); + CPPUNIT_ASSERT(*n2 == *bucket.getCachedNodes()[0]); bucket.cacheNode(n3); CPPUNIT_ASSERT_EQUAL((size_t)2, bucket.getCachedNodes().size()); - CPPUNIT_ASSERT(n3 == bucket.getCachedNodes()[0]); - CPPUNIT_ASSERT(n2 == bucket.getCachedNodes()[1]); + CPPUNIT_ASSERT(*n3 == *bucket.getCachedNodes()[0]); + CPPUNIT_ASSERT(*n2 == *bucket.getCachedNodes()[1]); } void DHTBucketTest::testDropNode() @@ -432,7 +432,7 @@ void DHTBucketTest::testDropNode() { std::deque<SharedHandle<DHTNode> > tnodes = bucket.getNodes(); CPPUNIT_ASSERT_EQUAL((size_t)8, tnodes.size()); - CPPUNIT_ASSERT(nodes[3] == tnodes[3]); + CPPUNIT_ASSERT(*nodes[3] == *tnodes[3]); } bucket.cacheNode(cachedNode1); @@ -442,11 +442,13 @@ void DHTBucketTest::testDropNode() { std::deque<SharedHandle<DHTNode> > tnodes = bucket.getNodes(); CPPUNIT_ASSERT_EQUAL((size_t)8, tnodes.size()); - CPPUNIT_ASSERT(tnodes.end() == std::find(tnodes.begin(), tnodes.end(), nodes[3])); - CPPUNIT_ASSERT(cachedNode2 == tnodes[7]); + CPPUNIT_ASSERT(tnodes.end() == + std::find_if(tnodes.begin(), tnodes.end(), + derefEqual(nodes[3]))); + CPPUNIT_ASSERT(*cachedNode2 == *tnodes[7]); } CPPUNIT_ASSERT_EQUAL((size_t)1, bucket.getCachedNodes().size()); - CPPUNIT_ASSERT(cachedNode1 == bucket.getCachedNodes()[0]); + CPPUNIT_ASSERT(*cachedNode1 == *bucket.getCachedNodes()[0]); } } // namespace aria2 diff --git a/test/DHTFindNodeMessageTest.cc b/test/DHTFindNodeMessageTest.cc index c9f0ce54..984c7238 100644 --- a/test/DHTFindNodeMessageTest.cc +++ b/test/DHTFindNodeMessageTest.cc @@ -103,8 +103,8 @@ void DHTFindNodeMessageTest::testDoReceivedAction() SharedHandle<MockDHTResponseMessage> m (dynamic_pointer_cast<MockDHTResponseMessage> (dispatcher.messageQueue_[0].message_)); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("find_node"), m->getMessageType()); CPPUNIT_ASSERT_EQUAL(msg.getTransactionID(), m->getTransactionID()); CPPUNIT_ASSERT_EQUAL((size_t)1, m->nodes_.size()); diff --git a/test/DHTGetPeersMessageTest.cc b/test/DHTGetPeersMessageTest.cc index 11a280c1..be100ea2 100644 --- a/test/DHTGetPeersMessageTest.cc +++ b/test/DHTGetPeersMessageTest.cc @@ -124,8 +124,8 @@ void DHTGetPeersMessageTest::testDoReceivedAction() SharedHandle<MockDHTResponseMessage> m (dynamic_pointer_cast<MockDHTResponseMessage> (dispatcher.messageQueue_[0].message_)); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("get_peers"), m->getMessageType()); CPPUNIT_ASSERT_EQUAL(msg.getTransactionID(), m->getTransactionID()); CPPUNIT_ASSERT_EQUAL(tokenTracker.generateToken(infoHash, remoteNode->getIPAddress(), remoteNode->getPort()), m->token_); @@ -161,13 +161,13 @@ void DHTGetPeersMessageTest::testDoReceivedAction() SharedHandle<MockDHTResponseMessage> m (dynamic_pointer_cast<MockDHTResponseMessage> (dispatcher.messageQueue_[0].message_)); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("get_peers"), m->getMessageType()); CPPUNIT_ASSERT_EQUAL(msg.getTransactionID(), m->getTransactionID()); CPPUNIT_ASSERT_EQUAL(tokenTracker.generateToken(infoHash, remoteNode->getIPAddress(), remoteNode->getPort()), m->token_); CPPUNIT_ASSERT_EQUAL((size_t)1, m->nodes_.size()); - CPPUNIT_ASSERT(returnNode1 == m->nodes_[0]); + CPPUNIT_ASSERT(*returnNode1 == *m->nodes_[0]); CPPUNIT_ASSERT_EQUAL((size_t)0, m->peers_.size()); } } diff --git a/test/DHTIDCloserTest.cc b/test/DHTIDCloserTest.cc index 032aba6d..a5e3a944 100644 --- a/test/DHTIDCloserTest.cc +++ b/test/DHTIDCloserTest.cc @@ -58,11 +58,11 @@ void DHTIDCloserTest::testOperator() std::sort(entries.begin(), entries.end(), DHTIDCloser(e3->node->getID())); - CPPUNIT_ASSERT(e3 == entries[0]); - CPPUNIT_ASSERT(e2 == entries[1]); - CPPUNIT_ASSERT(e4 == entries[2]); - CPPUNIT_ASSERT(e1 == entries[3]); - CPPUNIT_ASSERT(e5 == entries[4]); + CPPUNIT_ASSERT(*e3 == *entries[0]); + CPPUNIT_ASSERT(*e2 == *entries[1]); + CPPUNIT_ASSERT(*e4 == *entries[2]); + CPPUNIT_ASSERT(*e1 == *entries[3]); + CPPUNIT_ASSERT(*e5 == *entries[4]); } } // namespace aria2 diff --git a/test/DHTMessageFactoryImplTest.cc b/test/DHTMessageFactoryImplTest.cc index c28b80de..f2515158 100644 --- a/test/DHTMessageFactoryImplTest.cc +++ b/test/DHTMessageFactoryImplTest.cc @@ -100,8 +100,8 @@ void DHTMessageFactoryImplTest::testCreatePingMessage() remoteNode->setIPAddress("192.168.0.1"); remoteNode->setPort(6881); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } @@ -125,8 +125,8 @@ void DHTMessageFactoryImplTest::testCreatePingReplyMessage() remoteNode->getIPAddress(), remoteNode->getPort()))); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } @@ -151,8 +151,8 @@ void DHTMessageFactoryImplTest::testCreateFindNodeMessage() remoteNode->setIPAddress("192.168.0.1"); remoteNode->setPort(6881); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); CPPUNIT_ASSERT_EQUAL(util::toHex(targetNodeID, DHT_ID_LENGTH), @@ -196,11 +196,11 @@ void DHTMessageFactoryImplTest::testCreateFindNodeReplyMessage() remoteNode->getIPAddress(), remoteNode->getPort()))); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL((size_t)DHTBucket::K, m->getClosestKNodes().size()); - CPPUNIT_ASSERT(nodes[0] == m->getClosestKNodes()[0]); - CPPUNIT_ASSERT(nodes[7] == m->getClosestKNodes()[7]); + CPPUNIT_ASSERT(*nodes[0] == *m->getClosestKNodes()[0]); + CPPUNIT_ASSERT(*nodes[7] == *m->getClosestKNodes()[7]); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } catch(Exception& e) { @@ -249,11 +249,11 @@ void DHTMessageFactoryImplTest::testCreateFindNodeReplyMessage6() remoteNode->getIPAddress(), remoteNode->getPort()))); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL((size_t)DHTBucket::K, m->getClosestKNodes().size()); - CPPUNIT_ASSERT(nodes[0] == m->getClosestKNodes()[0]); - CPPUNIT_ASSERT(nodes[7] == m->getClosestKNodes()[7]); + CPPUNIT_ASSERT(*nodes[0] == *m->getClosestKNodes()[0]); + CPPUNIT_ASSERT(*nodes[7] == *m->getClosestKNodes()[7]); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } catch(Exception& e) { @@ -282,8 +282,8 @@ void DHTMessageFactoryImplTest::testCreateGetPeersMessage() remoteNode->setIPAddress("192.168.0.1"); remoteNode->setPort(6881); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); CPPUNIT_ASSERT_EQUAL(util::toHex(infoHash, DHT_ID_LENGTH), @@ -343,15 +343,15 @@ void DHTMessageFactoryImplTest::testCreateGetPeersReplyMessage() remoteNode->getIPAddress(), remoteNode->getPort()))); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("token"), m->getToken()); CPPUNIT_ASSERT_EQUAL((size_t)DHTBucket::K, m->getClosestKNodes().size()); - CPPUNIT_ASSERT(nodes[0] == m->getClosestKNodes()[0]); - CPPUNIT_ASSERT(nodes[7] == m->getClosestKNodes()[7]); + CPPUNIT_ASSERT(*nodes[0] == *m->getClosestKNodes()[0]); + CPPUNIT_ASSERT(*nodes[7] == *m->getClosestKNodes()[7]); CPPUNIT_ASSERT_EQUAL((size_t)4, m->getValues().size()); - CPPUNIT_ASSERT(peers[0] == m->getValues()[0]); - CPPUNIT_ASSERT(peers[3] == m->getValues()[3]); + CPPUNIT_ASSERT(*peers[0] == *m->getValues()[0]); + CPPUNIT_ASSERT(*peers[3] == *m->getValues()[3]); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } catch(Exception& e) { @@ -416,15 +416,15 @@ void DHTMessageFactoryImplTest::testCreateGetPeersReplyMessage6() remoteNode->getIPAddress(), remoteNode->getPort()))); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("token"), m->getToken()); CPPUNIT_ASSERT_EQUAL((size_t)DHTBucket::K, m->getClosestKNodes().size()); - CPPUNIT_ASSERT(nodes[0] == m->getClosestKNodes()[0]); - CPPUNIT_ASSERT(nodes[7] == m->getClosestKNodes()[7]); + CPPUNIT_ASSERT(*nodes[0] == *m->getClosestKNodes()[0]); + CPPUNIT_ASSERT(*nodes[7] == *m->getClosestKNodes()[7]); CPPUNIT_ASSERT_EQUAL((size_t)4, m->getValues().size()); - CPPUNIT_ASSERT(peers[0] == m->getValues()[0]); - CPPUNIT_ASSERT(peers[3] == m->getValues()[3]); + CPPUNIT_ASSERT(*peers[0] == *m->getValues()[0]); + CPPUNIT_ASSERT(*peers[3] == *m->getValues()[3]); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } catch(Exception& e) { @@ -458,8 +458,8 @@ void DHTMessageFactoryImplTest::testCreateAnnouncePeerMessage() remoteNode->setIPAddress("192.168.0.1"); remoteNode->setPort(6882); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(token, m->getToken()); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); @@ -490,8 +490,8 @@ void DHTMessageFactoryImplTest::testCreateAnnouncePeerReplyMessage() remoteNode->getIPAddress(), remoteNode->getPort()))); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(util::toHex(transactionID, DHT_TRANSACTION_ID_LENGTH), util::toHex(m->getTransactionID())); } diff --git a/test/DHTPingMessageTest.cc b/test/DHTPingMessageTest.cc index dd12c20d..671a2734 100644 --- a/test/DHTPingMessageTest.cc +++ b/test/DHTPingMessageTest.cc @@ -92,8 +92,8 @@ void DHTPingMessageTest::testDoReceivedAction() SharedHandle<MockDHTResponseMessage> m (dynamic_pointer_cast<MockDHTResponseMessage> (dispatcher.messageQueue_[0].message_)); - CPPUNIT_ASSERT(localNode == m->getLocalNode()); - CPPUNIT_ASSERT(remoteNode == m->getRemoteNode()); + CPPUNIT_ASSERT(*localNode == *m->getLocalNode()); + CPPUNIT_ASSERT(*remoteNode == *m->getRemoteNode()); CPPUNIT_ASSERT_EQUAL(std::string("ping_reply"), m->getMessageType()); CPPUNIT_ASSERT_EQUAL(msg.getTransactionID(), m->getTransactionID()); } diff --git a/test/DefaultBtRequestFactoryTest.cc b/test/DefaultBtRequestFactoryTest.cc index 941fa0e4..43d4d0bc 100644 --- a/test/DefaultBtRequestFactoryTest.cc +++ b/test/DefaultBtRequestFactoryTest.cc @@ -210,16 +210,16 @@ void DefaultBtRequestFactoryTest::testRemoveTargetPiece() { requestFactory_->addTargetPiece(piece1); - CPPUNIT_ASSERT(std::find(requestFactory_->getTargetPieces().begin(), - requestFactory_->getTargetPieces().end(), - piece1) != + CPPUNIT_ASSERT(std::find_if(requestFactory_->getTargetPieces().begin(), + requestFactory_->getTargetPieces().end(), + derefEqual(piece1)) != requestFactory_->getTargetPieces().end()); requestFactory_->removeTargetPiece(piece1); - CPPUNIT_ASSERT(std::find(requestFactory_->getTargetPieces().begin(), - requestFactory_->getTargetPieces().end(), - piece1) == + CPPUNIT_ASSERT(std::find_if(requestFactory_->getTargetPieces().begin(), + requestFactory_->getTargetPieces().end(), + derefEqual(piece1)) == requestFactory_->getTargetPieces().end()); } diff --git a/test/DefaultPeerStorageTest.cc b/test/DefaultPeerStorageTest.cc index 9b1f7149..1394cf1f 100644 --- a/test/DefaultPeerStorageTest.cc +++ b/test/DefaultPeerStorageTest.cc @@ -121,7 +121,8 @@ void DefaultPeerStorageTest::testAddPeer() { CPPUNIT_ASSERT(ps.addPeer(peer4)); // peer2 was deleted. While peer1 is oldest, its cuid is not 0. CPPUNIT_ASSERT_EQUAL((size_t)3, ps.countPeer()); - CPPUNIT_ASSERT(std::find(ps.getPeers().begin(), ps.getPeers().end(), peer2) == ps.getPeers().end()); + CPPUNIT_ASSERT(std::find_if(ps.getPeers().begin(), ps.getPeers().end(), + derefEqual(peer2)) == ps.getPeers().end()); SharedHandle<Peer> peer5(new Peer("192.168.0.4", 0)); @@ -224,13 +225,14 @@ void DefaultPeerStorageTest::testReturnPeer() ps.returnPeer(peer2); // peer2 removed from the container CPPUNIT_ASSERT_EQUAL((size_t)2, ps.getPeers().size()); - CPPUNIT_ASSERT(std::find(ps.getPeers().begin(), ps.getPeers().end(), peer2) - == ps.getPeers().end()); + CPPUNIT_ASSERT(std::find_if(ps.getPeers().begin(), ps.getPeers().end(), + derefEqual(peer2)) == ps.getPeers().end()); CPPUNIT_ASSERT_EQUAL((size_t)1, ps.getDroppedPeers().size()); ps.returnPeer(peer1); // peer1 is removed from the container CPPUNIT_ASSERT_EQUAL((size_t)1, ps.getPeers().size()); - CPPUNIT_ASSERT(std::find(ps.getPeers().begin(), ps.getPeers().end(), peer1) == ps.getPeers().end()); + CPPUNIT_ASSERT(std::find_if(ps.getPeers().begin(), ps.getPeers().end(), + derefEqual(peer1)) == ps.getPeers().end()); CPPUNIT_ASSERT_EQUAL((size_t)1, ps.getDroppedPeers().size()); } diff --git a/test/SequentialPickerTest.cc b/test/SequentialPickerTest.cc index 171e201a..347e617f 100644 --- a/test/SequentialPickerTest.cc +++ b/test/SequentialPickerTest.cc @@ -35,7 +35,7 @@ void SequentialPickerTest::testPick() picker.pickNext(); CPPUNIT_ASSERT(picker.isPicked()); - CPPUNIT_ASSERT_EQUAL(Integer(new int(1)), picker.getPickedEntry()); + CPPUNIT_ASSERT_EQUAL(*Integer(new int(1)), *picker.getPickedEntry()); picker.dropPickedEntry(); @@ -44,7 +44,7 @@ void SequentialPickerTest::testPick() picker.pickNext(); - CPPUNIT_ASSERT_EQUAL(Integer(new int(2)), picker.getPickedEntry()); + CPPUNIT_ASSERT_EQUAL(*Integer(new int(2)), *picker.getPickedEntry()); CPPUNIT_ASSERT(!picker.hasNext()); } diff --git a/test/SingletonHolderTest.cc b/test/SingletonHolderTest.cc index 6b939824..b2dc2c84 100644 --- a/test/SingletonHolderTest.cc +++ b/test/SingletonHolderTest.cc @@ -51,7 +51,7 @@ void SingletonHolderTest::testInstance() IntHandle i(new int(100)); SingletonHolder<IntHandle>::instance(i); - std::cerr << SingletonHolder<IntHandle>::instance() << std::endl; + std::cerr << *SingletonHolder<IntHandle>::instance() << std::endl; std::cerr << SingletonHolder<MHandle>::instance()->greeting() << std::endl; |
