Bug #3933
closed
Nack test incorrect counting of Nacks
Added by Junxiao Shi almost 8 years ago.
Updated over 7 years ago.
Category:
Integration Tests
Description
In IntegrationTests Nack scenario test 3, two Nacks were returned from two upstream faces, and NFD is expected to return only one Nack to the downstream face.
This was implemented as parsing the log of the consumer application and expecting the consumer application to receive only one Nack.
Since #3908, after a consumer expresses multiple Interests with same Name+Selectors+Link, when a Nack with specified Name+Selectors+Link arrives, all pending Interests would be Nacked, breaking the test assumption. Furthermore, in case NFD incorrectly returns more than one Nack, excessive Nacks received by ndn::Face would be dropped.
Thus, the test should be re-implemented by counting the number of Nacks between router NFD and consumer NFD (A-D), through parsing of ndndump or tcpdump logs.
Files
Test log is attached.
This is assigned to Hila who implemented this scenario.
Thus, the test should be re-implemented by counting the number of Nacks between router NFD and consumer NFD (A-D), through parsing of ndndump or tcpdump logs.
If I understand it correctly, steps #10 and #12 in #3157 should be changed to expect D to receive exactly two Nacks, one for each nonce sent. In addition, we should check that no additional nacks were incorrectly sent, and dropped by ndn::Face, by checking the actual packets sent using ndndump.
Am I right? Is it possible to reopen #3157 with an up-to-date version of the test's steps?
Reply to note-2:
No, the test assertions (expected number of Nacks) are correct. However, the actual number of Nacks cannot be counted from the consumer application, but should be counted on the network link by parsing ndndump logs.
- Assignee changed from Hila Ben Abraham to Eric Newberry
Eric volunteered to fix all broken integ on 20170419 call.
- Related to Bug #4058: nfdc input and output format changes cause integration tests to fail added
- Status changed from New to In Progress
Junxiao Shi wrote:
Reply to note-2:
No, the test assertions (expected number of Nacks) are correct. However, the actual number of Nacks cannot be counted from the consumer application, but should be counted on the network link by parsing ndndump logs.
So therefore changes should be made to both test 3 and test 4?
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
- Status changed from Code review to Closed
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by