tree 8fec0b4abc29359177d35d3e595346c699f5dc10
parent 7e3c936898d05eb9f52e5ce37f463f903ee16ed5
author Felipe Monteiro <felipe.monteiro@att.com> 1537558376 +0100
committer Felipe Monteiro <felipe.monteiro@att.com> 1537558376 +0100

fix: Re-clarify unit test REVIEWING documentation

This patch set corrects some misleading documentation under "Unit
Tests" section in REVIEWING.rst.

It currently claims that service clients do not require unit test
coverage -- but this is false. This is because Tempest now places
all of its service clients in tempest.lib. And as per
https://docs.openstack.org/tempest/latest/library.html#testing it
is required to add unit tests for all service client interfaces.

Thus this makes the documentation language clear that service clients
require unit tests.

Change-Id: Iac6ba71496e20b6724a6a2ca4ec2beb42f2a58c8