Test #263
Updated by Ivan Herrero over 9 years ago
h1. Requirements review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of requirements:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal requirements.
h1. Architecture & design review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of architecture & design document:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal design.
h1. Code and documentation review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of source code:* 0.7
*Result of review:* PASS [PASS|FAIL]
*List of non-conformities:* N/A [N/A|list]
h1. Unit test review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of unit tests:* 0.7
*Result of coverage tests review:* PASS [PASS|FAIL]
*Result of functional tests review:* PASS [PASS|FAIL]
*Result of review:* PASS [PASS|FAIL]
*List of non-conformities:* [N/A|list]
[#241] As a note, as the number of error reports is increasing, the coverage is struggling to keep up to date. We should try to find a way to tell lcov to accept as covered when an error is reported.
[#242] Few incarnations of the templated functions are not tested due to escessive nesting.
[#245] The current implementation uses a temporary version of SocketSelect, which will be further developed in the following sprints. For this reason the low code coverage of SocketSelect is not considered an issue.
h1. Integration test review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of integration tests:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal integration tests.
h1. Acceptance test review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of acceptance tests:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal acceptance tests.
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of requirements:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal requirements.
h1. Architecture & design review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of architecture & design document:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal design.
h1. Code and documentation review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of source code:* 0.7
*Result of review:* PASS [PASS|FAIL]
*List of non-conformities:* N/A [N/A|list]
h1. Unit test review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of unit tests:* 0.7
*Result of coverage tests review:* PASS [PASS|FAIL]
*Result of functional tests review:* PASS [PASS|FAIL]
*Result of review:* PASS [PASS|FAIL]
*List of non-conformities:* [N/A|list]
[#241] As a note, as the number of error reports is increasing, the coverage is struggling to keep up to date. We should try to find a way to tell lcov to accept as covered when an error is reported.
[#242] Few incarnations of the templated functions are not tested due to escessive nesting.
[#245] The current implementation uses a temporary version of SocketSelect, which will be further developed in the following sprints. For this reason the low code coverage of SocketSelect is not considered an issue.
h1. Integration test review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of integration tests:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal integration tests.
h1. Acceptance test review
*Date of the review:* 30/11/2015 06/11/2015
*Person who did the review:* Ivan Herrero
*Version of acceptance tests:* 0.7
*Result of review:* N/A
*List of non-conformities:* N/A
*Comments*: As per the original MARTe implementation, the user stories of this sprint are support functions and do not require formal acceptance tests.