You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Tests are missing in the JavaScript layer entirely. Note that C++ tests will not cover the JavaScript, and the sloppy development operation in JavaScript has led to configuration faults that have led to blame at the engine (mozilla/firefox-translations#51) despite getting it correct in #269, due to manual movement human error in place of continuous testing and integration.
I'm currently capturing possibilities in #330, but this is outside my assigned tasks - so it will have to come out of spare time.
@jerinphilip As you are not responsible for any JS/WASM work, please feel free to not do it. Nobody is asking you to do it.
The feature was added to the engine and it was a major API change. So C++ tests are kind of needed. There is no assignee on this issue. I created it so that we could close the original one (which unblocked the integration in extension) and not forget about tests.
filed because of #345 (comment) and #345 (comment).
The text was updated successfully, but these errors were encountered: