Do they actually review their source code at Embarcadero or why does almost every fix introduce a new bug?

-
Do they actually review their source code at Embarcadero or why does almost every fix introduce a new bug?

http://qc.embarcadero.com/wc/qcmain.aspx?d=111048
http://qc.embarcadero.com/wc/qcmain.aspx?d=116833

Comments

  1. Dorin DuminicaJune 25, 2013 at 6:41 AM

    maybe because there's little to no interest? I don't think the devs like it either, but they have no choice, most of their time must be spent on shiny things that never deliver what they promise, so, they go with the wind....

    ReplyDelete
  2. Bill MeyerJune 25, 2013 at 6:45 AM

    Could it be... no unit testing?  TimeInRange is one of those things which is deceptively simple, and yet the simplest thing about it is to introduce a defect. Precisely the sort of thing for which unit tests are so valuable.

    ReplyDelete
  3. Nick HodgesJune 25, 2013 at 7:04 AM

    At least the first one is marked fixed.

    ReplyDelete
  4. Nick HodgesJune 25, 2013 at 7:06 AM

    I wish they'd consider something like open sourcing their unit tests so people could contribute.  

    I did a lot of work on the unit tests for DateUtils (I even wrote a series of blog posts about it), but I don't remember testing those methods.

    ReplyDelete
  5. Stefan GlienkeJune 25, 2013 at 7:06 AM

    Bill Meyer I blame Nick for not writing enough of them :p
    http://blogs.embarcadero.com/nickhodges/2010/03/10/39369

    Nick Hodges Yeah, and while fixing that dev also got rid of these "useless" TimeOf calls, eh? ;)

    ReplyDelete
  6. Nick HodgesJune 25, 2013 at 7:09 AM

    Hey!  I could have sworn I wrote tests for every routine, though of course, I might not  have tested those particular scenarios.

    I did close 26 bug reports as a result of all the improvements I made after testing that unit.  

    But not quite enough, I guess.  ;-)

    ReplyDelete
  7. Lars FosdalJune 25, 2013 at 7:33 AM

    I wonder if they fixed the lacking SecondsSinceMidnight div 2 bug in the automatic version number yet.  Such as small fix, with such a big effect.

    ReplyDelete
  8. Bill MeyerJune 25, 2013 at 12:13 PM

    Lars Fosdal Now you're just being picky. ;)

    ReplyDelete
  9. Arioch TheJuly 29, 2013 at 11:25 AM

    Those who write documentation never run their sampels in the real IDE. Those who translate MS VC++ code do not run their translation in even once - they are wekk satsisfied that the RTL merely compiles. Now you ask if QA guys are obliged to be testing their fixes - like if they they didn't earn the same privileges as rest of the team. How could you even think of such inequality!

    ReplyDelete

Post a Comment