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

Test case for OL issue 29893. #30431

Open
wants to merge 2 commits into
base: integration
Choose a base branch
from

Conversation

anija-anil
Copy link
Contributor

Test case added for : #29893

@anija-anil anija-anil self-assigned this Dec 17, 2024
@anija-anil anija-anil changed the title Ecl jakrta data issues Test case for OL issue 29893. Dec 17, 2024
@anija-anil
Copy link
Contributor Author

anija-anil commented Dec 17, 2024

#build (view Open Liberty Personal Build - ❌ completed with errors/failures.)

Note: Target locations of links might be accessible only to IBM employees.

tx.begin();
try {
result = em.createQuery("FROM Vehicle WHERE LOWER(ID(THIS)) = ?1", Vehicle.class)
.setParameter(1, idToSearch.toLowerCase())
Copy link
Member

Choose a reason for hiding this comment

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

nitpick: (do not need to change) the variable idToSearch is unnecessary. You could have just used vehicleId.toLowerCase()

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • Please describe in a separate comment how you tested your changes.

  • 2 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

@LibbyBot
Copy link

Your personal build request is at https://wasrtc.hursley.ibm.com:9443/jazz/resource/itemOid/com.ibm.team.build.BuildResult/_EOB-8LznEe-V7IPhyP5jiw

The build will run your changed FAT bucket(s) 10 times.

Target locations of links might be accessible only to IBM employees.

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.

3 participants