[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