On Wed, 2017-04-19 at 10:34 -0700, Bill Deegan wrote:
> Plenty of existing end to end tests function with mocked out external
> tools.
> Any reason this can't be done for your pull request?

The current pull request doesn't need such a thing as it doesn't
include the dub tool. I believe the tests in place do cover the new
builder. I am just doing the documentation side of things – the D tool
XML files were inadequate, so as requested I am getting them 'ship
shape' as part of this pull request.

I'll address the issue of executable discovery in the D tools as a
separate pull request, to avoid interference with the addition of the
new builder.

When I get to the dub pull request, I hope to have tests in place that
cover all this – I am just getting onto doing proper tests for the dub
tool. All the email exchanges have been very helpful on that front.

-- 
Russel.
=============================================================================
Dr Russel Winder      t: +44 20 7585 2200   voip: sip:russel.win...@ekiga.net
41 Buckmaster Road    m: +44 7770 465 077   xmpp: rus...@winder.org.uk
London SW11 1EN, UK   w: www.russel.org.uk  skype: russel_winder

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Scons-dev mailing list
Scons-dev@scons.org
https://pairlist2.pair.net/mailman/listinfo/scons-dev

Reply via email to