Closed Bug 1444994 Opened 7 years ago Closed 7 years ago

Update Hybrid-content documentation with testing details

Categories

(Toolkit :: Telemetry, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: gfritzsche, Assigned: janerik)

Details

Attachments

(1 file, 2 obsolete files)

Bug 1444114, comment 4 has details how to test HCT integrations outside of production systems. We should integrate this into the documentation.
Priority: P2 → P1
Assignee: nobody → jrediger
Status: NEW → ASSIGNED
Comment on attachment 8961778 [details] [diff] [review] Update Hybrid-content documentation with testing details Review of attachment 8961778 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/components/telemetry/docs/collection/hybrid-content.rst @@ +276,5 @@ > + > +The respective ``hc_telemetry`` permission needs to be set before any pages on that host load the ``HybridContentTelemetry-lib.js`` file. > + > +For manual testing the permission can be set from the Browser Console. > +For automated testing in test frameworks with privileged access the permission can be set in the respective ``head.js``. Could you include examples of using these APIs to run tests manually and automatically? I think they would be most illustrative.
Attachment #8961778 - Flags: review?(chutten)
Attachment #8961778 - Attachment is obsolete: true
Comment on attachment 8962256 [details] [diff] [review] Update Hybrid-content documentation with testing details Review of attachment 8962256 [details] [diff] [review]: ----------------------------------------------------------------- ::: toolkit/components/telemetry/docs/collection/hybrid-content.rst @@ +278,5 @@ > + > +Manual testing > +-------------- > + > +After starting the browser, open the Browser Console (Web Developer -> Browser Console). Does this need a "Tools -> " to start?
Attachment #8962256 - Flags: review?(chutten)
Added the "Tools ->" as suggested.
Attachment #8962359 - Flags: review?(chutten)
Attachment #8962256 - Attachment is obsolete: true
Comment on attachment 8962359 [details] [diff] [review] Update Hybrid-content documentation with testing details Review of attachment 8962359 [details] [diff] [review]: ----------------------------------------------------------------- LGTM
Attachment #8962359 - Flags: review?(chutten) → review+
Keywords: checkin-needed
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/6c7b42072d08 Update Hybrid-content documentation with testing details. r=chutten
Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: