[AF-788][BUG] Migrate CommonJS modules

[svizzari]

✌️ 🐛 🐛

/cc @zendesk/vegemite

Description

The work carried out in #13 didn't solve all the issues with migrating Common JS. @nick-f tested and found that require statements in app.js weren't being rewritten to include a relative path (to lib). With this change require("module.js") in app.js, will become require("./lib/module.js") in legacy_app.js when migrated.

Tasks

• [x] Write tests (May as well add tests for this step now) 5f8eda4

References

• JIRA: https://zendesk.atlassian.net/browse/AF-788

Risks

• N/A

[svizzari]

@zendesk/vegemite 💇‍♂️ This now has tests, and is ready for review

[svizzari]

@danielbreves 💇