Bug 1407695 - Allows cookie 'expiry' to be optional; r?ato draft
authorØyvind Strømmen <insula.ventus@gmail.com>
Mon, 23 Oct 2017 22:23:10 +0200
changeset 685557 6de8067d9992fcea12364563c8a02db0fab00cce
parent 685556 77658db2e3dbb1ce91a0702cc044eac14ea470aa
child 685558 75fde7092ec087c1c83c71347e4dab94d58b359a
push id85969
push userbmo:insula.ventus@gmail.com
push dateTue, 24 Oct 2017 20:00:56 +0000
reviewersato
bugs1407695
milestone58.0a1
Bug 1407695 - Allows cookie 'expiry' to be optional; r?ato When getting a session cookie we expect 'expiry' to be either null, or to be missing. MozReview-Commit-ID: JmSfrR0ypya
testing/web-platform/tests/webdriver/tests/cookies/get_named_cookie.py
--- a/testing/web-platform/tests/webdriver/tests/cookies/get_named_cookie.py
+++ b/testing/web-platform/tests/webdriver/tests/cookies/get_named_cookie.py
@@ -1,12 +1,12 @@
 from tests.support.inline import inline
 from tests.support.fixtures import clear_all_cookies
 
-def test_get_named_cookie(session, url):
+def test_get_named_session_cookie(session, url):
     session.url = url("/common/blank.html")
     clear_all_cookies(session)
     session.execute_script("document.cookie = 'foo=bar'")
 
     result = session.transport.send("GET", "session/%s/cookie/foo" % session.session_id)
     assert result.status == 200
     assert isinstance(result.body["value"], dict)
 
@@ -21,17 +21,17 @@ def test_get_named_cookie(session, url):
     assert isinstance(cookie["path"], basestring)
     assert "domain" in cookie
     assert isinstance(cookie["domain"], basestring)
     assert "secure" in cookie
     assert isinstance(cookie["secure"], bool)
     assert "httpOnly" in cookie
     assert isinstance(cookie["httpOnly"], bool)
     assert "expiry" in cookie
-    assert isinstance(cookie["expiry"], (int, long))
+    assert cookie.get("expiry") is None
 
     assert cookie["name"] == "foo"
     assert cookie["value"] == "bar"
 
 def test_duplicated_cookie(session, url):
     session.url = url("/common/blank.html")
     clear_all_cookies(session)
     create_cookie_request = {