Some methods on EventBuffer (get_command_events, get_successful_command_execution) are a combination of search and mutate methods; these make it hard to reason about test behavior and in most cases the mutability isn't actually needed.  We should remove them in favor of immutable methods and have additional explicit mutability for when it's really necessary.