Hello Jason, 2014/1/7 Jason Herr <[email protected]>
> Lukas, > > Since this only hits test code (nothing else I do sets the id column on an > insert), I can work around it. I was really trying to do some archaeology > and understand what caused the change and why. I was unable to pinpoint it > in the notes, > Yes, unfortunately, these things had been quite subtle already in the past. > My tests were based on my old architecture which didn't do nice things > like return the id column upon insert/update. So, my tests can be > refactored without having a workaround. > > While you're thinking about 4.0 behavior, I don't really mind either way > whether the column is there or not upon insert. What is important is that > unless the value is set, I'd rather the DataType control its content and > use the default value. I'm not at the point in understanding the codebase > to be able to suggest a fix. > Yes, that is what is supposed to happen. Note that DDL's DEFAULT values weren't really considered up until recently. There is an open issue related to this topic, here: https://github.com/jOOQ/jOOQ/issues/2698 > Finally, what are your plans for jOOQ-test? Are you planning to add it as > a submodule and have it run all the tests? OR are you thinking about > moving it under jOOQ into src/test/*? I wouldn't mind playing with getting > that under CI in my spare time, but wanted to head in the right direction. > Tests help me grok a codebase better than docs anyhow. > I'm working on this right now: https://github.com/jOOQ/jOOQ/issues/682 Currently, the jOOQ integration tests have to be triggered manually from Eclipse. Now that Travis-CI is working, I'm finally refactoring the integration tests, such that they can be run easily on any machine - at least the H2 / HSQLDB / Derby / SQLite ones that do not require an external database installation. Before that, I don't see too many options for contributions Cheers Lukas -- You received this message because you are subscribed to the Google Groups "jOOQ User Group" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. For more options, visit https://groups.google.com/groups/opt_out.
