From 0a30179f00f5a1a6c8422281b9580138b6f83074 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ga=C3=ABl=20Reyrol?= <me@gaelreyrol.dev>
Date: Sat, 9 Dec 2023 15:07:39 +0100
Subject: [PATCH] Add support for updating buildComposerProject's vendorHash

Update tests/testpkgs/default.nix
---
 .gitignore                  |  1 +
 README.md                   |  1 +
 nix_update/eval.py          |  2 ++
 nix_update/update.py        |  8 ++++++++
 tests/test_composer.py      | 27 +++++++++++++++++++++++++++
 tests/testpkgs/composer.nix | 17 +++++++++++++++++
 tests/testpkgs/default.nix  |  1 +
 7 files changed, 57 insertions(+)
 create mode 100644 tests/test_composer.py
 create mode 100644 tests/testpkgs/composer.nix

diff --git a/.gitignore b/.gitignore
index 5a98314..4dd4db2 100644
--- a/.gitignore
+++ b/.gitignore
@@ -89,6 +89,7 @@ venv/
 ENV/
 env.bak/
 venv.bak/
+.direnv
 
 # Spyder project settings
 .spyderproject
diff --git a/README.md b/README.md
index cb4fbc6..a290929 100644
--- a/README.md
+++ b/README.md
@@ -17,6 +17,7 @@ designed to work with nixpkgs but also other package sets.
 - update buildRustPackage's cargoHash/cargoSha256 and cargoSetupHook's cargoDeps
 - update buildGoModule's vendorHash/vendorSha256
 - update buildNpmPackage's npmDepsHash and npmConfigHook's npmDeps
+- update buildComposerProject's vendorHash
 - update fetchYarnDeps offlineCache output hash
 - update flake outputs (see `--flake`)
 - build and run the resulting package (see `--build`,
diff --git a/nix_update/eval.py b/nix_update/eval.py
index 66e1008..e018dd5 100644
--- a/nix_update/eval.py
+++ b/nix_update/eval.py
@@ -54,6 +54,7 @@ class Package:
     cargo_deps: str | None
     npm_deps: str | None
     yarn_deps: str | None
+    composer_deps: str | None
     tests: list[str]
     has_update_script: bool
 
@@ -157,6 +158,7 @@ def eval_expression(
       if res.success then res.value.file else false
     else
       null;
+  composer_deps = pkg.composerRepository.outputHash or null;
   npm_deps = pkg.npmDeps.outputHash or null;
   yarn_deps = pkg.offlineCache.outputHash or null;
   tests = builtins.attrNames (pkg.passthru.tests or {{}});
diff --git a/nix_update/update.py b/nix_update/update.py
index d886edb..d2bbb35 100644
--- a/nix_update/update.py
+++ b/nix_update/update.py
@@ -243,6 +243,11 @@ def update_cargo_lock(
                 print(line, end="")
 
 
+def update_composer_deps_hash(opts: Options, filename: str, current_hash: str) -> None:
+    target_hash = nix_prefetch(opts, "composerRepository")
+    replace_hash(filename, current_hash, target_hash)
+
+
 def print_hashes(hashes: dict[str, str], indent: str) -> None:
     if not hashes:
         return
@@ -367,6 +372,9 @@ def update(opts: Options) -> Package:
         ):
             update_cargo_lock(opts, package.filename, package.cargo_lock)
 
+        if package.composer_deps:
+            update_composer_deps_hash(opts, package.filename, package.composer_deps)
+
         if package.npm_deps:
             update_npm_deps_hash(opts, package.filename, package.npm_deps)
 
diff --git a/tests/test_composer.py b/tests/test_composer.py
new file mode 100644
index 0000000..771a524
--- /dev/null
+++ b/tests/test_composer.py
@@ -0,0 +1,27 @@
+import subprocess
+
+import conftest
+
+from nix_update.options import Options
+from nix_update.update import update
+
+
+def test_update(helpers: conftest.Helpers) -> None:
+    with helpers.testpkgs() as path:
+        opts = Options(attribute="composer", import_path=str(path))
+        update(opts)
+        version = subprocess.run(
+            [
+                "nix",
+                "eval",
+                "--raw",
+                "--extra-experimental-features",
+                "nix-command",
+                "-f",
+                path,
+                "composer.version",
+            ],
+            text=True,
+            stdout=subprocess.PIPE,
+        ).stdout.strip()
+        assert tuple(map(int, version.split("."))) >= (0, 11, 1)
diff --git a/tests/testpkgs/composer.nix b/tests/testpkgs/composer.nix
new file mode 100644
index 0000000..1643872
--- /dev/null
+++ b/tests/testpkgs/composer.nix
@@ -0,0 +1,17 @@
+{ fetchFromGitHub
+, php
+}:
+
+php.buildComposerProject (finalAttrs: {
+  pname = "castor";
+  version = "0.10.0";
+
+  src = fetchFromGitHub {
+    owner = "jolicode";
+    repo = "castor";
+    rev = "v${finalAttrs.version}";
+    hash = "sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=";
+  };
+
+  vendorHash = "sha256-AAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAAA=";
+})
diff --git a/tests/testpkgs/default.nix b/tests/testpkgs/default.nix
index 0da7f34..11003da 100644
--- a/tests/testpkgs/default.nix
+++ b/tests/testpkgs/default.nix
@@ -4,6 +4,7 @@
   bitbucket-snapshot = pkgs.callPackage ./bitbucket.nix { isSnapshot = true; };
   cargoLock.expand = pkgs.callPackage ./cargo-lock-expand { };
   cargoLock.update = pkgs.callPackage ./cargo-lock-update { };
+  composer = pkgs.callPackage ./composer.nix { };
   crate = pkgs.callPackage ./crate.nix { };
   gitea = pkgs.callPackage ./gitea.nix { };
   github = pkgs.callPackage ./github.nix { };