search

nruth / show_me_the_cookies

Cookie manipulation for Capybara (view, delete, …)
161 stars 44 forks source link

Delete cookie doesn't delete cookies with CAPITAL keys #42

Open vasilakisfil opened 7 years ago

vasilakisfil commented 7 years ago

Is this intentional? https://github.com/nruth/show_me_the_cookies/blob/2f81986cd0925aa5d63083ca04f1f1fbd92b1b80/lib/show_me_the_cookies/adapters/rack_test.rb#L26

nruth commented 6 years ago

It certainly looks weird doesn't it. If there's no comment or test for it let's remove it because it's probably just leftover from when I was first playing around with getting something to work in my own app.

vasilakisfil commented 6 years ago

will fix this and #31 on Thursday.