Skip to content

Conversation

balajirajput96
Copy link

This pull request adds new tests for the GitMergeHandler to improve coverage of merge operations in the JupyterLab Git extension. The main focus is verifying both successful and conflict scenarios when merging branches.

Added tests for GitMergeHandler:

Merge operation scenarios:

  • Added test_merge_handler_success to verify that merging a branch returns a successful response and the correct payload when there are no conflicts.
  • Added test_merge_handler_conflict to verify that merging a branch with conflicts returns a 500 error and the appropriate conflict payload.
    [Copilot is generating a summary...]

Copilot AI and others added 2 commits October 4, 2025 14:43
Co-authored-by: balajirajput96 <124477404+balajirajput96@users.noreply.github.com>
Co-authored-by: balajirajput96 <124477404+balajirajput96@users.noreply.github.com>
Copy link

github-actions bot commented Oct 7, 2025

Binder 👈 Launch a Binder on branch balajirajput96/jupyterlab-git/main

@balajirajput96
Copy link
Author

This pull request adds new unit tests for the Git merge handler to improve test coverage and ensure proper handling of both successful and conflict scenarios during branch merges.

Testing improvements

  • Added test_merge_handler_success to verify that the merge handler correctly merges a branch and returns a success response when there are no conflicts.
  • Added test_merge_handler_conflict to check that the merge handler responds appropriately with an error code and message when a merge conflict occurs.

@balajirajput96 balajirajput96 reopened this Oct 7, 2025
@balajirajput96 balajirajput96 changed the base branch from main to 0.9.x October 7, 2025 01:14
@balajirajput96 balajirajput96 changed the base branch from 0.9.x to 0.32.x October 7, 2025 01:15
@balajirajput96 balajirajput96 changed the base branch from 0.32.x to dependabot/npm_and_yarn/form-data-4.0.4 October 7, 2025 01:16
@balajirajput96 balajirajput96 changed the base branch from dependabot/npm_and_yarn/form-data-4.0.4 to jlab-3 October 7, 2025 01:16
@balajirajput96 balajirajput96 changed the base branch from jlab-3 to main October 7, 2025 01:17
Co-authored-by: balajirajput96 <124477404+balajirajput96@users.noreply.github.com> (#2)

* Initial plan

* Add missing unit tests for GitMergeHandler

Co-authored-by: balajirajput96 <124477404+balajirajput96@users.noreply.github.com>

* Format merge tests with black

Co-authored-by: balajirajput96 <124477404+balajirajput96@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants