Test #164
Updated by Ivan Herrero over 9 years ago
h1. Requirements review
*Date of the review:* 07/08/2015
*Person who did the review:* André Neto
*Version of requirements:* 0.1
*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:* 07/08/2015
*Person who did the review:* André Neto
*Version of architecture & design document:* 0.1
*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:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of source code:* 0.1
*Result of review:* PASS
*List of non-conformities:* N/A
*Comments*: The error infrastructure's name has changed between Logger, LogManagement and ErrorManagement. It should be analyzed if the last name reflects its purpose.
h1. Unit test review
*Date of the review:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of unit tests:* 0.1
*Result of coverage tests review:* PASS
*Result of functional tests review:* PASS
*Result of review:* PASS
*List of non-conformities:* N/A
*Comments*: The function ErrorManagementTest::CheckParameters does not verify ErrorInformation::header::isObject, ErrorInformation::hrtTime, ErrorInformation::objectPointer, and ErrorInformation::className, because they are not implemented.
h1. Integration test review
*Date of the review:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of integration tests:* 0.1
*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:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of acceptance tests:* 0.1
*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:* 07/08/2015
*Person who did the review:* André Neto
*Version of requirements:* 0.1
*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:* 07/08/2015
*Person who did the review:* André Neto
*Version of architecture & design document:* 0.1
*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:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of source code:* 0.1
*Result of review:* PASS
*List of non-conformities:* N/A
*Comments*: The error infrastructure's name has changed between Logger, LogManagement and ErrorManagement. It should be analyzed if the last name reflects its purpose.
h1. Unit test review
*Date of the review:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of unit tests:* 0.1
*Result of coverage tests review:* PASS
*Result of functional tests review:* PASS
*Result of review:* PASS
*List of non-conformities:* N/A
*Comments*: The function ErrorManagementTest::CheckParameters does not verify ErrorInformation::header::isObject, ErrorInformation::hrtTime, ErrorInformation::objectPointer, and ErrorInformation::className, because they are not implemented.
h1. Integration test review
*Date of the review:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of integration tests:* 0.1
*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:* 07/08/2015
*Person who did the review:* Ivan Herrero
*Version of acceptance tests:* 0.1
*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.