Use absolute import paths in test code

[kumar303]

The test code has some gnarly imports for local library files like:

import build from '../../../src/cmd/build'

We should fix this so it can be:

import build from 'cmd/build'

We fixed this in addons-frontend by using better-npm-run and setting NODE_PATH for each command.

[kumar303]

I'm nominating this as a good first bug because there is a clear example (linked above) to show how to do it.

[lavish205]

@kumar303 I'm new to web-ext and I would love to work on this bug.

[kumar303]

Hi @lavish205 and welcome! Let me know if you have any questions.

[lavish205]

I'm confused how to proceed with it. Can you please provide more details so that I can start working on it.

[kumar303]

I took a closer look at this and actually I don't think it's a good first bug to work on. The problem that I found is that all of the test code needs to be put in subdirectory, like tests/testlib/ so that we can more safely add it to the NODE_PATH for importing. This is a complicated change to get right because it involves updating some obscure webpack and karma configs.

Sorry about the confusion. I would suggest taking a look at what else may interest you under the good first bug label.