Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Add tests to cover RAML v1.0.1 patch #130

Open
jstoiko opened this issue Nov 20, 2019 · 5 comments
Open

Add tests to cover RAML v1.0.1 patch #130

jstoiko opened this issue Nov 20, 2019 · 5 comments
Assignees

Comments

@jstoiko
Copy link
Contributor

jstoiko commented Nov 20, 2019

In preparation of the RAML v1.0.1 patch release, we need to make sure TCK tests are updated to reflect the clarifications.

@postatum
Copy link
Contributor

postatum commented Nov 21, 2019

  • Clarify how to resolve !includes and libraries inside Overlay and Extension fragments
  • Clarify that libraries may be used in any type of fragment
  • Clarify requirement level of the pattern facet for type string
  • Clarify usage of enum in a union type
  • Clarify use of type-specific facets in a union type
  • Clarify how to infer/determine the default type for ALL built-in types
  • Clarify formatting of examples
  • Clarify nilable property alternative syntax
  • Clarify that references to inner elements only apply to JSON Schema / XSD
  • Specify that RAML documents MUST be valid YAML 1.2 documents: Nothing to add here.
  • Clarify that value of is in Applying Traits MUST be array

Wontfix:

  • Clarify use of trailing question mark: Already covered by existing tests.
  • Make all examples use / instead of /: No tests to add here. It's just a change of wording.
  • Issue with second "Introduction" link in RAML 1.0 table of contents: Nothing to test here.
  • Clarify "NamedExample" typed fragment description and usage: Already covered in tests/raml-1.0/Fragments/
  • Add missing "format" facet in Date type: Already covered by existing tests.

@jstoiko jstoiko changed the title Add tests to cover RAML v1.0.1 patch Add tests to cover RAML v1.0.1 patch (WIP) Nov 21, 2019
@jstoiko
Copy link
Contributor Author

jstoiko commented Nov 25, 2019

Specify that RAML documents MUST be valid YAML 1.2 documents: Nothing to add here.

how about adding one or more tests with YAML 1.2 -specific syntax?

@postatum
Copy link
Contributor

Specify that RAML documents MUST be valid YAML 1.2 documents: Nothing to add here.

how about adding one or more tests with YAML 1.2 -specific syntax?

Added.

@jstoiko
Copy link
Contributor Author

jstoiko commented Dec 3, 2019

Good. Let's move the YAML 1.2 line above out of Wontfix...

@postatum
Copy link
Contributor

postatum commented Dec 4, 2019

Good. Let's move the YAML 1.2 line above out of Wontfix...

Done

@jstoiko jstoiko changed the title Add tests to cover RAML v1.0.1 patch (WIP) Add tests to cover RAML v1.0.1 patch Sep 2, 2020
jstoiko added a commit that referenced this issue Apr 9, 2021
Fixes #130: Add tests to cover RAML v1.0.1 patch
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

2 participants