{"action":"created","issue":{"url":"https://api.github.com/repos/raml-org/raml-js-parser-2/issues/367","repository_url":"https://api.github.com/repos/raml-org/raml-js-parser-2","labels_url":"https://api.github.com/repos/raml-org/raml-js-parser-2/issues/367/labels{/name}","comments_url":"https://api.github.com/repos/raml-org/raml-js-parser-2/issues/367/comments","events_url":"https://api.github.com/repos/raml-org/raml-js-parser-2/issues/367/events","html_url":"https://github.com/raml-org/raml-js-parser-2/issues/367","id":166715329,"number":367,"title":"Additional Properties must be allowed in responses","user":{"login":"eleonoraortega","id":5630419,"avatar_url":"https://avatars.githubusercontent.com/u/5630419?v=3","gravatar_id":"","url":"https://api.github.com/users/eleonoraortega","html_url":"https://github.com/eleonoraortega","followers_url":"https://api.github.com/users/eleonoraortega/followers","following_url":"https://api.github.com/users/eleonoraortega/following{/other_user}","gists_url":"https://api.github.com/users/eleonoraortega/gists{/gist_id}","starred_url":"https://api.github.com/users/eleonoraortega/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/eleonoraortega/subscriptions","organizations_url":"https://api.github.com/users/eleonoraortega/orgs","repos_url":"https://api.github.com/users/eleonoraortega/repos","events_url":"https://api.github.com/users/eleonoraortega/events{/privacy}","received_events_url":"https://api.github.com/users/eleonoraortega/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/raml-org/raml-js-parser-2/labels/bug","name":"bug","color":"fc2929"},{"url":"https://api.github.com/repos/raml-org/raml-js-parser-2/labels/Critical","name":"Critical","color":"eb6420"},{"url":"https://api.github.com/repos/raml-org/raml-js-parser-2/labels/problem","name":"problem","color":"b60205"},{"url":"https://api.github.com/repos/raml-org/raml-js-parser-2/labels/raml-1.0","name":"raml-1.0","color":"fbca04"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-07-21T00:58:55Z","updated_at":"2016-07-22T21:59:35Z","closed_at":null,"body":"```yaml\r\n\r\n#%RAML 1.0\r\ntitle: hello\r\nmediaType: application/json\r\ntypes:\r\n  MyType:\r\n    properties:\r\n      x: string\r\n\r\n/hello:\r\n  get:\r\n  post:\r\n    responses:\r\n      202:\r\n        body:\r\n          application/json:\r\n            type: object\r\n            properties:\r\n              hello:\r\n                type: number\r\n                minimum: 0\r\n                maximum: 10\r\n                format: int\r\n              key:\r\n                type: object\r\n                properties:\r\n                  x: string\r\n                example:\r\n                  x2: 123\r\n          application/xml:\r\n            type: string\r\n/world:\r\n\r\n```\r\n\r\n**Actual result:**\r\n```\r\n{\r\n      \"code\": 11,\r\n      \"message\": \"Unknown property:x2\",\r\n      \"path\": \"lala.raml\",\r\n      \"start\": 522,\r\n      \"end\": 524,\r\n      \"line\": 27,\r\n      \"column\": 18,\r\n      \"range\": [\r\n        {\r\n          \"line\": 27,\r\n          \"column\": 18,\r\n          \"position\": 522\r\n        },\r\n        {\r\n          \"line\": 27,\r\n          \"column\": 20,\r\n          \"position\": 524\r\n        }\r\n      ],\r\n      \"isWarning\": true\r\n    }\r\n```\r\n\r\n\r\n**Expected result:**\r\n\r\nAdditional properties are by default true, therefore x2 is valid\r\n"},"comment":{"url":"https://api.github.com/repos/raml-org/raml-js-parser-2/issues/comments/234667494","html_url":"https://github.com/raml-org/raml-js-parser-2/issues/367#issuecomment-234667494","issue_url":"https://api.github.com/repos/raml-org/raml-js-parser-2/issues/367","id":234667494,"user":{"login":"sichvoge","id":498114,"avatar_url":"https://avatars.githubusercontent.com/u/498114?v=3","gravatar_id":"","url":"https://api.github.com/users/sichvoge","html_url":"https://github.com/sichvoge","followers_url":"https://api.github.com/users/sichvoge/followers","following_url":"https://api.github.com/users/sichvoge/following{/other_user}","gists_url":"https://api.github.com/users/sichvoge/gists{/gist_id}","starred_url":"https://api.github.com/users/sichvoge/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sichvoge/subscriptions","organizations_url":"https://api.github.com/users/sichvoge/orgs","repos_url":"https://api.github.com/users/sichvoge/repos","events_url":"https://api.github.com/users/sichvoge/events{/privacy}","received_events_url":"https://api.github.com/users/sichvoge/received_events","type":"User","site_admin":false},"created_at":"2016-07-22T21:59:35Z","updated_at":"2016-07-22T21:59:35Z","body":"The question is if the TCK should test against warnings as well. That is something that we don't call out explicitely in the spec and only do for convenience inside the js parser. So let's not test against warnings.\r\n\r\n@svacas @KonstantinSviridov @eleonoraortega any objections?  "}}
{"actor":{"display_login":"sichvoge"}}