To do this, we'll start collecting warnings as logged to the console by React.js and compare the number of warnings at the end of the run with the amount we expect.

Why only react? Why not any warnings?

Also, I think you should be able to do something similar for the ui-showcase. See the setTimeout in the DOMContentLoaded listener at the bottom of ui-showcase.jsx

Comment on attachment 8627607 [details] [diff] [review] Patch v2: count the amount of warnings we generate Since I made bug 1171940 depend on this bug, I'd like to get this in before that.

Comment on attachment 8627607 [details] [diff] [review] Patch v2: count the amount of warnings we generate Review of attachment 8627607 [details] [diff] [review]: ----------------------------------------------------------------- Looks good. You might just want to give it a quick check on Windows & Linux on try - just in case there's a different count for some reason. Up to you. ::: browser/components/loop/test/desktop-local/index.html @@ +95,5 @@ > }); > }); > > + describe("Unexpected Warnings Check", function() { > + it("should not log more warnings than we expect", function() { Minor nit, might be clearer as "should long only the warnings we expect".

> chai.expect(caughtWarnings.length).to.eql(128); This shouldn't have been a strict equality check because but a `<=` imo. Reducing warnings means this test will fail. I addressed this in the NPS patch and set it to `<=` and also lowered the number after fixing some parts of the code.

(In reply to Andrei Oprea [:andreio] from comment #10) > > chai.expect(caughtWarnings.length).to.eql(128); > > This shouldn't have been a strict equality check because but a `<=` imo. > Reducing warnings means this test will fail. I addressed this in the NPS > patch and set it to `<=` and also lowered the number after fixing some parts > of the code. No! If we set this to be lesser or equal, we'll never get a grasp on what the current level of warnings is. If you provide patches that lower the amount of warnings, you can simply update the amount of warnings in the respective index.html file and we'll _know_ that we're actively lowering this amount closer to 0.