[IMP] Implement otools-submodule push: ported from invoke submodule.push

Author: ivantodorovich

changes.d/+56a73459.feat.rst

@@ -0,0 +1 @@
+Implement otools-submodule push (ported invoke submodule.push)

odoo_tools/cli/submodule.py

@@ -130,5 +130,21 @@ def sync_remote(submodule_path=None, repo=None, force_remote=False):
                 git.checkout(branch_name=odoo_version)
 
 
+@cli.command()
+@click.argument("submodule_path")
+@click.option(
+    "--target-branch",
+    default=None,
+    help="Target branch name. If omitted, computed automatically.",
+)
+def push(submodule_path, target_branch=None):
+    """Push the current state of a submodule to the company remote."""
+    repo = pm_utils.Repo(submodule_path)
+    target_branch = target_branch or pm_utils.gh.get_target_branch()
+    ui.echo(f"Pushing {repo.name} to {repo.company_git_remote}/{target_branch}")
+    repo.push_to_remote(target_branch=target_branch)
+    ui.echo("Done.")
+
+
 if __name__ == "__main__":
     cli()

odoo_tools/utils/pending_merge.py

@@ -602,6 +602,13 @@ def _handle_empty_merges_file(self, delete_file=False):
         ):
             self.abs_merges_path.unlink()
 
+    def push_to_remote(self, target_branch=None):
+        target_branch = target_branch or gh.get_target_branch()
+        aggregator = self.get_aggregator(target_branch=target_branch)
+        with cd(self.abs_path):
+            aggregator._switch_to_branch(target_branch)
+            aggregator.push()
+
     def rebuild_consolidation_branch(self, push=False):
         aggregator = self.get_aggregator()
         aggregator.aggregate()

tests/test_submodule.py

@@ -5,7 +5,7 @@
 
 from odoo_tools.cli import submodule
 
-from .common import get_fixture_path, mock_subprocess_run
+from .common import get_fixture_path, mock_pending_merge_repo_paths, mock_subprocess_run
 
 
 @pytest.mark.project_setup(
@@ -152,6 +152,26 @@ def test_ls(project):
     ]
 
 
+@pytest.mark.project_setup(
+    manifest=dict(odoo_version="16.0"),
+    proj_version="16.0.1.2.3",
+)
+def test_push(project):
+    mock_pending_merge_repo_paths("some-repo", src=True, pending=True)
+    with mock.patch.object(
+        submodule.pm_utils.Repo, "push_to_remote"
+    ) as mock_push_to_remote:
+        result = project.invoke(
+            submodule.push,
+            ["some-repo", "--target-branch", "my-target-branch"],
+            catch_exceptions=False,
+        )
+    assert result.exit_code == 0
+    mock_push_to_remote.assert_called_once_with(target_branch="my-target-branch")
+    assert "my-target-branch" in result.output
+    assert "Done." in result.output
+
+
 @pytest.mark.project_setup(
     manifest=dict(odoo_version="16.0"),
     proj_version="16.0.1.2.3",