Project

General

Profile

Task #1952

Feature #3744: Modularize NLSR

Decouple classes from Nlsr object for easier unit testing

Added by Vince Lehman over 4 years ago. Updated 8 days ago.

Status:
Code review
Priority:
Normal
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Currently, multiple objects use an Nlsr object as a parameter in their constructor. This makes unit tests difficult since an Nlsr object must be initialized to test these components.

Classes:

  • HelloProtocol

  • Lsdb

  • NamePrefixTable

  • FileProcessor

Instead, the objects should take the information they require from Nlsr as parameters.


Related issues

Blocks NLSR - Task #4288: Move sequence number file setting to the constructorCode review

History

#1 Updated by Nicholas Gordon over 1 year ago

#2 Updated by Nicholas Gordon over 1 year ago

  • Parent task set to #3744

#3 Updated by Nicholas Gordon over 1 year ago

  • Target version set to v0.5.0

#4 Updated by Nicholas Gordon about 1 year ago

  • Assignee set to Nicholas Gordon
  • Target version changed from v0.5.0 to Minor release v0.5.1

#5 Updated by Ashlesh Gawande about 1 year ago

  • Related to Task #4288: Move sequence number file setting to the constructor added

#6 Updated by Nicholas Gordon about 1 year ago

  • Related to deleted (Task #4288: Move sequence number file setting to the constructor)

#7 Updated by Nicholas Gordon about 1 year ago

  • Blocks Task #4288: Move sequence number file setting to the constructor added

#8 Updated by Nicholas Gordon 8 months ago

  • Assignee deleted (Nicholas Gordon)

#9 Updated by Ashlesh Gawande 8 days ago

  • Status changed from New to Code review
  • Assignee set to Ashlesh Gawande
  • Target version changed from Minor release v0.5.1 to v0.5.0
  • Start date deleted (08/29/2014)

#10 Updated by Ashlesh Gawande 8 days ago

Current change, https://gerrit.named-data.net/c/NLSR/+/4398, does not pass individual components in the ctor if they are too many (example lsdb) but instead takes ConfParameter which has all parameters configured at the time of ctor initialization. It does resolve passing Nlsr to ConfFileProcessor and making things easier to initialize along with making classes independent of Nlsr object. Current change also does not enhance any tests, just make them compile and pass.

Also available in: Atom PDF