Arazzo Parser#1510
Conversation
jgaleotti
left a comment
There was a problem hiding this comment.
We have several issues regarding this PR. We might address them during our meeting today.
| import kotlin.jvm.java | ||
| import kotlin.text.startsWith | ||
|
|
||
| object ArazzoAccess { |
There was a problem hiding this comment.
Add a Javadoc of the goal of this class
| import io.swagger.v3.oas.models.media.Schema | ||
| import org.evomaster.core.problem.rest.arazzo.models.* | ||
|
|
||
| class ArazzoReferenceResolver( |
There was a problem hiding this comment.
Add a Javadoc describing this class
| private var currentPosition = 0 | ||
|
|
||
| init { | ||
| // Esta Regex identifica todo el vocabulario válido de Arazzo |
There was a problem hiding this comment.
Remove all comments in Spanish, only English is allowed for comments and names.
| @@ -0,0 +1,103 @@ | |||
| package org.evomaster.core.problem.rest.arazzo.parser | |||
|
|
|||
| class SimpleConditionParser(condition: String) { | |||
There was a problem hiding this comment.
Why is this parser needed? It does not seem to be used at all.
| p0: JsonParser, | ||
| p1: DeserializationContext | ||
| ): AnyExpression { | ||
| val node: JsonNode = p0.codec.readTree(p0) |
There was a problem hiding this comment.
Is this code automatically generated?
| @@ -0,0 +1,10 @@ | |||
| package org.evomaster.core.problem.rest.arazzo.models | |||
|
|
|||
| class SuccessAction( | |||
| import io.swagger.v3.oas.models.media.Schema | ||
| import org.evomaster.core.problem.rest.arazzo.models.commons.WorkflowCommon | ||
|
|
||
| class Workflow( |
| @@ -0,0 +1,7 @@ | |||
| package org.evomaster.core.problem.rest.arazzo.models | |||
|
|
|||
| sealed class SuccessReusable { | |||
| import org.evomaster.core.problem.rest.schema.SchemaArazzo | ||
| import org.evomaster.core.problem.rest.schema.SchemaOpenAPI | ||
|
|
||
| object ArazzoParser { |
| import org.evomaster.core.problem.rest.arazzo.models.RuntimeExpression | ||
| import org.evomaster.core.problem.rest.arazzo.models.Source | ||
|
|
||
| object ExpressionParser { |
There was a problem hiding this comment.
Is this really necessary?
|
This class should be moved into core-extra as a submodule arazzo-jvm. It must ablso be written with JDK8 |
A first version of an Arazzo parser is created. It only recognizes well-formed Arazzo and has no validations.