postcss / autoprefixer Public
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
Add test for at-rules #1272
Add test for at-rules #1272
Conversation
test/autoprefixer.test.js
Outdated
it('ignore prefix in vendor at rules', () => { | ||
let input = read('at-rules') | ||
let output = read('at-rules.out') | ||
let result = postcss([autoprefixer({ cascade: false })]).process(input) |
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.
autoprefixer
will use current browsers and could change output in the future
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 fixed that.
test/autoprefixer.test.js
Outdated
@@ -547,6 +547,13 @@ it('works with CSS Modules', () => { | |||
postcss([autoprefixer()]).process(':export { selectors: _1q6ho_2 }').css | |||
}) | |||
|
|||
it('ignore prefix in vendor at rules', () => { | |||
let input = read('at-rules') |
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.
We have a shortcut to do this test. You can find it in hacks
secltion in this test file.
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 fixed that.
test/cases/at-rules.out.css
Outdated
body { | ||
-webkit-appearance: none; | ||
-moz-appearance: none; | ||
appearance: none; |
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 recommend enabling cascade
here to make test readable and more standard
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 fixed that.
No description provided.