1037450 - Don't package test data in GaiaTest for PyPI

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::UI Tests, defect)

Description

[James Graham [:jgraham]]

Many parts of GaiaTest are very reusable for multiple projects working with the Gaia UI. However it is entirely unnecessary for any of these consumers to get the Gaia UI tests themselves. AIUI the UI tests are run directly from Git, so there is no need for them to be packaged with the python code for that reason.

Comment 1

[James Graham [:jgraham]]

diff --git a/tests/python/gaia-ui-tests/MANIFEST.in b/tests/python/gaia-ui-tests/MANIFEST.in
index e95b4a1..13b54a8 100644
--- a/tests/python/gaia-ui-tests/MANIFEST.in
+++ b/tests/python/gaia-ui-tests/MANIFEST.in
@@ -1,4 +1,4 @@
-recursive-include gaiatest/atoms *.js
-recursive-include gaiatest/resources *.*
 exclude MANIFEST.in
 include requirements.txt
+exclude gaiatest/tests
+exclude gaiatest/resources/*


I'm not sure this is the right change (in particular I'm not sure about resources/; there seem to be a couple of places that assume it exists, but I don't know how important they are outside of the Gaia UI Test context), but it leaves me with a 73 kB package rather than a 2.7Mb one.

Comment 2

[Zac C (:zac)]

The one place I can see where resources is used is within GaiaTestCase which is a test specific class (mostly) so I think it's fine to package without the resources folder.

Comment 3

[James Graham [:jgraham]]

Attachment: Don't package test data or resources in zip file

Comment 4

[Zac C (:zac)]

Comment on attachment 8457949 [details] [review]
Don't package test data or resources in zip file

f+ from me, but I think Dave is the man to r? this as he is the main man for gaiatest releases.

Comment 5

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 8457949 [details] [review]
Don't package test data or resources in zip file

We provide a helper for accessing the resources from tests, and although this won't break our tests, it's possible users of gaiatest are currently dependent on these resources. The removal of the atoms almost certainly be a problem though.

Comment 6

[Dave Hunt [:davehunt] [he/him] ⌚BST]

Comment on attachment 8457949 [details] [review]
Don't package test data or resources in zip file

Landed in:
https://github.com/mozilla-b2g/gaia/commit/7fbcfbacf286c2a8e41a3a96b6d82f1541880617