enable mozApps debug logging when running tests

RESOLVED FIXED in mozilla33

Status

Core Graveyard
DOM: Apps
RESOLVED FIXED
4 years ago
2 months ago

People

(Reporter: myk, Assigned: myk)

Tracking

unspecified
mozilla33

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
Created attachment 8453427 [details] [diff] [review]
enable mozApps debug logging when running tests

Now that we enable mozApps debug logging with a pref, we should enable it when running tests.
Attachment #8453427 - Flags: review?(fabrice)
Yes please!
Attachment #8453427 - Flags: review?(fabrice) → review+
(Assignee)

Comment 2

4 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/ebd23dac96c4
https://hg.mozilla.org/mozilla-central/rev/ebd23dac96c4
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
Why does this need to be on when running all tests? It generates a large amount of console spew that makes trying to debug tests more annoying than it needs to be
Flags: needinfo?(myk)
(Assignee)

Comment 5

3 years ago
(In reply to Dave Townsend [:mossop] from comment #4)
> Why does this need to be on when running all tests? It generates a large
> amount of console spew that makes trying to debug tests more annoying than
> it needs to be

It doesn't need to be on, but it's been quite helpful for debugging issues with the DOM app registry and the various app runtimes.

Is there some spew that particularly annoyed you recently? It's quite possible that we're overlogging, and perhaps we can reduce the spew enough to stop being a bother while still getting useful info for debugging issues.
Flags: needinfo?(myk) → needinfo?(dtownsend)
Here is the top of a test log:

-*- Webapps.jsm : Saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
-*- Webapps.jsm : Saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
-*- Webapps.jsm : Success saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
-*- Webapps.jsm : Success saving /var/folders/sv/3hcj25m92fs2szqn3hhhkwtw0000gp/T/tmp5NJUm5.mozrunner/webapps/webapps.json
Shumway is registered
0 INFO *** Start BrowserChrome Test Results ***
1 INFO checking window state
2 INFO TEST-START | browser/base/content/test/general/browser_tab_dragdrop.js
3 INFO Entering test 
SystemMessageCache: init-*-*- Langpacks: getAdditionalLanguages http://example.org/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://test1.example.org/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://test1.example.org:8000/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://sub1.test1.example.org/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_priv.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_cert.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_cert.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_inst.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://example.com/manifest_csp_priv.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://acertified.com/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages https://aprivileged.com/manifest.webapp
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://127.0.0.1:8888/sample.manifest
-*-*- Langpacks: Languages found: ({langs:{}})
-*-*- Langpacks: getAdditionalLanguages http://127.0.0.1:8888/sample.manifest
-*-*- Langpacks: Languages found: ({langs:{}})

The Langpacks lines also repeat whenever a content process restarts which in some of the tests I work on happens frequently.

I guess I don't understand why you can't just enable the apps logging for the apps tests.
Flags: needinfo?(dtownsend)

Updated

2 months ago
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.