isStringAnExpression() in planner.cpp needs testing

    • Type: Task
    • Resolution: Unresolved
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: None
    • None
    • Atlas Streams
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      Part of our "all parsers need unit tests" effort, given the multiple recent CVEs that have come out of parser bugs.

       

      Also, could this be a shared utility function, used by multiple teams? I'm pretty sure I've seen "first character is $" checks elsewhere in our code.

            Assignee:
            Calvin Nix
            Reporter:
            Ronald Steinke
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: