New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
added more tests for YAML DSL (using a schema) #364
Conversation
Signed-off-by: Lars Heinemann <lhein.smx@gmail.com>
completion called at the right place Signed-off-by: Aurélien Pupier <apupier@redhat.com>
Signed-off-by: Lars Heinemann <lhein.smx@gmail.com>
Signed-off-by: Lars Heinemann <lhein.smx@gmail.com>
Signed-off-by: Lars Heinemann <lhein.smx@gmail.com>
@@ -5,9 +5,9 @@ | |||
uri: "timer:tick" | |||
parameters: | |||
period: "5s" | |||
steps: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this change was required because the yaml structure changed
…yaml file Signed-off-by: Lars Heinemann <lhein.smx@gmail.com>
Signed-off-by: Lars Heinemann <lhein.smx@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- needs rebase.
|
||
export const updateSettings = (setting: any, value: any) => { | ||
const yamlConfiguration = vscode.workspace.getConfiguration('yaml'); | ||
return yamlConfiguration.update(setting, value, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong indentation
|
||
export const resetSettings = (setting: any, value: any) => { | ||
const yamlConfiguration = vscode.workspace.getConfiguration('yaml'); | ||
return yamlConfiguration.update(setting, value, false); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
wrong indentation
const expect = chai.expect; | ||
|
||
describe('YAML DSL and endpoint uri completion tests backed by schema', () => { | ||
const yamlSchemaUri: vscode.Uri = vscode.Uri.parse('https://raw.githubusercontent.com/lburgazzoli/apache-camel-k-runtime/github-358/camel-k-loader-yaml/camel-k-loader-yaml/src/generated/resources/yaml-schema.json'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it wasn't available when the PR was written but now that it is on master, maybe better to point to https://github.com/apache/camel-k-runtime/blob/master/camel-k-loader-yaml/camel-k-loader-yaml/src/generated/resources/camel-yaml-dsl.json
that said, it migth be worthy to wait for the schemastore to be available and rely on it SchemaStore/schemastore#1138 (or can be done in another iteration)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it is now available on schemastore so it can be completely removed
closing as it has not been touched for a while. |
Pull Request informations
Rebase & Merge default requirements
PR labels default process
Tests
PR workflow progress