Skip to content

Commit 822a862

Browse files
committed
(Temporarily) Fixed issue with Get Cookies test
The `Get Cookies` test started to fail. It was written to expect just the another and test cookies but was also receiving the far_future cookie. I'm a bit unsure as to why this wasn't failing sooner. I would have expected this cookie to also appear. So not sure what was happening here. Git blame shows this has been unchanged for a while. I have updated the expected return value from get cookies. I do see some possible improvements. The previous check allowed for the order to be swapped. With three that is a bit more complicated. Instead of going into a complex check for any order I am going to leave this till we see that appear.
1 parent 5c113c6 commit 822a862

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

atest/acceptance/keywords/cookies.robot

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@ Library DateTime
99
*** Test Cases ***
1010
Get Cookies
1111
${cookies}= Get Cookies
12-
Should Match Regexp ${cookies}
13-
... ^(test=seleniumlibrary; another=value)|(another=value; test=seleniumlibrary)$
12+
Should Be Equal As Strings ${cookies}
13+
... another=value; test=seleniumlibrary; far_future=timemachine
1414

1515
Get Cookies As Dict
1616
${cookies}= Get Cookies as_dict=True

0 commit comments

Comments
 (0)