Remove browser/components/enterprisepolicies/tests/live test suite
Categories
(Firefox Enterprise :: Client, task)
Tracking
(Not tracked)
People
(Reporter: janika, Assigned: janika)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [size=1])
Attachments
(1 file)
The tests in browser/components/enterprisepolicies/tests/ test each policy and whether it is applied correctly. Hereby it doesn't add value to test each policy once applied via a local JSON file, and a second time by fetching policies remotely. Especially as the time it gets applied - which is at startup - doesn't change.
What can add value is to test whether each policy is still applied correctly when applied after startup. This is a bigger effort and needs to be spec-ed out first. So for now we can remove the live test suite.
|
||
Comment 1•28 days ago
|
||
|
||
Comment 2•28 days ago
|
||
(In reply to Janika Neuberger [:janika] from comment #0)
The tests in
browser/components/enterprisepolicies/tests/test each policy and whether it is applied correctly. Hereby it doesn't add value to test each policy once applied via a local JSON file, and a second time by fetching policies remotely. Especially as the time it gets applied - which is at startup - doesn't change.What can add value is to test whether each policy is still applied correctly when applied after startup. This is a bigger effort and needs to be spec-ed out first. So for now we can remove the
livetest suite.
/me is crying, but the argument holds
|
|
||
Comment 3•27 days ago
|
||
Authored by https://github.com/1rneh
https://github.com/mozilla/enterprise-firefox/commit/577a0580c563e8f00992da6938e91644b4d9ab3f
[enterprise-main] Bug 2034434 - Remove enterprisepolicies live test suite (#781)
Description
•