Re: [IndexedDB-tests] IndexedDB: Opera's tests (#63)

Nah, I don't really see the point. I see you've merget #64 already now, but I guess I can change this one to overwrite those tests now that they're there.

You see, these tests all follow the same pattern, and that's a good thing. It makes it easier to understand one test and then get them all. MS' tests are more barebones, so they might be a bit easier to understand at first, but these ones also test much more. Because they set up fail listeners for events that really shouldn't be fired.

So these tests found some errors in never-released presto IndexedDB that wouldn't have been caught otherwise.


So it's not mostly only on formatting. :) Also, the tests were made after I did a review of the original MS tests, and said everything that was wrong, so yes, the tests you just merged were of a much higher quality than those I originally ported over. All credits should be kept in the files however, with respect to original author, but I think my way makes the tests better and more useful.

View on GitHub: https://github.com/w3c/web-platform-tests/pull/63#issuecomment-29693183

Received on Tuesday, 3 December 2013 09:03:57 UTC