Remove @todo and workaround in Cookie ResourceTestTrait

Converted the patch to an MR. Given #18 was green, we might have just fixed this elsewhere without removing the @todo.

Since all I did was convert the patch to an MR, I think I can RTBC this. Updated the issue title and summary.

Tagging that this came up as a random triage target today in #bugsmash, which is where the new life came from.

Also reviewed the MR code and test results. RTBC++.

Hah, this was random #bugsmash ~1.5 years ago for patch #17, but wasn't tagged then, either. 😅

Saving credits for everyone.

Minor summary edit, but I didn't bother with the full default template since none of the other headings are relevant here.

Backported to 10.2.x as a test only change.

Committed and pushed eed7ea3a50 to 11.x and 0d4a5883d4 to 10.3.x and 4d9ec8d74a to 10.2.x. Thanks!

Automatically closed - issue fixed for 2 weeks with no activity.