cordova-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Filip Maj (JIRA)" <>
Subject [jira] [Commented] (CB-12901) Remove lazy loading of node modules from cordova tooling
Date Mon, 12 Jun 2017 15:08:00 GMT


Filip Maj commented on CB-12901:

Unfortunately the simple removal of this functionality prompted a refactor, which in turn
prompted tests to fail! So this branch is a larger work in progress in further refactoring
the {{platform}} command, and laying out a structure for its unit tests.

> Remove lazy loading of node modules from cordova tooling
> --------------------------------------------------------
>                 Key: CB-12901
>                 URL:
>             Project: Apache Cordova
>          Issue Type: Task
>          Components: cordova-common, cordova-lib
>            Reporter: Filip Maj
>            Assignee: Filip Maj
> There are various {{addProperty*}} methods that execute {{require}} at runtime. While
this improves load time and only loads modules that are necessary, it also makes it harder
to test, as spying and mocking is more difficult.
> The 300ms load time improvement, in the best case, is not worth the difficulty in writing
unit tests.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message