diff --git a/tests/test_check_upstream_updates.py b/tests/test_check_upstream_updates.py index 6eb49a0..a78b4c3 100644 --- a/tests/test_check_upstream_updates.py +++ b/tests/test_check_upstream_updates.py @@ -98,6 +98,22 @@ class DirectCloneFallbackTests(UpstreamCheckerRepoFixture): self.assertNotIn("does not point to the ai-job-search template repo", result.stdout) self.assertIn("up to date with origin/master", result.stdout) + def test_clone_with_lowercased_template_url_falls_back_without_fork_warning(self): + # GitHub serves repo paths case-insensitively, so a clone from + # https://github.com/madslorentzen/ai-job-search is still the template. + subprocess.run( + ["git", "remote", "set-url", "origin", TEMPLATE_URL.lower()], + cwd=self.root, + check=True, + capture_output=True, + ) + + result = self.run_checker("--remote", "upstream") + + self.assertEqual(result.returncode, 0, result.stdout + result.stderr) + self.assertIn("Falling back to 'origin'", result.stdout) + self.assertNotIn("does not point to the ai-job-search template repo", result.stdout) + class UpstreamRemotePresentTests(UpstreamCheckerRepoFixture): def setUp(self): diff --git a/tools/check_upstream_updates.py b/tools/check_upstream_updates.py index f431fcb..0c9a1b1 100755 --- a/tools/check_upstream_updates.py +++ b/tools/check_upstream_updates.py @@ -85,7 +85,8 @@ def main() -> int: # A fork's own 'origin' can never reveal upstream updates: warn so the # user is not misled by the final '[OK]' line below. (Direct clones of # the template repo have origin == the upstream repo, so no warning.) - if remote != args.remote and UPSTREAM_REPO_SLUG not in get_remote_url(remote): + # GitHub serves repo paths case-insensitively, so compare lowercased. + if remote != args.remote and UPSTREAM_REPO_SLUG.lower() not in get_remote_url(remote).lower(): print( f"Warning: Remote '{remote}' does not point to the ai-job-search " f"template repo ({UPSTREAM_REPO_SLUG}), so this check compares your "