Code review comment for lp:~adiroiban/pocket-lint/pocket-lint-css-lint

Revision history for this message
Adi Roiban (adiroiban) wrote :

The test input texts are a bit cryptic.
Maybe I could put them as module constants and define them using multiline string... but then they will be far from the actual test.

Since they are not that long, I think that string concatenation on multiple lines could also solve this problem

So instead of

    text ='rule1{st1\n}\n@font-face {\n src: url("u\n u"); \n }\nr2{st2}'

It would be

    text = (
        'rule1{st1\n'
        '}\n'
        '@font-face {\n'
        ' src: url("u\n u"); \n'
        ' }\n'
        'r2{st2}')

« Back to merge proposal