Skip to content

Comments

Code remove#390

Closed
cx-shubham-singhal wants to merge 16 commits intomainfrom
CodeRemove
Closed

Code remove#390
cx-shubham-singhal wants to merge 16 commits intomainfrom
CodeRemove

Conversation

@cx-shubham-singhal
Copy link
Contributor

@cx-shubham-singhal cx-shubham-singhal commented Jan 19, 2026

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Describe the purpose of this PR along with any background information and the impacts of the proposed change.

References

Include supporting link to GitHub Issue/PR number

Testing

Describe how this change was tested. Be specific about anything not tested and reasons why. If this solution has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

Please include any manual steps for testing end-to-end or functionality not covered by unit/integration tests.

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

cx-shubham-yadav and others added 16 commits January 5, 2026 16:09
@cx-shubham-yadav
Code refactor
1a5b880
@cx-shubham-yadav
Merge pull request #377 from Checkmarx/other/welcomeUiTest
1628bae 
Refactoring the existing UI code
@cx-dhanashree-wani
Latest UI fixes
fc9f77a
@cx-shubham-yadav
adding validation for welcomepage
cbaad39
@cx-shubham-yadav
Merge branch 'other/test-ui' into validateWelcomePage
65e6f6e
@cx-shubham-yadav
Merge pull request #379 from Checkmarx/validateWelcomePage
b564db0 
Validate welcome page
@cx-shubham-yadav
Adding test cases for Assist Page Validations
c7d9e98
@cx-shubham-yadav
Merge pull request #380 from Checkmarx/validateDevAssistCheckbox
83d4d07 
Adding test cases for Assist Page Validations
@cx-shubham-singhal
OAuth Valid and Invalid Test cases (#383)
f6e2618 
* Valid and Invalid OAth Test Cases

* Created new Test Constant File

* Constant Changes

* Constant change

* Description Changes
@cx-shubham-yadav
Fixing General Test Cases and Created structure for it
050ce38
@cx-shubham-yadav
Merge branch 'other/test-ui' into fixFailingCases
c57e7b8
@cx-shubham-yadav
Merge pull request #385 from Checkmarx/fixFailingCases
8985b84 
Fix failing cases - Phase1
@cx-dhanashree-wani
Adding Test Cases for ASCA and OSS Realtime Scan Validations (#382)
5d11943 
* Added New UI test cases for ASCA and OSS

* Updated code as per review comments

* Added extends class to tests

* Updated comments

* Updated method
@cx-shubham-yadav
Fixing the Scan and CxOne Result related test cases (#388)
f76f06b 
* Fixing the General Test

* removing comment

* removing comments

* removing comments

* fixing pipeline failure

* fixing pipeline failure

* fixing pipeline failure

* fixing pipeline failure
UI Run Dummy
b63a093
UI Run Dummyy
53b7655
@github-actions
Copy link

github-actions bot commented Jan 19, 2026

Logo
Checkmarx One – Scan Summary & Details – 20a5fa24-0f5f-4fbf-9ba3-efc05c2f1e4b

Great job! No new security vulnerabilities introduced in this pull request

@cx-shubham-singhal cx-shubham-singhal deleted the CodeRemove branch January 20, 2026 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cx-shubham-singhal @cx-shubham-yadav @cx-dhanashree-wani