Uploaded image for project: 'MyFaces Core'
  1. MyFaces Core
  2. MYFACES-4557

feature/MYFACES-4466: the last ajax integration test fails

Details

    • Task
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 4.0.0-RC5
    • None
    • None

    Description

      The test is derived from my internal protocol tests, which all pass on this codebase.

      However it might test functionality not in the spec but in the original implementation.

      There have been minor changes on how error reporting is handled, given we have newer browsers now

      (ie errors are now logged not alerted anymore for instance)

      As far as I can see the test tests for a client error condition. I will need to investigate this next week. 

      Given I have tests on error conditions in my newer integration tests as well, the test might be even outdated!

      It is not critical and very likely just a test issue itself, because the TCKs pass and my newer integration tests as well!

      But I want this covered so that I can pick up a small investigation sometime next week!

       

      Attachments

        Activity

          Is this fixed?  I don't see any PRs / commits: https://github.com/apache/myfaces/search?q=MYFACES-4557

          Should the fix version stay RC5 or be RC6?

          volosied Volodymyr Siedlecki added a comment - Is this fixed?  I don't see any PRs / commits: https://github.com/apache/myfaces/search?q=MYFACES-4557 Should the fix version stay RC5 or be RC6?
          werpu Werner Punz added a comment - - edited

          RC5... this is fixed... the integration tests pass!

          To be more precise, this was fixed along the way when we merged the new scripts in. The test case simply was wrong.

           

          werpu Werner Punz added a comment - - edited RC5... this is fixed... the integration tests pass! To be more precise, this was fixed along the way when we merged the new scripts in. The test case simply was wrong.  

          People

            werpu Werner Punz
            werpu Werner Punz
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: