`addEvent` and `recordError` should take `clock` argument

[slashmo]

addEvent and its related recordError method should take a clock argument, defaulting to DefaultTracerClock(). This would allow us to mock the clock similarly to how we can already do it when calling Span.end().

[ktoso]

Agreed, on it 👍

[ktoso]

in addEvent no, because the Event creation will have it:

    func addEvent(_ event: SpanEvent)

    public init<Instant: TracerInstant>(name: String,
                                    at instant: @autoclosure () -> Instant,
                                    attributes: SpanAttributes = [:])
    {

record error is a good call tho, added:

    func recordError<Instant: TracerInstant>(_ error: Error,
                                             attributes: SpanAttributes,
                                             at instant: @autoclosure () -> Instant)

    public func recordError(_ error: Error, attributes: SpanAttributes) {
        self.recordError(error, attributes: attributes, at: DefaultTracerClock.now)
    }