sailfin
  1. sailfin
  2. SAILFIN-1965

4 Ericsson tests failed with timeout errors

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 2.0
    • Fix Version/s: milestone 1
    • Component/s: tests
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

      Description

      After the integration of latest tests from Ericsson, there are 4 tests which
      have been failing in B30 and B31. Even with reruns, these tests are still
      failing. In B29, these tests have passed.

      InviteTester => test3_84
      IssuesTester => test_issue_60
      ScenariosTester => test7_49, test7_104

      Error message for test3_84
      ***************************
      Test case failed. Failures detected when executing the
      scenarios/invite/slingshot/3_84.xml file. Details: Failure number: 1 in Step: a8
      Actor: TMa3_84 Message: Timeout before receiving a response (waited 5000 ms in
      transaction transa4). Probable Cause:

      junit.framework.AssertionFailedError: Test case failed. Failures detected when
      executing the scenarios/invite/slingshot/3_84.xml file. Details: Failure number:
      1 in Step: a8 Actor: TMa3_84 Message: Timeout before receiving a response
      (waited 5000 ms in transaction transa4). Probable Cause: at
      com.ericsson.eas.common.BaseTestCase.runSlingshotScenario(BaseTestCase.java:224)
      at
      com.ericsson.eas.common.BaseTestCase.runSlingshotScenario(BaseTestCase.java:154)
      at com.ericsson.eas.ft.invite.test.InviteTester.test3_84(InviteTester.java:248)

        Activity

        Hide
        binod added a comment -

        Ok, here is the analysis of 7_104. This is a new test, that is added on 27th.

        Why it was passing?
        The test sleeps for a second in the doNotify() after sending the 200OK. After one second, it sends a BYE
        to the client. But while it is sleeping, the client is ALSO sending a BYE and hence the dialog would have
        ended by the time servlet sends the BYE. Now, sending the BYE from servlet would fail because of the
        illegalstateexception. The servlet swallows that exception and no one knows about that. Every thing is
        fine.

        Why it is failing now?
        After our change to allow only one message per dialog, the BYE from the client would wait until the
        doNotify() completes. That means, the BYE from the client is processed only AFTER the BYE is sent form
        the server. Once the BYE is sent from the server, the test fails.

        I do not know why the servlet is trying to send the BYE. There doesnt seem to be any reason. If I
        remove the sleep and sending the BYE from the servlet, everything passes. I will assign the test back to
        Jagadesh to take a look at this test.

        Please fix 3_84 and 7_104 tests. I have checked in the fixes to other tests.

        Show
        binod added a comment - Ok, here is the analysis of 7_104. This is a new test, that is added on 27th. Why it was passing? The test sleeps for a second in the doNotify() after sending the 200OK. After one second, it sends a BYE to the client. But while it is sleeping, the client is ALSO sending a BYE and hence the dialog would have ended by the time servlet sends the BYE. Now, sending the BYE from servlet would fail because of the illegalstateexception. The servlet swallows that exception and no one knows about that. Every thing is fine. Why it is failing now? After our change to allow only one message per dialog, the BYE from the client would wait until the doNotify() completes. That means, the BYE from the client is processed only AFTER the BYE is sent form the server. Once the BYE is sent from the server, the test fails. I do not know why the servlet is trying to send the BYE. There doesnt seem to be any reason. If I remove the sleep and sending the BYE from the servlet, everything passes. I will assign the test back to Jagadesh to take a look at this test. Please fix 3_84 and 7_104 tests. I have checked in the fixes to other tests.
        Hide
        jagadesh added a comment -

        Thanks Binod for analysing and fixing the test issues based on the latest
        changes in the server code. I will take a look at the 2 pending tests and do as
        per your suggestions.

        Show
        jagadesh added a comment - Thanks Binod for analysing and fixing the test issues based on the latest changes in the server code. I will take a look at the 2 pending tests and do as per your suggestions.
        Hide
        prasads added a comment -

        Added a keyword for this issue, since there is no checkin required.

        Show
        prasads added a comment - Added a keyword for this issue, since there is no checkin required.
        Hide
        meenap added a comment -

        test_issue_60 and test7_49 have passed on RH with Build 31d.

        Show
        meenap added a comment - test_issue_60 and test7_49 have passed on RH with Build 31d.
        Hide
        prasads added a comment -

        Temporarily marking these issues as P4, for the Sailfin 2.0 release. The
        priority will be restored for the next release.

        Show
        prasads added a comment - Temporarily marking these issues as P4, for the Sailfin 2.0 release. The priority will be restored for the next release.

          People

          • Assignee:
            jagadesh
            Reporter:
            meenap
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated: