Menu

#32 utassert2.ieqminus - missing parenthesis

Closed-Fixed
utAssert (5)
[other - please specify]
[Unknown]
Medium
2.2.2
2014-05-04
2009-04-02
No

I tried to use utAssert.eqquery and added as against_this_in parameter value SELECT 1 FROM dual UNION ALL SELECT 2 FROM dual. It is propagated to procedure ieminus() where the UNION ALL makes it fail. I can make a workaround when I add parenthesis around the parameter but I think that it makes much more sense when ieminus() do that.
BTW I am willing to join the developers team if you allow me to. BR

Related

Feature Request Archive (Use GitHub now): #5
News: 2014/05/new-release-v222

Discussion

  • Paul Walker

    Paul Walker - 2013-10-12
    • Status: open --> Open
    • Priority: 5 --> Medium
     
  • Paul Walker

    Paul Walker - 2013-10-12
    • Fixed in version: --> [none]
     
  • Svyatoslav Chatchenko

    I did changes, that ticket creator had mentioned
    attachements:
    ut_assert2.pkb.zip - patch
    ut_utassert.zip - test package. just two simple tests. test 1 - reveal bug. test 2 - show workarround
    testresult_before_changes.zip - log for test run before changes
    testresult_after_changes.zip - log for test run after changes

     
  • Paul Walker

    Paul Walker - 2014-04-06

    Great, thanks. I'll setup an SVN branch with your changes as soon as I get chance

     
  • Paul Walker

    Paul Walker - 2014-04-06
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -1,2 +1,2 @@
    -I tried to use utAssert.eqquery and added as against\_this\_in parameter value "SELECT 1 FROM dual UNION ALL SELECT 2 FROM dual". It is propagated to procedure ieminus\(\) where the UNION ALL makes it fail. I can make a workaround when I add parenthesis around the parameter but I think that it makes much more sense when ieminus\(\) do that.
    +I tried to use `utAssert.eqquery` and added as `against_this_in` parameter value `SELECT 1 FROM dual UNION ALL SELECT 2 FROM dual`. It is propagated to procedure `ieminus()` where the `UNION ALL` makes it fail. I can make a workaround when I add parenthesis around the parameter but I think that it makes much more sense when `ieminus()` do that.
     BTW I am willing to join the developers team if you allow me to. BR
    
    • assigned_to: Paul Walker
    • Seen in version: --> [other - please specify]
    • Oracle version: --> [Unknown]
     
  • Paul Walker

    Paul Walker - 2014-04-21

    [r160] - Made a development branch
    [r161] - Applied patch as provided by Svyatoslav Chatchenko above

     

    Related

    Commit: [r160]
    Commit: [r161]


    Last edit: Paul Walker 2014-04-21
  • Paul Walker

    Paul Walker - 2014-04-21

    I have tested this and it appears to work as expected. Merging with trunk ready for next release [r162]

     

    Related

    Commit: [r162]

  • Paul Walker

    Paul Walker - 2014-04-21
    • status: Open --> Closed-Fixed
    • Fixed in version: [none] --> [next]
     

Log in to post a comment.

Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.