Project

General

Profile

Actions

Bug #3933

closed

Nack test incorrect counting of Nacks

Added by Junxiao Shi almost 8 years ago. Updated over 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Integration Tests
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:

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

20170125191715.tgz (839 KB) 20170125191715.tgz integ 3621,1 Junxiao Shi, 01/25/2017 08:08 PM

Related issues 1 (0 open1 closed)

Related to NFD - Bug #4058: nfdc input and output format changes cause integration tests to failClosedEric Newberry04/19/2017

Actions
Actions #1

Updated by Junxiao Shi almost 8 years ago

Test log is attached.

This is assigned to Hila who implemented this scenario.

Actions #2

Updated by Hila Ben Abraham almost 8 years ago

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?

Actions #3

Updated by Junxiao Shi over 7 years ago

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.

Actions #4

Updated by Junxiao Shi over 7 years ago

  • Assignee changed from Hila Ben Abraham to Eric Newberry

Eric volunteered to fix all broken integ on 20170419 call.

Actions #5

Updated by Eric Newberry over 7 years ago

  • Related to Bug #4058: nfdc input and output format changes cause integration tests to fail added
Actions #6

Updated by Eric Newberry over 7 years ago

  • Status changed from New to In Progress
Actions #7

Updated by Eric Newberry over 7 years ago

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?

Actions #8

Updated by Eric Newberry over 7 years ago

  • Status changed from In Progress to Code review
  • % Done changed from 0 to 100
Actions #9

Updated by Eric Newberry over 7 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF