Ignasi, have you by chance timed the differences on the tests? I can run them here, but just wanted to see if you had any data first. It is great to get a speed increase on just about anything! ;)
On Wed, Dec 18, 2013 at 7:08 AM, Ignasi Barrera <[email protected]>wrote: > Yes, removing this it speeds up a bit the tests, and also improves > some tests that perform operations on big files (such as an upload). > I think we could safely remove that (given that neither unit nor live > tests use those options), but I'd prefer to ask first for opinions. > > On 18 December 2013 16:04, Andrew Phillips <[email protected]> wrote: > >> Out of curiosity, Does anyone know why we are explicitly configuring > >> the memory for the "integration" tests [1], but not for the "unit" and > >> "live" tests? > >> > >> Can we just remove that line? > > > > > > Do you have any information on what effect that change with have? Does it > > speed things up? Are there any problems that the line is causing for you > > right now..? > > > > ap >
