chore(former): Comment out debug attributes in tests

Author: wanguardd

module/core/former/examples/former_custom_subform_scalar.rs

@@ -37,8 +37,8 @@ fn main()
 
   // Child struct with Former derived for builder pattern support
   #[ derive( Debug, PartialEq, Former ) ]
-  // Optional: Use `#[debug]` to expand and debug generated code.
-  // #[debug]
+  // Optional: Use `#[ debug ]` to expand and debug generated code.
+  // #[ debug ]
   pub struct Child
   {
     name : String,
@@ -47,8 +47,8 @@ fn main()
 
   // Parent struct designed to hold a single Child instance using subform scalar
   #[ derive( Debug, PartialEq, Former ) ]
-  // Optional: Use `#[debug]` to expand and debug generated code.
-  // #[debug]
+  // Optional: Use `#[ debug ]` to expand and debug generated code.
+  // #[ debug ]
   pub struct Parent
   {
     // The `subform_scalar` attribute is used to specify that the 'child' field has its own former

module/core/former/tests/inc/enum_named_tests/enum_named_fields_named_derive.rs

@@ -16,7 +16,7 @@
 //!
 //! Test Relevance/Acceptance Criteria:
 //! - Defines an enum `EnumWithNamedFields` with named variants covering zero, one, and two fields.
-//! - Applies `#[derive(Former)]`, `#[debug]`, and `#[standalone_constructors]` to the enum.
+//! - Applies `#[derive(Former)]`, `#[ debug ]`, and `#[standalone_constructors]` to the enum.
 //! - Applies `#[scalar]` and `#[subform_scalar]` to relevant variants.
 //! - Includes shared test logic from `enum_named_fields_named_only_test.rs`.
 //! - The included tests call the derived static methods (e.g., `EnumWithNamedFields::variant_zero_scalar()`, `EnumWithNamedFields::variant_one_scalar()`, `EnumWithNamedFields::variant_one_subform()`, etc.) and standalone constructors (e.g., `standalone_variant_zero_scalar()`).
@@ -33,7 +33,7 @@ pub struct InnerForSubform {
 
 // Define the enum with named field variants for testing.
 #[ derive( Debug, PartialEq, former::Former ) ]
-#[ debug ]
+// #[ debug ]
 #[ standalone_constructors ]
 pub enum EnumWithNamedFields
 {

module/core/former/tests/inc/enum_named_tests/standalone_constructor_args_named_derive.rs

@@ -13,7 +13,7 @@
 //!
 //! Test Relevance/Acceptance Criteria:
 //! - Defines an enum `TestEnumArgs` with single-field (`StructVariantArgs { field: String }`) and multi-field (`MultiStructArgs { a: i32, b: bool }`) named variants.
-//! - Applies `#[derive(Former)]`, `#[standalone_constructors]`, and `#[debug]` to the enum.
+//! - Applies `#[derive(Former)]`, `#[standalone_constructors]`, and `#[ debug ]` to the enum.
 //! - Applies `#[arg_for_constructor]` to the fields within both variants.
 //! - Includes shared test logic from `standalone_constructor_args_named_only_test.rs`.
 //! - The included tests call the derived standalone constructor functions (`struct_variant_args(value)`, `multi_struct_args(value1, value2)`) and assert that the returned enum instances match manually constructed expected values. This verifies that the standalone constructors are generated correctly as scalar functions when all fields have `#[arg_for_constructor]`.

module/core/former/tests/inc/enum_unit_tests/enum_named_fields_unit_derive.rs

@@ -10,7 +10,7 @@
 //! Test Relevance/Acceptance Criteria:
 //! - Defines an enum `EnumWithNamedFields` with unit variants `UnitVariantDefault` and `UnitVariantScalar`,
 //!   using named fields syntax (`{}`). `UnitVariantScalar` has the `#[scalar]` attribute. The enum has
-//!   `#[derive(Former)]`, `#[debug]`, and `#[standalone_constructors]`.
+//!   `#[derive(Former)]`, `#[ debug ]`, and `#[standalone_constructors]`.
 //! - Relies on the derived static methods (`EnumWithNamedFields::unit_variant_scalar()`, `EnumWithNamedFields::unit_variant_default()`)
 //!   defined in `enum_named_fields_unit_only_test.rs`.
 //! - Asserts that these constructors produce the correct `EnumWithNamedFields` enum instances by comparing
@@ -20,7 +20,7 @@ use super::*;
 
 // Define the enum with unit variants for testing.
 #[ derive( Debug, PartialEq, former::Former ) ]
-#[ debug ]
+// #[ debug ]
 #[ standalone_constructors ]
 pub enum EnumWithNamedFields
 {

module/core/former/tests/inc/enum_unit_tests/generic_enum_simple_unit_derive.rs

@@ -7,7 +7,7 @@
 //! - Rule 1a (Unit + `#[scalar]`): Verifies `EnumOuter::<T>::other_variant() -> EnumOuter<T>` (as default for unit is scalar) for a generic enum.
 //!
 //! Test Relevance/Acceptance Criteria:
-//! - Defines a generic enum `EnumOuter<X: Copy>` with a unit variant `OtherVariant`, and the `#[derive(Former)]` and `#[debug]` attributes.
+//! - Defines a generic enum `EnumOuter<X: Copy>` with a unit variant `OtherVariant`, and the `#[derive(Former)]` and `#[ debug ]` attributes.
 //! - Relies on the derived static method `EnumOuter::<MyType>::other_variant()`.
 //! - Asserts that the `got` instance is equal to an `expected` instance, which is manually
 //!   constructed as `EnumOuter::<MyType>::OtherVariant`. This confirms the constructor produces the correct variant instance for a generic enum.
@@ -19,7 +19,7 @@ use core::fmt::Debug; // Import Debug trait for bounds
 // --- Enum Definition with Bounds ---
 // Apply Former derive here. This is what we are testing.
 #[derive(Debug, PartialEq, former::Former)]
-// #[debug]
+// #[ debug ]
 pub enum EnumOuter< X : Copy + Debug + PartialEq > // Enum bound: Copy + Debug + PartialEq
 {
   // --- Unit Variant ---

module/core/former/tests/inc/enum_unit_tests/generics_in_tuple_variant_unit_derive.rs

@@ -7,7 +7,7 @@
 //! - Rule 1a (Unit + `#[scalar]`): Verifies `EnumOuter::<T>::other_variant() -> EnumOuter<T>` (as default for unit is scalar) for a generic enum.
 //!
 //! Test Relevance/Acceptance Criteria:
-//! - Defines a generic enum `EnumOuter<X: Copy>` with a unit variant `OtherVariant`, and the `#[derive(Former)]` and `#[debug]` attributes.
+//! - Defines a generic enum `EnumOuter<X: Copy>` with a unit variant `OtherVariant`, and the `#[derive(Former)]` and `#[ debug ]` attributes.
 //! - Relies on the derived static method `EnumOuter::<MyType>::other_variant()`.
 //! - Asserts that the `got` instance is equal to an `expected` instance, which is manually
 //!   constructed as `EnumOuter::<MyType>::OtherVariant`. This confirms the constructor produces the correct variant instance for a generic enum.
@@ -19,7 +19,7 @@ use std::marker::PhantomData; // Import PhantomData
 // --- Enum Definition with Bounds ---
 // Apply Former derive here. This is what we are testing.
 #[derive(Debug, PartialEq, former::Former)]
-// #[debug]
+// #[ debug ]
 pub enum EnumOuter< X : Copy > // Enum bound: Copy
 {
   // --- Unit Variant ---

module/core/former/tests/inc/enum_unnamed_tests/enum_named_fields_unnamed_derive.rs

@@ -9,7 +9,7 @@
 //!
 //! Test Relevance/Acceptance Criteria:
 //! - Defines an enum `EnumWithNamedFields` with two zero-field unnamed variants: `VariantZeroUnnamedDefault()` and `VariantZeroUnnamedScalar()`.
-//! - `VariantZeroUnnamedScalar` is annotated with `#[scalar]`. The enum has `#[derive(Former)]`, `#[debug]`, and `#[standalone_constructors]`.
+//! - `VariantZeroUnnamedScalar` is annotated with `#[scalar]`. The enum has `#[derive(Former)]`, `#[ debug ]`, and `#[standalone_constructors]`.
 //! - Relies on the derived static methods (`EnumWithNamedFields::variant_zero_unnamed_scalar()`, `EnumWithNamedFields::variant_zero_unnamed_default()`)
 //!   defined in `enum_named_fields_unnamed_only_test.rs`.
 //! - Asserts that these constructors produce the correct `EnumWithNamedFields` enum instances by comparing

module/core/former/tests/inc/enum_unnamed_tests/generics_in_tuple_variant_tuple_derive.rs

@@ -9,7 +9,7 @@
 //! Test Relevance/Acceptance Criteria:
 //! - Defines a generic enum `EnumOuter<X: Copy + Debug + Default + PartialEq>` with a single-field tuple variant `Variant(InnerGeneric<X>)`.
 //! - The inner struct `InnerGeneric<T: Debug + Copy + Default + PartialEq>` has its own generic `T` and bounds, and is instantiated with the enum's generic `X` in the variant.
-//! - The enum has `#[derive(Former)]` and `#[debug]`.
+//! - The enum has `#[derive(Former)]` and `#[ debug ]`.
 //! - Relies on the derived static method `EnumOuter::<X>::variant()` provided by this file (via `include!`).
 //! - Asserts that this constructor returns the expected subformer (`InnerGenericFormer<X>`) and that using the subformer's setter (`.inner_field()`) and `.form()` results in the correct `EnumOuter` enum instance.
 //! - Verifies that the bounds (`Copy`, `Debug`, `Default`, `PartialEq`) are correctly handled by using types that satisfy them.
@@ -35,7 +35,7 @@ impl< T : Debug + Copy + Default + PartialEq > From< T > for InnerGeneric< T >
 // --- Enum Definition with Bounds ---
 // Apply Former derive here. This is what we are testing.
 #[derive(Debug, PartialEq, former::Former)]
-// #[debug]
+// #[ debug ]
 pub enum EnumOuter< X : Copy + Debug + Default + PartialEq > // Enum bound: Copy
 {
   // --- Tuple Variant with Generics ---

plan.md

@@ -1,14 +1,15 @@
 # Project Plan: Audit, Improve, and Run Clippy Lints for `former` Crate
 
 ### Goal
-*   Audit, improve, and run `module/core/former/task_clippy_lints.md` to ensure it follows codestyle rules, has concise documentation, and avoids breaking the working crate. **Additionally, ensure `cargo test` passes for the `former` crate without any warnings.**
+*   Audit, improve, and run `module/core/former/task_clippy_lints.md` to ensure it follows codestyle rules, has concise documentation, and avoids breaking the working crate. **Additionally, ensure `cargo test` passes for the `former` crate without any warnings and without debug output from `#[ debug ]` attributes.**
 
 ### Progress
 *   🚀 Increment 1 Complete
 *   🚀 Increment 2 Complete
 *   🚀 Increment 3 Complete
 *   🚀 Increment 4 Complete
 *   🚀 Increment 5 Complete
+*   🚀 Increment 6 Complete
 
 ### Target Crate
 *   `module/core/former`
@@ -19,16 +20,15 @@
     *   `module/core/former/Cargo.toml`
     *   `module/core/former/src/lib.rs`
     *   `Cargo.toml` (workspace root)
-    *   `module/core/former/tests/inc/enum_unit_tests/generic_enum_simple_unit_derive.rs`
-    *   `module/core/former/tests/inc/enum_unnamed_tests/tuple_zero_fields_derive.rs`
-    *   `module/core/former/tests/inc/enum_unnamed_tests/tuple_zero_fields_manual.rs`
+    *   All test files within `module/core/former/tests/` that contain `#[ debug ]`.
 *   Crates for Documentation:
     *   `former`
 
 ### Expected Behavior Rules / Specifications (for Target Crate)
 *   The `module/core/former/task_clippy_lints.md` file should be well-formatted, concise, and adhere to the codestyle rules.
 *   The `module/core/former` crate should compile without warnings when `cargo clippy -p former` is run with the recommended lints.
-*   `cargo test -p former` should pass without errors **and without any warnings.**
+*   `cargo test -p former` should pass without errors and without any warnings.
+*   **`cargo test -p former` should not produce any debug output related to `#[ debug ]` attributes.**
 *   No existing knowledge or functionality should be lost or broken.
 
 ### Increments
@@ -80,6 +80,16 @@
     *   Verification Strategy: Execute `cargo test -p former` via `execute_command` and analyze output for warnings.
     *   Commit Message: `fix(former): Resolve cargo test warnings`
 
+*   ✅ Increment 6: Comment out active `#[ debug ]` attributes in `former` crate tests.
+    *   Detailed Plan Step 1: Search for `#[ debug ]` in `module/core/former/tests/` directory.
+    *   Detailed Plan Step 2: For each file found, read its content.
+    *   Detailed Plan Step 3: Comment out all occurrences of `#[ debug ]` attributes.
+    *   Pre-Analysis: `#[ debug ]` attributes are used for macro debugging and should not be active in final code.
+    *   Crucial Design Rules: [Enhancements: Only Implement What’s Requested].
+    *   Relevant Behavior Rules: `cargo test -p former` should not produce debug output.
+    *   Verification Strategy: Execute `cargo test -p former` via `execute_command` and visually inspect output for debug messages.
+    *   Commit Message: `chore(former): Comment out debug attributes in tests`
+
 ### Task Requirements
 *   Do only conservative changes.
 *   Avoid breaking working crate.