fix invalid memory access - by making test trader keep smart ptrs of its bid+request portfolios.  This allows its bid/req associated with the folios to stay in scope until the tests is done