-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow filtering on pass/fail and browser #57
Conversation
d58ed1e
to
9f18f79
Compare
9f18f79
to
ff0b1d5
Compare
position: sticky; | ||
top: 0; | ||
z-index: 10; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I stole most of these styles from the Daylight Site. They work for now and we can tweak them later!
`]; | ||
constructor() { | ||
super(); | ||
this._filterBrowsers = data.browsers.map(b => b.name); | ||
this._filterStatus = data.numFailed > 0 ? FILTER_STATUS.FAILED : FILTER_STATUS.ALL; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Default to showing only failed tests, assuming there are any.
src/server/report/test-result.js
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This fixes a bug Stacey noticed where the images always grow to fill the viewport. Now they'll fill the viewport but not exceed the actual size of the screenshots.
@@ -52,7 +63,6 @@ function flattenResults(session, browserData, fileData) { | |||
const testData = fileData.tests.get(testName); | |||
if (!t.passed) { | |||
browserData.numFailed++; | |||
fileData.numFailed++; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Counting it here caused it to think that a test failed 3 times if it failed in all 3 browsers, which isn't quite what I want to surface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking great!
Co-authored-by: Stacey Van Herk <[email protected]>
🎉 This PR is included in version 0.15.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Introduces a fixed sidebar that contains filters for "status" and "browser". Still just basic styling, but starting to add a bit of colour!