Menu

#315 Can not set PROPERTY_IDENTITY_COLUMN_FILTER

v2.3.*
closed-fixed
None
2.7.0
5
2020-02-23
2011-04-06
Anonymous
No

DatabaseConfig default constructor does not set a default value so convertIfNeeded throws a IllegalArgumentException.

Discussion

  • Luc Feys

    Luc Feys - 2019-02-25

    Wondering if there is any update on this? This issue already seems quite old, but I still ran into it today with DbUnit version 2.5.4 and 2.6.
    Is anybody aware of a workaround? Because the default IdentityColumnFilter does not work for me. Would like to try the extended one or write my own if needed.

     

    Last edit: Luc Feys 2019-02-25
    • Jeff Jensen

      Jeff Jensen - 2019-02-25

      Hi, since you have a broken scenario, can you make a test(s) that reproduce the problem and attach as a patch or open a git merge request? If you know the fix, please include that too. I will review, apply, and publish as 2.6.1-SNAPSHOT with the fix.

       
  • Jeff Jensen

    Jeff Jensen - 2019-02-25
    • status: open --> pending
    • assigned_to: Jeff Jensen
    • Fixed Release: --> (not fixed)
    • Found Release: --> v2.3.*
     
  • Luc Feys

    Luc Feys - 2019-02-26

    OK. Had a look at it. Seems like there is already a test for this feature, which actually fails if it is activated. But currently it is disabled in the mssql41 profile because IDENTITY_INSERT is added to the list of unsupported features. Which is why this went unnoticed so far.
    So I will try to come up with a fix.

     
    • Jeff Jensen

      Jeff Jensen - 2019-02-26

      Good find, thank you for investigating.

       
  • Luc Feys

    Luc Feys - 2019-02-26

    I have a fix that works for me. Created a merge request.

     
    • Jeff Jensen

      Jeff Jensen - 2019-03-01

      Thanks! Please see MR comments.

       
  • Jeff Jensen

    Jeff Jensen - 2019-03-22
    • status: pending --> open-fixed
    • Fixed Release: (not fixed) --> 2.6.1
     
  • Jeff Jensen

    Jeff Jensen - 2019-03-22
    • status: open-fixed --> closed-fixed
     

Log in to post a comment.