testsuite failure #264

Closed
strk opened this Issue Mar 22, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@strk
Contributor

strk commented Mar 22, 2013

✖ 2 of 134 tests failed:

  1) Rendering should render identity.mml correctly:
     Error: failed: xml /home/src/node/modules/carto/test/rendering/identity.result in json form does not match expected result:
/home/src/node/modules/carto/test/rendering/identity-actual.json (actual)
/home/src/node/modules/carto/test/rendering/identity-expected.json (expected)
      at /home/src/node/modules/carto/test/rendering.test.js:46:31

  2) Rendering should render rastercolorizer.mml correctly:
     Error: rastercolorizer.mss:3:2 Unrecognized rule: raster-colorizer-default-mode
rastercolorizer.mss:4:2 Unrecognized rule: raster-colorizer-default-color
rastercolorizer.mss:5:2 Unrecognized rule: raster-colorizer-epsilon
rastercolorizer.mss:6:2 Unrecognized rule: raster-colorizer-stops
      at Object.carto.Parser.parser.parse.root.toList.env.error (/home/src/node/modules/carto/lib/carto/parser.js:235:55)
      at Rule.tree.Rule.toXML (/home/src/node/modules/carto/lib/carto/tree/rule.js:56:20)
      at Definition.tree.Definition.symbolizersToXML (/home/src/node/modules/carto/lib/carto/tree/definition.js:137:46)
      at Definition.tree.Definition.toXML (/home/src/node/modules/carto/lib/carto/tree/definition.js:198:29)
      at /home/src/node/modules/carto/lib/carto/tree/style.js:23:27

A diff between identity-actual.json and identity-expected.json:

7,30d6
<             "Style": [
<                 {
<                     "name": "world",
<                     "filter-mode": "first",
<                     "__order__": 1,
<                     "Rule": [
<                         {
<                             "__order__": 2,
<                             "Filter": [
<                                 {
<                                     "__order__": 3,
<                                     "text": "([foo] = 'bar')"
<                                 }
<                             ],
<                             "MarkersSymbolizer": [
<                                 {
<                                     "width": "5",
<                                     "__order__": 4
<                                 }
<                             ]
<                         }
<                     ]
<                 }
<             ],
35c11
<                     "__order__": 5,

---
>                     "__order__": 1,
38c14
<                             "__order__": 6,

---
>                             "__order__": 2,
44c20
<                             "__order__": 7,

---
>                             "__order__": 3,
48c24
<                                     "__order__": 8,

---
>                                     "__order__": 4,
53c29
<                                     "__order__": 9,

---
>                                     "__order__": 5,
@tmcw

This comment has been minimized.

Show comment
Hide comment
@tmcw

tmcw Mar 22, 2013

Contributor

Intended, read last commit. 60661b6

Contributor

tmcw commented Mar 22, 2013

Intended, read last commit. 60661b6

@tmcw tmcw closed this Mar 22, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment