-
Notifications
You must be signed in to change notification settings - Fork 270
Labels
priority:p1High priority but not blocking. Causes major but not critical loss of functionality SLA <=7daysHigh priority but not blocking. Causes major but not critical loss of functionality SLA <=7daystype:bugA broken experienceA broken experience
Milestone
Description
In #2041 and #2042 I've fixed issues where due to the layering of the serialization code, a V2 serialization method ended up calling a v3 serialization method when it shouldn't have.
We should implement use tests controlling for that with spies where we effectively:
- spy all v3 methods for objects that have a v2 serialization method
- populate a large DOM, trying to cover all API surface
- run v2 serialization
- control that no v3/3.1 method has been called
We should probably implement the other way around (v3 doesn't call v2) and deserialization tests as well
Metadata
Metadata
Assignees
Labels
priority:p1High priority but not blocking. Causes major but not critical loss of functionality SLA <=7daysHigh priority but not blocking. Causes major but not critical loss of functionality SLA <=7daystype:bugA broken experienceA broken experience