1042858 - Remove unit tests template from minified JS

Status: RESOLVED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P5)

Description

[Christopher Van Wiemeersch [:cvan]]

https://marketplace.cdn.mozilla.net/media/fireplace/js/include.js?b=1406052479313 contains the source to `templates/tests.html`:

    ,e["tests.html"]=function(){function e(e,t,n,r,o){var a=null,i=null,s="";try{s+='<section class="main infobox">\n<div>\n<h2>Unit Tests</h2>\n<progress value="0"></progress>\n<table>\n<tr>\n<th>Started</th>\n<th>Passed</th>\n<th>Failed</th>\n</tr>\n<tr>\n<td id="c_started">0</td>\n<td id="c_passed">0</td>\n<td id="c_failed">0</td>\n</tr>\n</table>\n<ol class="tests"></ol>\n</div>\n</section>\n<script type="text/javascript" src="/tests/apps.js"></script>\n<script type="text/javascript" src="/tests/cache.js"></script>\n<script type="text/javascript" src="/tests/l10n.js"></script>\n<script type="text/javascript" src="/tests/mobilenetwork.js"></script>\n<script type="text/javascript" src="/tests/navigation.js"></script>\n<script type="text/javascript" src="/tests/models.js"></script>\n<script type="text/javascript" src="/tests/requests.js"></script>\n<script type="text/javascript" src="/tests/rewriters.js"></script>\n<script type="text/javascript" src="/tests/urls.js"></script>\n<script type="text/javascript" src="/tests/utils.js"></script>',o(null,s)}catch(l){o(r.handleError(l,a,i))}}return{root:e}}()

We don't need this in our minified source.

Comment 1

[Trishul]

Hi Christopher, can you guide me to resole this bug?

Comment 2

[Trishul]

Hi Wil, any updates?

Comment 3

[Christopher Van Wiemeersch [:cvan]]

Hi Trishul,

Apologies for the delayed response. The docs for the frontend code are here:

    http://marketplace-frontend.readthedocs.org/

The code that includes the `tests/` AMD modules is here:

    https://github.com/mozilla/fireplace/blob/723eba8a/src/media/js/routes.js#L69

There's a Gulp task that gets runs during our deploys (`gulp build`, which calls `gulp js_build`):

    https://github.com/mozilla/marketplace-gulp/blob/7fbedbf1/index.js#L399-L400
    https://github.com/mozilla/marketplace-gulp/blob/7fbedbf1/index.js#L269-L271

And this is the line where the JS paths are included for the optimised, minified JS bundle:

    https://github.com/mozilla/marketplace-gulp/blob/7fbedbf1/index.js#L256

Fellow Marketplace developer, Kevin Ngo, has a good understanding of the architecture here. And I would follow up with him if you have any questions.

Comment 4

[Trishul]

HI Christopher,
I think I need a bit more understanding of the marketplace frontend architecture. 
Also about gulp tasks.

Thanks.

Comment 5

[Kristján Oddsson [:koddsson]]

I hope nobody minds that I made a PR on github to fix this => https://github.com/mozilla/marketplace-gulp/pull/13

Comment 6

[Kevin Ngo [:ngoke]]

Hey Kristjan, you can resolve this bug by removing src/templates/tests.html from mozilla/fireplace :)

Comment 7

[Kristján Oddsson [:koddsson]]

Made another PR => https://github.com/mozilla/fireplace/pull/1074

Comment 8

[Kevin Ngo [:ngoke]]

https://github.com/mozilla/fireplace/pull/1074