Skip to content

Commit eccbcc4

Browse files
committed
Remove browser warning as all browsers now support reading the rendered HTML
1 parent 122ff10 commit eccbcc4

File tree

5 files changed

+181
-333
lines changed

5 files changed

+181
-333
lines changed

RegressionRunner.html

Lines changed: 181 additions & 186 deletions
Original file line numberDiff line numberDiff line change
@@ -1,203 +1,198 @@
11
<!DOCTYPE html>
22
<html>
3-
<head>
4-
<title>Regression Runner</title>
5-
<meta charset="utf-8"/>
3+
<head>
4+
<title>Regression Runner</title>
5+
<meta charset="utf-8" />
66

7-
<script src="csscritic.js"></script>
7+
<script src="csscritic.js"></script>
88

9-
<script>
10-
window.onload = function() {
11-
csscritic
12-
.addReporter(csscritic.NiceReporter())
13-
.component('Passing test')
14-
.add({
15-
desc: 'should show the rendered page',
16-
url: 'test/ui/niceReporterPassedTest.html'
17-
})
18-
.add({
19-
desc: 'should underline the title',
20-
url: 'test/ui/niceReporterPassedTest.html',
21-
hover: '.titleLink'
22-
})
23-
.add({
24-
desc: 'should switch to "DOM view" on toggle button click',
25-
url: 'test/ui/niceReporterPassedTestRealView.html'
26-
})
27-
.add({
28-
desc: 'should emboss "DOM view" button on hover',
29-
url: 'test/ui/niceReporterPassedTest.html',
30-
hover: '.toggleView'
31-
})
32-
.add({
33-
desc: 'should switch back from "DOM view"',
34-
url: 'test/ui/niceReporterPassedTestRealViewBack.html'
35-
})
9+
<script>
10+
window.onload = function () {
11+
csscritic
12+
.addReporter(csscritic.NiceReporter())
13+
.component("Passing test")
14+
.add({
15+
desc: "should show the rendered page",
16+
url: "test/ui/niceReporterPassedTest.html",
17+
})
18+
.add({
19+
desc: "should underline the title",
20+
url: "test/ui/niceReporterPassedTest.html",
21+
hover: ".titleLink",
22+
})
23+
.add({
24+
desc: 'should switch to "DOM view" on toggle button click',
25+
url: "test/ui/niceReporterPassedTestRealView.html",
26+
})
27+
.add({
28+
desc: 'should emboss "DOM view" button on hover',
29+
url: "test/ui/niceReporterPassedTest.html",
30+
hover: ".toggleView",
31+
})
32+
.add({
33+
desc: 'should switch back from "DOM view"',
34+
url: "test/ui/niceReporterPassedTestRealViewBack.html",
35+
})
3636

37-
.component('Failing test')
38-
.add({
39-
desc: 'should show reference and actual rendering',
40-
url: 'test/ui/niceReporterFailingTest.html'
41-
})
42-
.add({
43-
desc: 'should fade reference image on hover',
44-
url: 'test/ui/niceReporterFailingTest.html',
45-
hover: '.failed.comparison .referenceImageContainer'
46-
})
47-
.add({
48-
desc: 'should show a tooltip on hover',
49-
url: 'test/ui/niceReporterFailingTest.html',
50-
hover: '.failed.comparison'
51-
})
52-
.add({
53-
desc: 'should hide reference rendering and deactive button on accept',
54-
url: 'test/ui/niceReporterFailingTestAccepted.html'
55-
})
56-
.add({
57-
desc: 'should deal with growing page size',
58-
url: 'test/ui/niceReporterFailingTestWithGrowingContent.html',
59-
hover: '.failed.comparison .referenceImageContainer'
60-
})
61-
.add({
62-
desc: 'should disable accept button in "real view"',
63-
url: 'test/ui/niceReporterFailingTestRealView.html'
64-
})
37+
.component("Failing test")
38+
.add({
39+
desc: "should show reference and actual rendering",
40+
url: "test/ui/niceReporterFailingTest.html",
41+
})
42+
.add({
43+
desc: "should fade reference image on hover",
44+
url: "test/ui/niceReporterFailingTest.html",
45+
hover: ".failed.comparison .referenceImageContainer",
46+
})
47+
.add({
48+
desc: "should show a tooltip on hover",
49+
url: "test/ui/niceReporterFailingTest.html",
50+
hover: ".failed.comparison",
51+
})
52+
.add({
53+
desc: "should hide reference rendering and deactive button on accept",
54+
url: "test/ui/niceReporterFailingTestAccepted.html",
55+
})
56+
.add({
57+
desc: "should deal with growing page size",
58+
url: "test/ui/niceReporterFailingTestWithGrowingContent.html",
59+
hover: ".failed.comparison .referenceImageContainer",
60+
})
61+
.add({
62+
desc: 'should disable accept button in "real view"',
63+
url: "test/ui/niceReporterFailingTestRealView.html",
64+
})
6565

66-
.component('Missing reference')
67-
.add({
68-
desc: 'should show the rendered page',
69-
url: 'test/ui/niceReporterMissingReference.html'
70-
})
71-
.add({
72-
desc: 'should highlight the button on hover',
73-
url: 'test/ui/niceReporterMissingReference.html',
74-
hover: '.accept'
75-
})
76-
.add({
77-
desc: 'should highlight the button on activation',
78-
url: 'test/ui/niceReporterMissingReference.html',
79-
active: '.accept'
80-
})
81-
.add({
82-
desc: 'should deactivate the accept button on click',
83-
url: 'test/ui/niceReporterMissingReferenceAccepted.html'
84-
})
66+
.component("Missing reference")
67+
.add({
68+
desc: "should show the rendered page",
69+
url: "test/ui/niceReporterMissingReference.html",
70+
})
71+
.add({
72+
desc: "should highlight the button on hover",
73+
url: "test/ui/niceReporterMissingReference.html",
74+
hover: ".accept",
75+
})
76+
.add({
77+
desc: "should highlight the button on activation",
78+
url: "test/ui/niceReporterMissingReference.html",
79+
active: ".accept",
80+
})
81+
.add({
82+
desc: "should deactivate the accept button on click",
83+
url: "test/ui/niceReporterMissingReferenceAccepted.html",
84+
})
8585

86-
.component('Broken test')
87-
.add({
88-
desc: 'should show a warning',
89-
url: 'test/ui/niceReporterBrokenUrl.html'
90-
})
86+
.component("Broken test")
87+
.add({
88+
desc: "should show a warning",
89+
url: "test/ui/niceReporterBrokenUrl.html",
90+
})
9191

92-
.component('Additional parameters')
93-
.add({
94-
desc: 'should prefer the description over the url as a title',
95-
url: 'test/ui/niceReporterPassedTestWithDescription.html'
96-
})
97-
.add({
98-
desc: 'should show an additional parameter',
99-
url: 'test/ui/niceReporterTestWithASingleParameter.html'
100-
})
101-
.add({
102-
desc: 'should show two additional paramaters',
103-
url: 'test/ui/niceReporterTestWithMultipleParameters.html'
104-
})
92+
.component("Additional parameters")
93+
.add({
94+
desc: "should prefer the description over the url as a title",
95+
url: "test/ui/niceReporterPassedTestWithDescription.html",
96+
})
97+
.add({
98+
desc: "should show an additional parameter",
99+
url: "test/ui/niceReporterTestWithASingleParameter.html",
100+
})
101+
.add({
102+
desc: "should show two additional paramaters",
103+
url: "test/ui/niceReporterTestWithMultipleParameters.html",
104+
})
105105

106-
.component('Starting run')
107-
.add({
108-
desc: 'should show an opaque rendering of the reference image',
109-
url: 'test/ui/niceReporterStartingTest.html'
110-
})
111-
.add({
112-
desc: 'should handle a missing reference image',
113-
url: 'test/ui/niceReporterStartingTestNoReference.html'
114-
})
115-
.add({
116-
desc: 'should update header according to progress',
117-
url: 'test/ui/niceReporterOngoingRun.html'
118-
})
106+
.component("Starting run")
107+
.add({
108+
desc: "should show an opaque rendering of the reference image",
109+
url: "test/ui/niceReporterStartingTest.html",
110+
})
111+
.add({
112+
desc: "should handle a missing reference image",
113+
url: "test/ui/niceReporterStartingTestNoReference.html",
114+
})
115+
.add({
116+
desc: "should update header according to progress",
117+
url: "test/ui/niceReporterOngoingRun.html",
118+
})
119119

120-
.component('Missing resources')
121-
.add({
122-
desc: 'should warn about render errors for a passing test',
123-
url: 'test/ui/niceReporterPassedTestMissingResources.html'
124-
})
125-
.add({
126-
desc: 'should warn about render errors for a failing test',
127-
url: 'test/ui/niceReporterFailingTestMissingResources.html'
128-
})
129-
.add({
130-
desc: 'should warn about render errors for a missing reference',
131-
url: 'test/ui/niceReporterMissingReferenceMissingResources.html'
132-
})
120+
.component("Missing resources")
121+
.add({
122+
desc: "should warn about render errors for a passing test",
123+
url: "test/ui/niceReporterPassedTestMissingResources.html",
124+
})
125+
.add({
126+
desc: "should warn about render errors for a failing test",
127+
url: "test/ui/niceReporterFailingTestMissingResources.html",
128+
})
129+
.add({
130+
desc: "should warn about render errors for a missing reference",
131+
url: "test/ui/niceReporterMissingReferenceMissingResources.html",
132+
})
133133

134-
.component('Accept all')
135-
.add({
136-
desc: 'should show the "Accept all" button for three failing tests',
137-
url: 'test/ui/niceReporterMultipleMissingReference.html'
138-
})
139-
.add({
140-
desc: 'should deactivate all accept buttons on accept',
141-
url: 'test/ui/niceReporterMultipleMissingReferencesAccepted.html'
142-
})
134+
.component("Accept all")
135+
.add({
136+
desc: 'should show the "Accept all" button for three failing tests',
137+
url: "test/ui/niceReporterMultipleMissingReference.html",
138+
})
139+
.add({
140+
desc: "should deactivate all accept buttons on accept",
141+
url: "test/ui/niceReporterMultipleMissingReferencesAccepted.html",
142+
})
143143

144-
.component('Selection')
145-
.add({
146-
desc: 'should only render selected tests',
147-
url: 'test/ui/niceReporterSelectedTest.html'
148-
})
149-
.add({
150-
desc: 'should show "Accept all" button for three selected tests',
151-
url: 'test/ui/niceReporterSelectedTestWithMultipleFailing.html'
152-
})
144+
.component("Selection")
145+
.add({
146+
desc: "should only render selected tests",
147+
url: "test/ui/niceReporterSelectedTest.html",
148+
})
149+
.add({
150+
desc: 'should show "Accept all" button for three selected tests',
151+
url: "test/ui/niceReporterSelectedTestWithMultipleFailing.html",
152+
})
153153

154-
.component('Component')
155-
.add({
156-
desc: 'should show a headline',
157-
url: 'test/ui/niceReporterComponentHeadline.html'
158-
})
159-
.add({
160-
desc: 'should show multiple headlines',
161-
url: 'test/ui/niceReporterComponentHeadlineMultiple.html'
162-
})
154+
.component("Component")
155+
.add({
156+
desc: "should show a headline",
157+
url: "test/ui/niceReporterComponentHeadline.html",
158+
})
159+
.add({
160+
desc: "should show multiple headlines",
161+
url: "test/ui/niceReporterComponentHeadlineMultiple.html",
162+
})
163163

164-
.component('Other')
165-
.add({
166-
desc: 'should render two different test outcomes',
167-
url: 'test/ui/niceReporterTwoTests.html'
168-
})
169-
.add({
170-
desc: 'should show a browser warning',
171-
url: 'test/ui/niceReporterMissingBrowserSupport.html'
172-
})
173-
.add({
174-
desc: 'should highlight the test when targeted through a link',
175-
url: 'test/ui/niceReporterProgressBarTarget.html'
176-
})
177-
.add({
178-
desc: 'should show an empty setup',
179-
url: 'test/ui/niceReporterNoTests.html'
180-
})
164+
.component("Other")
165+
.add({
166+
desc: "should render two different test outcomes",
167+
url: "test/ui/niceReporterTwoTests.html",
168+
})
169+
.add({
170+
desc: "should highlight the test when targeted through a link",
171+
url: "test/ui/niceReporterProgressBarTarget.html",
172+
})
173+
.add({
174+
desc: "should show an empty setup",
175+
url: "test/ui/niceReporterNoTests.html",
176+
})
181177

182-
.component('Fixed header')
183-
.add({
184-
desc: 'should show a large header on initial scroll',
185-
url: 'test/ui/niceReporterLargeHeader.html'
186-
})
187-
.add({
188-
desc: 'should show a small header on scroll',
189-
url: 'test/ui/niceReporterSmallHeader.html'
190-
})
191-
.add({
192-
desc: 'should show the large header when hovering on scroll',
193-
url: 'test/ui/niceReporterFixedHeaderHover.html',
194-
hover: 'header'
195-
})
178+
.component("Fixed header")
179+
.add({
180+
desc: "should show a large header on initial scroll",
181+
url: "test/ui/niceReporterLargeHeader.html",
182+
})
183+
.add({
184+
desc: "should show a small header on scroll",
185+
url: "test/ui/niceReporterSmallHeader.html",
186+
})
187+
.add({
188+
desc: "should show the large header when hovering on scroll",
189+
url: "test/ui/niceReporterFixedHeaderHover.html",
190+
hover: "header",
191+
})
196192

197-
.execute();
198-
}
199-
</script>
200-
</head>
201-
<body>
202-
</body>
193+
.execute();
194+
};
195+
</script>
196+
</head>
197+
<body></body>
203198
</html>

0 commit comments

Comments
 (0)