094d0a917f
This PR reclaims the changes introduced in this [PR](https://github.com/appsmithorg/appsmith/pull/19265) later which got reverted by this [PR](https://github.com/appsmithorg/appsmith/pull/20629) ## Description - The PR removes the empty password validation check for PostgreSQL while creating a new data source. - The JUnit test case is added for the same. Implemented using a separate PostgreSQL container with Trust Authentication enabled for the Same. - Please note for testing from UI perspective we have to create a user with Trust auth enabled to validate the above changes. Fixes #14003 Media > A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video ## Type of change > Please delete options that are not relevant. - Bug fix (non-breaking change which fixes an issue) ## How Has This Been Tested? - JUnit ### Test Plan > Add Testsmith test cases links that relate to this PR ### Issues raised during DP testing > Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR) ## Checklist: ### Dev activity - [ ] My code follows the style guidelines of this project - [ ] I have performed a self-review of my own code - [ ] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] New and existing unit tests pass locally with my changes - [ ] PR is being merged under a feature flag ### QA activity: - [ ] Test plan has been approved by relevant developers - [ ] Test plan has been peer reviewed by QA - [ ] Cypress test cases have been added and approved by either SDET or manual QA - [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA - [ ] Added Test Plan Approved label after reveiwing all Cypress test |
||
|---|---|---|
| .. | ||
| src | ||
| pom.xml | ||