Closed Bug 1288682 Opened 8 years ago Closed 8 years ago

Add tests for MessageContext

Categories

(L20n :: JS Library, defect)

Product:
L20n
Component:
JS Library
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: stas, Assigned: stas)

References

Details

(Whiteboard: [gecko-l20n]) 

Add tests for MessageContext which can be run in the l20n.js repo.
Blocks: 1288684
Whiteboard: [gecko-l20n]
No longer blocks: 1288639
Assignee: nobody → stas
Status: NEW → ASSIGNED
Depends on: 1306262
I've been working on this over the last week in https://github.com/l20n/l20n.js/pull/149.
I landed 118 new tests in https://github.com/l20n/l20n.js/commit/d8c312fb3c5bf203cf6b6bfea248283201582956.

5 of them are marked as known failures for now.  I filed the following bugs for them:

Bug 1307116 - When passed as external args, convert JS types to FTL types
Bug 1307120 - Accept complex types (e.g. members) as arguments to FTL Functions
Bug 1307124 - Gracefully handle wrong argument types passed into FTL Functions
Bug 1307126 - Protect the FTL Resolver against the billion laughs attack
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.