Uploaded image for project: 'Apache Cordova'
  1. Apache Cordova
  2. CB-12901

Remove lazy loading of node modules from cordova tooling

    Details

    • Type: Task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: cordova-common, cordova-lib
    • Labels:
      None

      Description

      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.

        Attachments

          Activity

            People

            • Assignee:
              filmaj Filip Maj
              Reporter:
              filmaj Filip Maj
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: