D339: tests: verify that peer instances only expose interface members

2017-08-15 Thread indygreg (Gregory Szorc)
This revision was automatically updated to reflect the committed changes. Closed by commit rHGb70029f355a3: tests: verify that peer instances only expose interface members (authored by indygreg). REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D339?vs=764&id

D339: tests: verify that peer instances only expose interface members

2017-08-13 Thread indygreg (Gregory Szorc)
indygreg added a comment. In https://phab.mercurial-scm.org/D339#5376, @durin42 wrote: > Might also be neat to have a test to assert the peer and legacy peer interfaces don't overlap? This can be done as a follow-up IMO. REPOSITORY rHG Mercurial REVISION DETAIL https://phab.

D339: tests: verify that peer instances only expose interface members

2017-08-11 Thread durin42 (Augie Fackler)
durin42 accepted this revision. durin42 added a comment. This revision is now accepted and ready to land. Might also be neat to have a test to assert the peer and legacy peer interfaces don't overlap? REPOSITORY rHG Mercurial REVISION DETAIL https://phab.mercurial-scm.org/D339 To: indyg

D339: tests: verify that peer instances only expose interface members

2017-08-10 Thread indygreg (Gregory Szorc)
indygreg created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY Our abstract interfaces are more useful if we guarantee that implementations conform to certain rules. Namely, we want to ensure that objects implementing inter