Requirements review¶

Date of the review: 08/09/2015

Person who did the review: Ivan Herrero

Version of requirements: 0.4

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.

Architecture & design review¶

Date of the review: 08/09/2015

Person who did the review: Ivan Herrero

Version of architecture & design document: 0.4

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.

Code and documentation review¶

Date of the review: 08/09/2015

Person who did the review: Ivan Herrero

Version of source code: 0.4

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.

Unit test review¶

Date of the review: 08/09/2015

Person who did the review: Ivan Herrero

Version of unit tests: 0.4

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.

Integration test review¶

Date of the review: 08/09/2015

Person who did the review: Ivan Herrero

Version of integration tests: 0.4

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.

Acceptance test review¶

Date of the review: 08/09/2015

Person who did the review: Ivan Herrero

Version of acceptance tests: 0.4

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.