[Xapian-tickets] [Xapian] #710: Public API for allowing custom Remote Protocol Server implementation
Xapian
nobody at xapian.org
Mon Dec 26 04:42:20 GMT 2016
#710: Public API for allowing custom Remote Protocol Server implementation
----------------------------+-------------------------------
Reporter: Kronuz | Owner: olly
Type: enhancement | Status: assigned
Priority: high | Milestone: 1.4.3
Component: Backend-Remote | Version: git master
Severity: normal | Resolution:
Keywords: | Blocked By:
Blocking: | Operating System: All
----------------------------+-------------------------------
Changes (by olly):
* milestone: 1.4.2 => 1.4.3
Comment:
> I'm still not familiar with the tests policies, so I'm not really sure
at all what tests would be desirable for this.
From HACKING:
{{{
2) Make sure the tests are right
--------------------------------
* If you're adding a feature, also add feature tests for it. These both
ensure that the feature isn't broken to start with and detect if later
changes stop it working as intended.
* If you've fixed a bug, make sure there's a regression test which
fails on the existing code and succeeds after your changes.
* Make sure all existing tests continue to pass.
}}}
For a new API feature we want to be confident that it works as documented
(and as intended!) and that future changes to the code which might break
it will be caught before they get into a release, and ideally before they
are even merged. Perfection in this regard is that the test suite
exercises every line of the source code - that can be hard to fully
achieve without disproportionate effort, but the other extreme of a new
API method with no test coverage whatsoever is really not acceptable.
I was hoping to merge at least some of this before 1.4.2, but I'd like to
get 1.4.2 into the next Debian stable release which basically means we're
out of time, so I'm bumping the milestone.
--
Ticket URL: <https://trac.xapian.org/ticket/710#comment:4>
Xapian <https://xapian.org/>
Xapian
More information about the Xapian-tickets
mailing list