importing gaiatest package is throwing AttributeError

RESOLVED WONTFIX

Status

defect
--
critical
RESOLVED WONTFIX
4 years ago
Last year

People

(Reporter: yogendra, Unassigned)

Tracking

unspecified
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(blocking-b2g:2.2?, b2g-v2.2 affected, b2g-v2.2r affected)

Details

Attachments

(1 attachment)

Reporter

Description

4 years ago
Importing gaia test package resulting in the following error:
"gaia\tests\python\gaia-ui-tests\gaiatest\file_manager.py", line 12, in <module> class GaiaFileManager(object): File "gaia\tests\python\gaia-ui-tests\gaiatest\file_manager.py", line 16, in GaiaFileManager def __init__(self, device, log_level=mozlog.ERROR): AttributeError: 'module' object has no attribute 'ERROR'

Apparently mozlog was updated [1] and caused breakage in some test suites as well. [2]
Very likely that’s the issue here and I guess we need to switch gaia ui tests into using Python logging instead of mozlog like it’s done in other moz test suites [3] [4]
i.e.
1. import logging instead of mozlog
2. Change mozlog instances to logging
 
[1] Bug 1014760 - [mozlog] Move structured logging out of 'structured' submodule and into the top-level one
[2] Bug 1185479 - Can't execute the UI tests because of mozlog
[3] Bug 1189714 - use python logging instead of mozlog
[4] https://hg.mozilla.org/mozilla-central/diff/5c2f87f734be/testing/mozbase/mozdevice/sut_tests/runtests.py
Reporter

Updated

4 years ago
blocking-b2g: --- → 2.2?
Fallout from bug 1185501?
Flags: needinfo?(pyang)
I guess patch from bug 1185501 solve this problem.  Can you pull latest master and try again?  Thanks.
Flags: needinfo?(pyang) → needinfo?(yogendra)
Reporter

Comment 3

4 years ago
Patch from bug 1185501 is not present in v2.2 master. Please port the patch to v2.2 where gaiatest is broken.
Paul, see comment 3.
Flags: needinfo?(pyang)
requirement of 2.2 shows

boto>=2.32.1
marionette_client-mozilla-b2g37_v2_2
mozdevice>=0.34
moztest>=0.6
requests
treeherder-client==1.5

I guess we can pin the version or actually port the patch.  Martijn, what do you think?
Flags: needinfo?(pyang)

Updated

4 years ago
Flags: needinfo?(martijn.martijn)
I guess pinning the version is just fine.
Flags: needinfo?(martijn.martijn)
ni myself for reminder.
Flags: needinfo?(pyang)
tested in my local repo and it looks ok.
Flags: needinfo?(pyang)
Reporter

Updated

4 years ago
Flags: needinfo?(yogendra)
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.