Details

    • Type: Task
    • Status: Closed
    • Priority: Major
    • Resolution: Cannot Reproduce
    • Affects Version/s: None
    • Fix Version/s: 2.4
    • Component/s: Code Standards
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required

      Description

      In WebDriverLegacyITBase.java, in assertJgrowlText, instead of an empty if followed by an else, there should just be an if statement. I'm referring to:
      if (jGrowl.getText() != null && jGrowl.getText().contains(""))

      So inverting that if will do the trick.

      However, I don't think this was written correctly to begin with. If jGrowl.getText() is null, it will then hit this code:
      sb.append(jGrowl.getText()).append("\n");

      And that will produce a null pointer exception.

        Attachments

          Activity

          Hide
          eghm-kuali-m Erik Meade added a comment -

          Already done in a previous commit.

          Show
          eghm-kuali-m Erik Meade added a comment - Already done in a previous commit.

            People

            • Assignee:
              eghm-kuali-m Erik Meade
              Reporter:
              matthew.wuertz Matthew Wuertz (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: