Bug Fix: Replace lstrip with replace function (#849)

* Replace lstrip with replace function

* replace only first occurrence of remote

* add unit test

* make formatter happy

Author: rrao24

jupyterlab_git/git.py

@@ -1104,7 +1104,7 @@ async def get_upstream_branch(self, current_path, branch_name):
             return {"code": code, "command": " ".join(command), "message": error}
 
         remote_name = output.strip()
-        remote_branch = rev_parse_output.strip().lstrip(remote_name + "/")
+        remote_branch = rev_parse_output.strip().replace(remote_name + "/", "", 1)
         return {
             "code": code,
             "remote_short_name": remote_name,

jupyterlab_git/tests/test_branch.py

@@ -408,6 +408,8 @@ async def test_get_current_branch_detached_failure():
         ("feature-foo", "master", "origin/withslash"),
         ("master", "master", "origin"),
         ("feature/bar", "feature-foo", ""),
+        # Test upstream branch name starts with a letter contained in remote name
+        ("rbranch", "rbranch", "origin"),
     ],
 )
 async def test_get_upstream_branch_success(branch, upstream, remotename):