Comment 1 for bug 1209594

Revision history for this message
Richard Gomes (frgomes) wrote :

It's not necessary to separate different exchange into different classes.
On the other hand, it would be interesting to make the code easier to read. At the moment there are several dates added to some exchanges and/or removed from other exchanges. This make the code to be confused. It would be better to repeat blocks but keep one exchange totally separated from each other in each test case.

I've done a code review and sorted out some issues but it's still necessary to organize better the test cases, separating different exchanges into different test cases.