feat(config): Add configuration for adding javascript version.

[Moumi]

Add the configuration to add a javascript version tag to the loaded scripts. Only applied when the Firefox browser is run.

Closes karma-runner#1719

[Moumi]

Please pull this fix to your own computer and perform grunt test. Make sure that all tests pass. If not, please provide which tests fail and the errors given. Also look if additional comments are needed in the code.

[liuyiran13]

Get it , I will do the tests soon.

[wesleyvanderlee]

My tests are failing due to this error:

TypeError: Cannot read property 'replace' of null

Seems to me that an object is null where it isn't supposed to be. Maybe you need to place an additional check or enforce object initialization.

See here for full test output.

Regards,

Wesley van der Lee

[Moumi]

@wesleyvanderlee It seems that the test failures you are getting are not due to my change. Something is probably wrongly setup in your development environment.

[ludai0106]

Tests are passed with the changes !

[liuyiran13]

My test also failed and get the error like this:

TypeError: Cannot read property 'replace' of null

[Moumi]

Let's look into this when we are together. I might be able to help you fix your environment.

[Moumi]

This is still awaiting a proper review. Please review!

[wesleyvanderlee]

I reconfigured my environment according to this site. Still get the same errors. Because everyone encounters those errors, I don't think it fully relies on the way our environment is set up. We should look together into this..

[Moumi]

@wesleyvanderlee @liuyiran13 Why not create a VM with a Linux version on it and test it out?

[Moumi]

Did not seem to squash the commits for this PR together, therefore I created a new PR #8. Please review that one.

This one is now closed.