Skip to content
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

kie-issues#1481: Test Scenario Editor: Create @kie-tools/scesim-editor-envelope #2687

Merged
merged 8 commits into from
Oct 22, 2024

Conversation

@yesamer yesamer removed the pr: wip PR is still under development label Oct 21, 2024
Copy link
Contributor

@jomarko jomarko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @yesamer . Using the vsix you shared privately, I was not able to open empty scesim file, is that expected? I didn't try to open non empty scesim file.

image

packages/scesim-editor/src/TestScenarioEditor.tsx Outdated Show resolved Hide resolved
packages/scesim-editor/src/TestScenarioEditor.tsx Outdated Show resolved Hide resolved
svg: "",
};
} else {
throw new Error(`Unknown extension '${ext}'.`);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe is more unsupported than unknown, or will this new editor has single option - DMN? previously we had DMN and DRL

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jomarko Only DMN files are directly imported. In case of RULE-based Test Scenario, we don't import the DRL file, but the Java Classes present in the user project.

@yesamer
Copy link
Contributor Author

yesamer commented Oct 22, 2024

@jomarko Thank you for testing the Extension. Indeed the issue reproduces to me as well, but with new scesim files only. If you open an existing *.scesim file, it works correctly
Screenshot 2024-10-22 at 15 11 09

I'll investigate more this behavior when working on apache/incubator-kie-issues#1552

@yesamer yesamer requested a review from jomarko October 22, 2024 13:15
Copy link
Contributor

@jomarko jomarko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for updates @yesamer

@tiagobento tiagobento merged commit 2fd5a2c into apache:main Oct 22, 2024
14 checks passed
@yesamer yesamer deleted the 1481-new branch October 22, 2024 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test Scenario Editor: Create @kie-tools/scesim-editor-envelope
3 participants