emphasize integration tests in README.md

Bug #1326247 reported by Loic Dachary
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ceph
Fix Released
Low
Guillaume Sainthillier

Bug Description

In README.md replace

 Each component is unit tested and an integration test shows that it performs as expected when used with a realistic scenario.

with

  Each feature is tested with an integration tests involving virtual machines to show that it performs as expected when used with a realistic scenario.

to emphasize the importance of integration tests. Knowing that integration tests exist is important to the person who discovers the module for the first time. Unit tests are also important but from the developer perspective.

Loic Dachary (dachary)
summary: - do not emphasize unit tests in README.md
+ emphasize integration tests in README.md
Changed in puppet-ceph:
assignee: nobody → Guillaume Sainthillier (x-guillaume)
pernot (pernotjln)
Changed in puppet-ceph:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceph (master)

Reviewed: https://review.openstack.org/100480
Committed: https://git.openstack.org/cgit/stackforge/puppet-ceph/commit/?id=8d1e1236941884395f3fcd2230bc5161db246b4d
Submitter: Jenkins
Branch: master

commit 8d1e1236941884395f3fcd2230bc5161db246b4d
Author: Baillagou Florian <email address hidden>
Date: Tue Jun 17 14:59:42 2014 +0200

    Implements : emphasize integration tests in README.md

    Closes-Bug: #1326247
    Change-Id: Ibd296ab0b4c50d8ab42d427bd711df9235c4b4f4

Changed in puppet-ceph:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on puppet-ceph (master)

Change abandoned by David Moreau Simard (<email address hidden>) on branch: master
Review: https://review.openstack.org/100594
Reason: Closing this review, it has been inactive for the past month. Please feel free to re-open and address the comments in the review if you would like this to go in.

Changed in puppet-ceph:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.