getConfigFromAnySource('unit.engine.tap.command') only pulls first command

[elsigh]

If you have two tap tests this line won't work, it pulls the first matching config line that matches "unit.engine.tap.command".

Why not pull from actual runtime config for this class and just call the option "command" - seems cleaner, no?

[elsigh]

Oh I see, each engine doesn't have its own config. Maybe it should. ;0

[reu]

Yeah, that's a hard problem to solve. The current implementation used to work, and Arcanist would pick the last defined one, looks like this behaviour was changed in more recent versions. Perhaps we should add something like a engine "name" when using MultiTestEngine, and then we could scope each config by this "name".

[elsigh]

It seems like the instance object of an engine should be able to hold a localized configuration somehow.