Only run exact cosmos EntityTooLarge tests on a real cosmos and add additional tests for emulator#37028
Conversation
|
I haven't ran against a real cosmos db, but am pretty sure they should now all succeed on either real or fake cosmos db. Tests might not fail when internal cosmos db sdk serialization logic changes, but I think we can be relatively sure we will remain within limits still |
|
Yes, these all pass against real Cosmos |
|
@AndriySvyryd Should I readd the deleted (flakish) tests aswell with [CosmosCondition(CosmosCondition.IsNotEmulator)] and a base value incremented with 7? (Can do that tomorrow) |
Sounds good |
…on a real cosmos db with CosmosCondition.IsNotEmulator
|
It might be a good idea to run this against a real cosmos before merging to be sure. @AndriySvyryd Could you do that easily? |
AndriySvyryd
left a comment
There was a problem hiding this comment.
Not at the moment. In any case it won't be worse than the current situation
Only run exact cosmos EntityTooLarge tests on a real cosmos and add additional tests for emulator