Issue 4719 - Edge packager does not generate an info module

packagerEdge.py

Index: packagerEdge.py
===================================================================
--- a/packagerEdge.py
+++ b/packagerEdge.py
@@ -171,6 +171,11 @@
 
     files['manifest.json'] = packagerChrome.createManifest(params, files)
 
+    if (metadata.has_option('general', 'backgroundScripts') and
+        'lib/info.js' in metadata.get('general', 'backgroundScripts').split()

[kzar, 2016/12/13 09:19:22]

IMO this looks kind of ugly, I don't like how the second clause does not end
with the "and". I'll leave it up to Vasily and Sebastian to decide anyway, (FWIW
I checked and Tox does pass.)

[Vasily Kuznetsov, 2016/12/13 18:49:58]

I would have computed the background scripts list beforehand to make the
condition shorter and more readable. Maybe something like this:

if metadata.has_option('general', 'backgroundScripts'):
    bg_scripts = metadata.get('general', 'backgroundScripts').split()
    if 'lib/info.js' in bg_scripts and 'lib/info.js' not in files:
        # yada yada yada

Not a big deal though, LGTM from me anyway.

[Sebastian Noack, 2016/12/15 13:58:06]

I agree with Vasily here. Also, is it even necessery to check for |'lib/info.js'
in files|? On the other hand, the code is consistent with what we have in
packagerChrome.py. So if we change it here we should also change it their for
consistency. No strong opinion from my side.

Also, we probably should adapt the tests to cover this code path and make sure
that the file got added.

[Oleksandr, 2016/12/22 02:53:24]

I think touching packagerChrome.py would require a separate issue. Omitting that
here.
Done.

+            and 'lib/info.js' not in files):
+        files['lib/info.js'] = packagerChrome.createInfoModule(params)
+
     move_files_to_extension(files)
 
     if metadata.has_section('appx_assets'):