[
https://issues.apache.org/jira/browse/CB-12901?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16046656#comment-16046656
]
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: https://issues.apache.org/jira/browse/CB-12901
> 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
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
