Skip to content

Add load testing with Jmeter for ABC123 for cargo handling flow - #338

Open
elanajam wants to merge 5 commits into
eclipse-ee4j:masterfrom
elanajam:master
Open

Add load testing with Jmeter for ABC123 for cargo handling flow#338
elanajam wants to merge 5 commits into
eclipse-ee4j:masterfrom
elanajam:master

Conversation

@elanajam

Copy link
Copy Markdown

feat(jmeter): Configure JMeter integration test profile with JaCoCo coverage

  • Add JMeter dependencies and configuration in pom.xml
  • Configure JaCoCo agent for code coverage during integration tests
  • Set up Liberty server start/stop for test execution
  • Add JMeter plugin configuration for test execution

test(jmeter): Add cargo event logging test scenarios

  • Create cargotracker-week30.jmx test plan
  • Add test cases for cargo handling events:
    • UNLOAD 0200T USDAL
    • LOAD 0300A USDAL
    • UNLOAD 0300A FIHEL
    • CLAIM 0300A FIHEL
  • Configure HTTP samplers and assertions

@elanajam
elanajam marked this pull request as ready for review July 31, 2025 14:22

@edburns edburns left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This looks good and finally provides some coverage for #17.

@edburns

edburns commented Aug 6, 2025

Copy link
Copy Markdown

Hello @elanajam ,

I have appealed to the committers to become a committer myself, so I can better influence the acceptance of this PR. This appeal may or may not succeed.

In the meantime, I have appealed to the committers to review your PR directly, with elevated priority.

Either way, we appreciate the PR.

@m-reza-rahman

m-reza-rahman commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

This PR really needs a lot of discussion and further work before it can be meaningfully considered. In truth, the discussion and planning should have taken place before the PR was issued. This is easily one of the most complicated bodies of work in this project, and it needs to be done right.

Is there still a genuine interest in pursuing this? I estimate doing this right is at least a few months of work for anyone not doing this full time. If there isn’t that kind of time commitment, I suggest simply closing this. I don’t have bandwidth to accept partial work and have it lay around for a long time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants