-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
BeanShells are changed to correct using of variables #22138
BeanShells are changed to correct using of variables #22138
Conversation
Hi @AnnaShepa. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
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.
Please leave only meaningful changes.
@orlangur formatting updated |
@AnnaShepa thanks! Changes looks good to me. After changes are applied and all builds are green, please squash them into a single commit so that we have perfectly clean history 😉 |
d3a6f6f
to
9364787
Compare
@orlangur done, squashed |
Hi @orlangur, thank you for the review. |
@AnnaShepa thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository. |
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.
Performance tests are failing on the validation stage with the following message:
Any changes to the Performance Toolkit JMeter script (setup/performance-toolkit/benchmark.jmx), must be made in the jmx generator fragments (magento2-infrastructure/build/core_dev/performance/mpaf/tool/fragments). The final version of the Performance Toolkit JMeter script (setup/performance-toolkit/benchmark.jmx) must be generated from fragments via perf.php (read documentation from magento2-infrastructure/build/core_dev/performance/mpaf/README.md)
@p-bystritsky could you please help to resolve this.
3d1a969
to
89664aa
Compare
Hi @p-bystritsky , performance build still failing due to scenario inconsistency. Could you please take a look? |
89664aa
to
50d4601
Compare
fcee492
to
578411d
Compare
Performance Acceptance Tests are failing at the CE and B2B validation stage (passing for EE) @p-bystritsky please check the error message, feel free contact me for any questions
|
Hi @AnnaShepa, thank you for your contribution! |
Description (*)
In loops to collect products in SetUp thread added variable declaration to avoid 500 error. As well as changed using of wrong variable in collection products for edit.
Manual testing scenarios (*)
Run the SetUp thread - as the result jMeter return no 500 error "org.apache.jorphan.util.JMeterException: Error invoking bsh method: eval"
Contribution checklist (*)