moved files around to fit new structure

merged black scholes tests
removed files with duplicated functionality
Is it even worth having this method in the trait? I don't think we need a Valuer hierarchy mirroring the Instrument one. Might we get more reuse if we just have a Valuer class that takes any instrument and then (potentially) passes the calculation onto somewhere else?