aboutsummaryrefslogtreecommitdiffhomepage
path: root/crates/shirabe/tests/command/init_command_test.rs
diff options
context:
space:
mode:
authornsfisis <nsfisis@gmail.com>2026-06-27 19:44:01 +0900
committernsfisis <nsfisis@gmail.com>2026-06-27 20:18:56 +0900
commite63ccb5cbb479df0f0e5010065ff57bbf4ca238f (patch)
treef91c7db9b3475e845b023f51cef2d6379e80ae5e /crates/shirabe/tests/command/init_command_test.rs
parent3cb13f1d7025351615c254b9225a4c5d1bcbd170 (diff)
downloadphp-shirabe-e63ccb5cbb479df0f0e5010065ff57bbf4ca238f.tar.gz
php-shirabe-e63ccb5cbb479df0f0e5010065ff57bbf4ca238f.tar.zst
php-shirabe-e63ccb5cbb479df0f0e5010065ff57bbf4ca238f.zip
fix(json): encode empty stdClass fields as {} not []
Empty PhpMixed::Array now serializes as [], so the several places that build a PHP `new \stdClass` via an empty Array were emitting [] where Composer writes {}. Use PhpMixed::Object for those empty-object cases: - InitCommand: require / require-dev - Locker::fixupJsonDataType: stability-flags / platform / platform-dev - JsonConfigSource fallback: require/config keys that must stay objects Align the affected test expectations with Composer: JsonFile::read() decodes with assoc=true, collapsing the on-disk {} back to [], so reads expect []. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Diffstat (limited to 'crates/shirabe/tests/command/init_command_test.rs')
-rw-r--r--crates/shirabe/tests/command/init_command_test.rs33
1 files changed, 15 insertions, 18 deletions
diff --git a/crates/shirabe/tests/command/init_command_test.rs b/crates/shirabe/tests/command/init_command_test.rs
index efb3f49..32226f6 100644
--- a/crates/shirabe/tests/command/init_command_test.rs
+++ b/crates/shirabe/tests/command/init_command_test.rs
@@ -159,9 +159,6 @@ fn test_namespace_from_missing_package_name() {
}
/// @return iterable<string, array{0: array<string, mixed>, 1: array<string, mixed>}>
-///
-/// Empty `require` is `{}` (PHP's `new \stdClass`), not `[]`, to match what the command writes:
-/// shirabe's `json_decode` keeps the empty object/array distinction that PHP collapses.
fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
vec![
// name argument
@@ -169,7 +166,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
}),
vec![opt("--name", "test/pkg")],
),
@@ -177,7 +174,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
(
serde_json::json!({
"name": "test/pkg",
- "require": {},
+ "require": [],
"authors": [{ "name": "Mr. Test", "email": "test@example.org" }],
}),
vec![
@@ -189,7 +186,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
(
serde_json::json!({
"name": "test/pkg",
- "require": {},
+ "require": [],
"authors": [{ "name": "Mr. Test" }],
}),
vec![opt("--name", "test/pkg"), opt("--author", "Mr. Test")],
@@ -199,7 +196,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"repositories": [{ "type": "vcs", "url": "http://packages.example.com" }],
}),
vec![
@@ -215,7 +212,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"repositories": [
{ "type": "vcs", "url": "http://vcs.example.com" },
{ "type": "composer", "url": "http://composer.example.com" },
@@ -243,7 +240,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"minimum-stability": "dev",
}),
vec![opt("--name", "test/pkg"), opt("--stability", "dev")],
@@ -277,7 +274,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"require-dev": { "first/pkg": "1.0.0" },
}),
vec![
@@ -290,7 +287,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"require-dev": { "first/pkg": "1.0.0", "second/pkg": "^3.4" },
}),
vec![
@@ -303,7 +300,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"autoload": { "psr-4": { "Test\\Pkg\\": "testMapping/" } },
}),
vec![opt("--name", "test/pkg"), opt("--autoload", "testMapping/")],
@@ -313,7 +310,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"homepage": "https://example.org/",
}),
vec![
@@ -326,7 +323,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"description": "My first example package",
}),
vec![
@@ -339,7 +336,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"type": "project",
}),
vec![opt("--name", "test/pkg"), opt("--type", "project")],
@@ -349,7 +346,7 @@ fn run_data_provider() -> Vec<(serde_json::Value, Vec<(PhpMixed, PhpMixed)>)> {
serde_json::json!({
"name": "test/pkg",
"authors": [default_authors()],
- "require": {},
+ "require": [],
"license": "MIT",
}),
vec![opt("--name", "test/pkg"), opt("--license", "MIT")],
@@ -510,7 +507,7 @@ fn test_run_guess_name_from_dir_sanitizes_dir() {
let expected = serde_json::json!({
"name": "vendor-name/foo-bar_baz.qux",
"authors": [default_authors()],
- "require": {},
+ "require": [],
});
assert_eq!(
expected,
@@ -562,7 +559,7 @@ fn test_interactive_run() {
"license": "AGPL-3.0-only",
"authors": [{ "name": "Mr. Test", "email": "test@example.org" }],
"minimum-stability": "stable",
- "require": {},
+ "require": [],
});
assert_eq!(expected, read_composer_json(&dir));
}