test(libstore): more s3 url tests

Author: lovesegfault

tests/nixos/s3-binary-cache-store.nix

@@ -82,15 +82,29 @@ in
 
       # Test that the format string in the error message is properly setup and won't display `%s` instead of the failed URI
       msg = client.fail("${env} nix eval --impure --expr 'builtins.fetchurl { name = \"foo\"; url = \"${objectThatDoesNotExist}\"; }' 2>&1")
-      if "S3 object '${objectThatDoesNotExist}' does not exist" not in msg:
+      if "unable to download '${objectThatDoesNotExist}': HTTP error 404" not in msg:
         print(msg) # So that you can see the message that was improperly formatted
         raise Exception("Error message formatting didn't work")
 
       # Copy a package from the binary cache.
       client.fail("nix path-info ${pkgA}")
 
+      # Test nix store info with various S3 URL formats
       client.succeed("${env} nix store info --store '${storeUrl}' >&2")
 
+      # Test with different region parameter (should work without warnings)
+      client.succeed("${env} nix store info --store 's3://my-cache?endpoint=http://server:9000&region=us-east-2' >&2")
+
+      # Test with just bucket name and region
+      client.succeed("${env} nix store info --store 's3://my-cache?region=eu-west-1&endpoint=http://server:9000' >&2")
+
+      # Test that store info shows the store is available
+      info = client.succeed("${env} nix store info --json --store '${storeUrl}'")
+      import json
+      store_info = json.loads(info)
+      assert store_info.get("url"), "Store should have a URL"
+      print(f"Store URL: {store_info.get('url')}")
+
       client.succeed("${env} nix copy --no-check-sigs --from '${storeUrl}' ${pkgA}")
 
       client.succeed("nix path-info ${pkgA}")