apache-airflow-providers-git 0.0.1rc1__tar.gz → 0.0.2__tar.gz

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: apache-airflow-providers-git
-Version: 0.0.1rc1
+Version: 0.0.2
 Summary: Provider package apache-airflow-providers-git for Apache Airflow
 Keywords: airflow-provider,git,airflow,integration
 Author-email: Apache Software Foundation <dev@airflow.apache.org>
@@ -20,11 +20,11 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Topic :: System :: Monitoring
-Requires-Dist: apache-airflow>=3.0.0rc0
+Requires-Dist: apache-airflow>=3.0.0
 Requires-Dist: GitPython>=3.1.44
 Project-URL: Bug Tracker, https://github.com/apache/airflow/issues
-Project-URL: Changelog, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1/changelog.html
-Project-URL: Documentation, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1
+Project-URL: Changelog, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2/changelog.html
+Project-URL: Documentation, https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2
 Project-URL: Mastodon, https://fosstodon.org/@airflow
 Project-URL: Slack Chat, https://s.apache.org/airflow-slack
 Project-URL: Source Code, https://github.com/apache/airflow
@@ -55,7 +55,7 @@ Project-URL: YouTube, https://www.youtube.com/channel/UCSXwxpWZQ7XZ1WL3wqevChA/
 
 Package ``apache-airflow-providers-git``
 
-Release: ``0.0.1``
+Release: ``0.0.2``
 
 
 `Distributed version control system (GIT) <https://git-scm.com/>`__
@@ -68,7 +68,7 @@ This is a provider package for ``git`` provider. All classes for this provider p
 are in ``airflow.providers.git`` python package.
 
 You can find package information and changelog for the provider
-in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1/>`_.
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2/>`_.
 
 Installation
 ------------
@@ -90,5 +90,5 @@ PIP package         Version required
 ==================  ==================
 
 The changelog for the provider package can be found in the
-`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1/changelog.html>`_.
+`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2/changelog.html>`_.
 

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/README.rst

@@ -23,7 +23,7 @@
 
 Package ``apache-airflow-providers-git``
 
-Release: ``0.0.1``
+Release: ``0.0.2``
 
 
 `Distributed version control system (GIT) <https://git-scm.com/>`__
@@ -36,7 +36,7 @@ This is a provider package for ``git`` provider. All classes for this provider p
 are in ``airflow.providers.git`` python package.
 
 You can find package information and changelog for the provider
-in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1/>`_.
+in the `documentation <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2/>`_.
 
 Installation
 ------------
@@ -58,4 +58,4 @@ PIP package         Version required
 ==================  ==================
 
 The changelog for the provider package can be found in the
-`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1/changelog.html>`_.
+`changelog <https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2/changelog.html>`_.

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/docs/changelog.rst

@@ -15,6 +15,26 @@
     specific language governing permissions and limitations
     under the License.
 
+0.0.2
+.....
+
+Bug Fixes
+~~~~~~~~~
+
+* ``Don't log repo_url in git dag bundle (#48909)``
+* ``Make git connection optional for git dag bundle (#49270)``
+* ``Use 'git_default' if the user defines nothing (#49359)``
+
+Misc
+~~~~
+
+* ``Include subdir in Gitbundle view url (#49239)``
+
+.. Below changes are excluded from the changelog. Move them to
+   appropriate section above if needed. Do not delete the lines(!):
+   * ``Prepare docs for Apr 3rd wave of providers (#49338)``
+   * ``Update documentation for edge3 and git provider (#49365)``
+
 0.0.1
 .....
 

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/docs/commits.rst

@@ -34,14 +34,31 @@ For high-level changelog, see :doc:`package information including changelog <ind
 
 
 
+0.0.2
+.....
+
+Latest change: 2025-04-16
+
+==================================================================================================  ===========  ============================================================
+Commit                                                                                              Committed    Subject
+==================================================================================================  ===========  ============================================================
+`2ae12c815b <https://github.com/apache/airflow/commit/2ae12c815bc704eff6890df56f7387da513d14f2>`__  2025-04-16   ``Update documentation for edge3 and git provider (#49365)``
+`dd0efe348b <https://github.com/apache/airflow/commit/dd0efe348b6ce94243ec7a727f9aa908eeafbc82>`__  2025-04-16   ``Use 'git_default' if the user defines nothing (#49359)``
+`ca3e4e75c6 <https://github.com/apache/airflow/commit/ca3e4e75c634afdceb23a86b7e0b0ff74614a7f1>`__  2025-04-16   ``Prepare docs for Apr 3rd wave of providers (#49338)``
+`192b337c86 <https://github.com/apache/airflow/commit/192b337c86eede5636f802d6f354fa97fe0f9030>`__  2025-04-16   ``Don't log repo_url in git dag bundle (#48909)``
+`2c7b56ee74 <https://github.com/apache/airflow/commit/2c7b56ee742409f65463534e34fa5f444d651af9>`__  2025-04-15   ``Make git connection optional for git dag bundle (#49270)``
+`b9052eed42 <https://github.com/apache/airflow/commit/b9052eed423f5f7f20c58815cce89e6cc659903c>`__  2025-04-15   ``Include subdir in Gitbundle view url (#49239)``
+==================================================================================================  ===========  ============================================================
+
 0.0.1
 .....
 
-Latest change: 2025-04-09
+Latest change: 2025-04-10
 
 ==================================================================================================  ===========  ==================================================================================
 Commit                                                                                              Committed    Subject
 ==================================================================================================  ===========  ==================================================================================
+`4a8567b20b <https://github.com/apache/airflow/commit/4a8567b20bdd6555cbdc936d6674bf4fa390b0d5>`__  2025-04-10   ``Prepare docs for Apr 2nd wave of providers (#49051)``
 `4a1e2ff8ab <https://github.com/apache/airflow/commit/4a1e2ff8ab8e6abca1cb273dc55d651d971e49f2>`__  2025-04-09   ``Mark git provider as ready (#48860)``
 `c68f31d3db <https://github.com/apache/airflow/commit/c68f31d3db6b957b4aeede7a257cc0ba59f12ce1>`__  2025-04-08   ``Remove dev0 suffix from Airlfow version (#48934)``
 `7b2ec33c7a <https://github.com/apache/airflow/commit/7b2ec33c7ad4998d9c9735b79593fcdcd3b9dd1f>`__  2025-04-08   ``Remove unnecessary entries in get_provider_info and update the schema (#48849)``

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/docs/index.rst

@@ -72,7 +72,7 @@ apache-airflow-providers-git package
 `Distributed version control system (GIT) <https://git-scm.com/>`__
 
 
-Release: 0.0.1
+Release: 0.0.2
 
 Provider package
 ----------------

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/provider.yaml

@@ -22,9 +22,10 @@ description: |
     `Distributed version control system (GIT) <https://git-scm.com/>`__
 
 state: ready
-source-date-epoch: 1742823216
+source-date-epoch: 1744792049
 # note that those versions are maintained by release manager - do not update them manually
 versions:
+  - 0.0.2
   - 0.0.1
 
 integrations:

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/pyproject.toml

@@ -25,7 +25,7 @@ build-backend = "flit_core.buildapi"
 
 [project]
 name = "apache-airflow-providers-git"
-version = "0.0.1.rc1"
+version = "0.0.2"
 description = "Provider package apache-airflow-providers-git for Apache Airflow"
 readme = "README.rst"
 authors = [
@@ -57,7 +57,7 @@ requires-python = "~=3.9"
 # Make sure to run ``breeze static-checks --type update-providers-dependencies --all-files``
 # After you modify the dependencies, and rebuild your Breeze CI image with ``breeze ci-image build``
 dependencies = [
-    "apache-airflow>=3.0.0rc0",
+    "apache-airflow>=3.0.0",
     "GitPython>=3.1.44",
 ]
 
@@ -95,8 +95,8 @@ apache-airflow-providers-common-sql = {workspace = true}
 apache-airflow-providers-standard = {workspace = true}
 
 [project.urls]
-"Documentation" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1"
-"Changelog" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.1/changelog.html"
+"Documentation" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2"
+"Changelog" = "https://airflow.apache.org/docs/apache-airflow-providers-git/0.0.2/changelog.html"
 "Bug Tracker" = "https://github.com/apache/airflow/issues"
 "Source Code" = "https://github.com/apache/airflow"
 "Slack Chat" = "https://s.apache.org/airflow-slack"

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/src/airflow/providers/git/__init__.py

@@ -29,7 +29,7 @@ from airflow import __version__ as airflow_version
 
 __all__ = ["__version__"]
 
-__version__ = "0.0.1"
+__version__ = "0.0.2"
 
 if packaging.version.parse(packaging.version.parse(airflow_version).base_version) < packaging.version.parse(
     "3.0.0"

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/src/airflow/providers/git/bundles/git.py

@@ -17,6 +17,7 @@
 from __future__ import annotations
 
 import os
+from contextlib import nullcontext
 from pathlib import Path
 from urllib.parse import urlparse
 
@@ -53,7 +54,7 @@ class GitDagBundle(BaseDagBundle):
         *,
         tracking_ref: str,
         subdir: str | None = None,
-        git_conn_id: str = "git_default",
+        git_conn_id: str | None = None,
         repo_url: str | None = None,
         **kwargs,
     ) -> None:
@@ -75,20 +76,26 @@ class GitDagBundle(BaseDagBundle):
             repo_path=self.repo_path,
             versions_path=self.versions_dir,
             git_conn_id=self.git_conn_id,
-            repo_url=self.repo_url,
         )
 
         self._log.debug("bundle configured")
-        try:
-            self.hook = GitHook(git_conn_id=self.git_conn_id, repo_url=self.repo_url)
-            self.repo_url = self.hook.repo_url
-            self._log.debug("repo_url updated from hook", repo_url=self.repo_url)
-        except AirflowException as e:
-            self._log.warning("Could not create GitHook", conn_id=self.git_conn_id, exc=e)
+        self.hook: GitHook | None = None
+        if not repo_url:
+            if not git_conn_id:
+                self._log.debug("Neither git_conn_id nor repo_url provided; loading 'git_default'")
+                git_conn_id = "git_default"
+            try:
+                self.hook = GitHook(git_conn_id=git_conn_id)
+            except AirflowException as e:
+                self._log.warning("Could not create GitHook", conn_id=git_conn_id, exc=e)
+            else:
+                self.repo_url = self.hook.repo_url
+                self._log.debug("repo_url updated from hook")
 
     def _initialize(self):
         with self.lock():
-            with self.hook.configure_hook_env():
+            cm = self.hook.configure_hook_env() if self.hook else nullcontext()
+            with cm:
                 self._clone_bare_repo_if_required()
                 self._ensure_version_in_bare_repo()
 
@@ -134,7 +141,7 @@ class GitDagBundle(BaseDagBundle):
                     url=self.repo_url,
                     to_path=self.bare_repo_path,
                     bare=True,
-                    env=self.hook.env,
+                    env=self.hook.env if self.hook else None,
                 )
             except GitCommandError as e:
                 raise AirflowException("Error cloning repository") from e
@@ -177,10 +184,10 @@ class GitDagBundle(BaseDagBundle):
 
     def _fetch_bare_repo(self):
         refspecs = ["+refs/heads/*:refs/heads/*", "+refs/tags/*:refs/tags/*"]
-        if self.hook.env:
-            with self.bare_repo.git.custom_environment(GIT_SSH_COMMAND=self.hook.env.get("GIT_SSH_COMMAND")):
-                self.bare_repo.remotes.origin.fetch(refspecs)
-        else:
+        cm = nullcontext()
+        if self.hook and (cmd := self.hook.env.get("GIT_SSH_COMMAND")):
+            cm = self.bare_repo.git.custom_environment(GIT_SSH_COMMAND=cmd)
+        with cm:
             self.bare_repo.remotes.origin.fetch(refspecs)
 
     def refresh(self) -> None:
@@ -188,7 +195,8 @@ class GitDagBundle(BaseDagBundle):
             raise AirflowException("Refreshing a specific version is not supported")
 
         with self.lock():
-            with self.hook.configure_hook_env():
+            cm = self.hook.configure_hook_env() if self.hook else nullcontext()
+            with cm:
                 self._fetch_bare_repo()
                 self.repo.remotes.origin.fetch(
                     ["+refs/heads/*:refs/remotes/origin/*", "+refs/tags/*:refs/tags/*"]
@@ -233,6 +241,8 @@ class GitDagBundle(BaseDagBundle):
             "gitlab.com": f"{url}/-/tree/{version}",
             "bitbucket.org": f"{url}/src/{version}",
         }
+        if self.subdir:
+            host_patterns = {k: f"{v}/{self.subdir}" for k, v in host_patterns.items()}
         for allowed_host, template in host_patterns.items():
             if host == allowed_host or host.endswith(f".{allowed_host}"):
                 return template

{apache_airflow_providers_git-0.0.1rc1 → apache_airflow_providers_git-0.0.2}/tests/unit/git/bundles/test_git.py

@@ -17,6 +17,7 @@
 
 from __future__ import annotations
 
+import json
 import os
 import re
 from unittest import mock
@@ -115,17 +116,17 @@ class TestGitDagBundle:
             tracking_ref=GIT_DEFAULT_BRANCH,
             repo_url="https://github.com/apache/zzzairflow",
         )
-        assert bundle.repo_url == f"https://{ACCESS_TOKEN}@github.com/apache/zzzairflow"
+        assert bundle.repo_url == "https://github.com/apache/zzzairflow"
 
     def test_falls_back_to_connection_host_when_no_repo_url_provided(self):
-        bundle = GitDagBundle(name="test", tracking_ref=GIT_DEFAULT_BRANCH)
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref=GIT_DEFAULT_BRANCH)
         assert bundle.repo_url == bundle.hook.repo_url
 
     @mock.patch("airflow.providers.git.bundles.git.GitHook")
     def test_get_current_version(self, mock_githook, git_repo):
         repo_path, repo = git_repo
         mock_githook.return_value.repo_url = repo_path
-        bundle = GitDagBundle(name="test", tracking_ref=GIT_DEFAULT_BRANCH)
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref=GIT_DEFAULT_BRANCH)
 
         bundle.initialize()
 
@@ -146,6 +147,7 @@ class TestGitDagBundle:
 
         bundle = GitDagBundle(
             name="test",
+            git_conn_id=CONN_HTTPS,
             version=starting_commit.hexsha,
             tracking_ref=GIT_DEFAULT_BRANCH,
         )
@@ -174,6 +176,7 @@ class TestGitDagBundle:
 
         bundle = GitDagBundle(
             name="test",
+            git_conn_id=CONN_HTTPS,
             version="test",
             tracking_ref=GIT_DEFAULT_BRANCH,
         )
@@ -195,7 +198,7 @@ class TestGitDagBundle:
         repo.index.add([file_path])
         repo.index.commit("Another commit")
 
-        bundle = GitDagBundle(name="test", tracking_ref=GIT_DEFAULT_BRANCH)
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref=GIT_DEFAULT_BRANCH)
         bundle.initialize()
 
         assert bundle.get_current_version() != starting_commit.hexsha
@@ -221,7 +224,7 @@ class TestGitDagBundle:
             writer.set_value("user", "name", "Test User")
             writer.set_value("user", "email", "test@example.com")
 
-        bundle = GitDagBundle(name="test", tracking_ref=GIT_DEFAULT_BRANCH)
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref=GIT_DEFAULT_BRANCH)
         bundle.initialize()
 
         assert bundle.get_current_version() == starting_commit.hexsha
@@ -252,7 +255,7 @@ class TestGitDagBundle:
         # add tag
         repo.create_tag("test123")
 
-        bundle = GitDagBundle(name="test", tracking_ref="test123")
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref="test123")
         bundle.initialize()
         assert bundle.get_current_version() == starting_commit.hexsha
 
@@ -279,7 +282,7 @@ class TestGitDagBundle:
         mock_githook.return_value.repo_url = repo_path
 
         repo.create_head("test")
-        bundle = GitDagBundle(name="test", tracking_ref="test")
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref="test")
         bundle.initialize()
         assert bundle.repo.head.ref.name == "test"
 
@@ -289,6 +292,7 @@ class TestGitDagBundle:
         mock_githook.return_value.repo_url = repo_path
         bundle = GitDagBundle(
             name="test",
+            git_conn_id=CONN_HTTPS,
             version="not_found",
             tracking_ref=GIT_DEFAULT_BRANCH,
         )
@@ -313,6 +317,7 @@ class TestGitDagBundle:
 
         bundle = GitDagBundle(
             name="test",
+            git_conn_id=CONN_HTTPS,
             tracking_ref=GIT_DEFAULT_BRANCH,
             subdir=subdir,
         )
@@ -406,6 +411,93 @@ class TestGitDagBundle:
     )
     @mock.patch("airflow.providers.git.bundles.git.Repo")
     def test_view_url(self, mock_gitrepo, repo_url, extra_conn_kwargs, expected_url, session):
+        session.query(Connection).delete()
+        conn = Connection(
+            conn_id="my_git_connection",
+            host=repo_url,
+            conn_type="git",
+            **(extra_conn_kwargs or {}),
+        )
+        session.add(conn)
+        session.commit()
+        bundle = GitDagBundle(
+            name="test",
+            git_conn_id="my_git_connection",
+            tracking_ref="main",
+        )
+        bundle.initialize = mock.MagicMock()
+        view_url = bundle.view_url("0f0f0f")
+        assert view_url == expected_url
+        bundle.initialize.assert_not_called()
+
+    @pytest.mark.parametrize(
+        "repo_url, extra_conn_kwargs, expected_url",
+        [
+            (
+                "git@github.com:apache/airflow.git",
+                None,
+                "https://github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            ("git@github.com:apache/airflow", None, "https://github.com/apache/airflow/tree/0f0f0f/subdir"),
+            (
+                "https://github.com/apache/airflow",
+                None,
+                "https://github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://github.com/apache/airflow.git",
+                None,
+                "https://github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "git@gitlab.com:apache/airflow.git",
+                None,
+                "https://gitlab.com/apache/airflow/-/tree/0f0f0f/subdir",
+            ),
+            (
+                "git@bitbucket.org:apache/airflow.git",
+                None,
+                "https://bitbucket.org/apache/airflow/src/0f0f0f/subdir",
+            ),
+            (
+                "git@myorg.github.com:apache/airflow.git",
+                None,
+                "https://myorg.github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://myorg.github.com/apache/airflow.git",
+                None,
+                "https://myorg.github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://github.com/apache/airflow",
+                {"password": "abc123"},
+                "https://github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://github.com/apache/airflow",
+                {"login": "abc123"},
+                "https://github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://github.com/apache/airflow",
+                {"login": "abc123", "password": "def456"},
+                "https://github.com/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://github.com:443/apache/airflow",
+                None,
+                "https://github.com:443/apache/airflow/tree/0f0f0f/subdir",
+            ),
+            (
+                "https://github.com:443/apache/airflow",
+                {"password": "abc123"},
+                "https://github.com:443/apache/airflow/tree/0f0f0f/subdir",
+            ),
+        ],
+    )
+    @mock.patch("airflow.providers.git.bundles.git.Repo")
+    def test_view_url_subdir(self, mock_gitrepo, repo_url, extra_conn_kwargs, expected_url, session):
         session.query(Connection).delete()
         conn = Connection(
             conn_id="git_default",
@@ -418,6 +510,8 @@ class TestGitDagBundle:
         bundle = GitDagBundle(
             name="test",
             tracking_ref="main",
+            subdir="subdir",
+            git_conn_id="git_default",
         )
         bundle.initialize = mock.MagicMock()
         view_url = bundle.view_url("0f0f0f")
@@ -440,7 +534,7 @@ class TestGitDagBundle:
 
         with mock.patch("airflow.providers.git.bundles.git.Repo.clone_from") as mock_clone:
             mock_clone.side_effect = GitCommandError("clone", "Simulated error")
-            bundle = GitDagBundle(name="test", tracking_ref="main")
+            bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref="main")
             with pytest.raises(
                 AirflowException,
                 match=re.escape("Error cloning repository"),
@@ -461,25 +555,46 @@ class TestGitDagBundle:
                 assert "Repository path: %s not found" in str(exc_info.value)
 
     @patch.dict(os.environ, {"AIRFLOW_CONN_MY_TEST_GIT": '{"host": "something"}'})
-    @pytest.mark.parametrize("conn_id, should_find", [("my_test_git", True), ("something-else", False)])
-    def test_repo_url_access_missing_connection_doesnt_error(self, conn_id, should_find):
+    @pytest.mark.parametrize(
+        "conn_id, expected_hook_type",
+        [("my_test_git", GitHook), ("something-else", type(None))],
+    )
+    def test_repo_url_access_missing_connection_doesnt_error(self, conn_id, expected_hook_type):
         bundle = GitDagBundle(
             name="testa",
             tracking_ref="main",
             git_conn_id=conn_id,
-            repo_url="some_repo_url",
         )
-        assert bundle.repo_url == "some_repo_url"
-        if should_find:
-            assert isinstance(bundle.hook, GitHook)
-        else:
-            assert not hasattr(bundle, "hook")
+        assert isinstance(bundle.hook, expected_hook_type)
 
     @mock.patch("airflow.providers.git.bundles.git.GitHook")
     def test_lock_used(self, mock_githook, git_repo):
         repo_path, repo = git_repo
         mock_githook.return_value.repo_url = repo_path
-        bundle = GitDagBundle(name="test", tracking_ref=GIT_DEFAULT_BRANCH)
+        bundle = GitDagBundle(name="test", git_conn_id=CONN_HTTPS, tracking_ref=GIT_DEFAULT_BRANCH)
         with mock.patch("airflow.providers.git.bundles.git.GitDagBundle.lock") as mock_lock:
             bundle.initialize()
             assert mock_lock.call_count == 2  # both initialize and refresh
+
+    @pytest.mark.parametrize(
+        "conn_json, repo_url, expected",
+        [
+            (
+                {"host": "git@github.com:apache/airflow.git"},
+                "git@github.com:apache/hello.git",
+                "git@github.com:apache/hello.git",
+            ),
+            ({"host": "git@github.com:apache/airflow.git"}, None, "git@github.com:apache/airflow.git"),
+            ({}, "git@github.com:apache/hello.git", "git@github.com:apache/hello.git"),
+        ],
+    )
+    def test_repo_url_precedence(self, conn_json, repo_url, expected):
+        conn_str = json.dumps(conn_json)
+        with patch.dict(os.environ, {"AIRFLOW_CONN_MY_TEST_GIT": conn_str}):
+            bundle = GitDagBundle(
+                name="test",
+                tracking_ref="main",
+                git_conn_id="my_test_git",
+                repo_url=repo_url,
+            )
+            assert bundle.repo_url == expected