Tapestry 5
  1. Tapestry 5
  2. TAP5-11

CookiesImplTest does specify a domain cookie with a domain not prefixed with a . (dot)

    Details

    • Type: Task Task
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 5.2, 5.1.0.7, 5.0.19
    • Fix Version/s: 5.2.0, 5.1.0.8, 5.0.20
    • Component/s: None
    • Labels:
      None

      Description

      CookiesImplTest does specify a domain cookie with a domain not prefixed with a . (dot) where the RFC 2109 explicitly says it should always do.

        Activity

        Hide
        Massimo Lusetti added a comment -

        The statement at line 144: assertEquals(cookies.get(0), expectedCookie); doesn't take into account the domain part of the cookie.
        Actually is the equals method of Cookie which probably doesn't, anyhow even commenting out line 142: expectedCookie.setDomain("fobar.com"); doesn't make the test fail.

        So it's probably better to add something like: assertEquals(cookies.get(0).getDomain(), expectedCookie.getDomain()); as final assert for testing domain wide cookies.

        Show
        Massimo Lusetti added a comment - The statement at line 144: assertEquals(cookies.get(0), expectedCookie); doesn't take into account the domain part of the cookie. Actually is the equals method of Cookie which probably doesn't, anyhow even commenting out line 142: expectedCookie.setDomain("fobar.com"); doesn't make the test fail. So it's probably better to add something like: assertEquals(cookies.get(0).getDomain(), expectedCookie.getDomain()); as final assert for testing domain wide cookies.

          People

          • Assignee:
            Ulrich Stärk
            Reporter:
            Massimo Lusetti
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development