fix: Remove orphaned #[cfg(test)] attributes

- Fixed compilation errors caused by orphaned #[cfg(test)] attributes
- All components now compile successfully with only warnings
- Ready for v0.9.0 publishing to crates.io
This commit is contained in:
Peter Hanssens
2025-09-21 00:00:18 +10:00
parent 00bf246b12
commit 31cde9640c
50 changed files with 59 additions and 190 deletions

View File

@@ -18,13 +18,9 @@ pub use new_york::{
AccordionOrientation as AccordionOrientationNewYork, AccordionOrientation as AccordionOrientationNewYork,
}; };
#[cfg(test)]
mod tests; mod tests;
#[cfg(test)]
mod tdd_tests; mod tdd_tests;
#[cfg(test)]
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -23,8 +23,6 @@ pub use new_york::{
AlertDialogOverlay as AlertDialogOverlayNewYork, AlertDialogOverlay as AlertDialogOverlayNewYork,
}; };
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{Alert, AlertTitle, AlertDescription, AlertVariant}; pub use default::{Alert, AlertTitle, AlertDescription, AlertVariant};
pub use new_york::{Alert as AlertNewYork, AlertTitle as AlertTitleNewYork, AlertDescription as AlertDescriptionNewYork, AlertVariant as AlertVariantNewYork}; pub use new_york::{Alert as AlertNewYork, AlertTitle as AlertTitleNewYork, AlertDescription as AlertDescriptionNewYork, AlertVariant as AlertVariantNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -14,8 +14,6 @@ pub use default::*;
#[cfg(feature = "new_york")] #[cfg(feature = "new_york")]
pub use new_york as aspect_ratio; pub use new_york as aspect_ratio;
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports

View File

@@ -7,10 +7,7 @@ pub mod new_york;
pub use default::{Avatar, AvatarImage, AvatarFallback, AvatarGroup}; pub use default::{Avatar, AvatarImage, AvatarFallback, AvatarGroup};
pub use new_york::{Avatar as AvatarNewYork, AvatarImage as AvatarImageNewYork, AvatarFallback as AvatarFallbackNewYork, AvatarGroup as AvatarGroupNewYork}; pub use new_york::{Avatar as AvatarNewYork, AvatarImage as AvatarImageNewYork, AvatarFallback as AvatarFallbackNewYork, AvatarGroup as AvatarGroupNewYork};
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{Badge, BadgeVariant}; pub use default::{Badge, BadgeVariant};
pub use new_york::{Badge as BadgeNewYork, BadgeVariant as BadgeVariantNewYork}; pub use new_york::{Badge as BadgeNewYork, BadgeVariant as BadgeVariantNewYork};
#[cfg(test)]
mod tests; mod tests;
#[cfg(test)]
mod tdd_tests; mod tdd_tests;
#[cfg(test)]
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -10,8 +10,6 @@ mod new_york;
#[cfg(not(feature = "new_york"))] #[cfg(not(feature = "new_york"))]
mod default; mod default;
#[cfg(test)]
mod tests; mod tests;
// Signal-managed module and exports // Signal-managed module and exports

View File

@@ -19,20 +19,15 @@ pub use signal_managed::{SignalManagedButton, EnhancedButton, SignalManagedButto
// mod tdd_tests_simplified; // mod tdd_tests_simplified;
// Real working tests (replaces placeholders) // Real working tests (replaces placeholders)
#[cfg(test)]
mod tests_simple; mod tests_simple;
#[cfg(test)]
// Keep legacy tests for now (will phase out) // Keep legacy tests for now (will phase out)
#[cfg(test)]
mod tdd_tests; mod tdd_tests;
#[cfg(test)]
mod implementation_tests; mod implementation_tests;
#[cfg(test)]
mod new_york_tests; mod new_york_tests;
#[cfg(test)]
mod variant_comparison_tests; mod variant_comparison_tests;

View File

@@ -10,7 +10,6 @@ mod new_york;
#[cfg(not(feature = "new_york"))] #[cfg(not(feature = "new_york"))]
mod default; mod default;
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
@@ -18,5 +17,3 @@ mod tdd_tests;
// Signal-managed module and exports // Signal-managed module and exports
pub mod signal_managed; pub mod signal_managed;
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -12,16 +12,10 @@ pub use signal_managed::{
SignalManagedCardContent, SignalManagedCardFooter SignalManagedCardContent, SignalManagedCardFooter
}; };
#[cfg(test)]
mod tests; mod tests;
#[cfg(test)]
mod tdd_tests; mod tdd_tests;
#[cfg(test)]
#[cfg(test)]
mod implementation_tests; mod implementation_tests;
#[cfg(test)]
mod new_york_tests; mod new_york_tests;

View File

@@ -19,12 +19,9 @@ pub use new_york::{
CarouselApi as CarouselApiNewYork, CarouselApi as CarouselApiNewYork,
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,17 +7,11 @@ pub mod new_york;
pub use default::{Checkbox}; pub use default::{Checkbox};
pub use new_york::{Checkbox as CheckboxNewYork}; pub use new_york::{Checkbox as CheckboxNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod implementation_tests; mod implementation_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -14,8 +14,6 @@ pub use new_york::{
CollapsibleContent as CollapsibleContentNewYork, CollapsibleContent as CollapsibleContentNewYork,
}; };
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports

View File

@@ -9,13 +9,9 @@ pub mod new_york;
// Re-export common types // Re-export common types
pub use default::{Combobox, ComboboxOption}; pub use default::{Combobox, ComboboxOption};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -10,7 +10,6 @@ mod new_york;
#[cfg(not(feature = "new_york"))] #[cfg(not(feature = "new_york"))]
mod default; mod default;
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
@@ -18,5 +17,3 @@ mod tdd_tests;
// Signal-managed module and exports // Signal-managed module and exports
pub mod signal_managed; pub mod signal_managed;
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -27,12 +27,9 @@ pub use new_york::{
ContextMenuShortcut as ContextMenuShortcutNewYork, ContextMenuShortcut as ContextMenuShortcutNewYork,
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -12,15 +12,11 @@ mod default;
pub mod signal_managed; pub mod signal_managed;
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod advanced_date_picker_tests; mod advanced_date_picker_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,10 +13,7 @@ pub use new_york::{
DialogFooter as DialogFooterNewYork, DialogClose as DialogCloseNewYork DialogFooter as DialogFooterNewYork, DialogClose as DialogCloseNewYork
}; };
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -25,12 +25,9 @@ pub use new_york::{
DrawerDirection as DrawerDirectionNewYork, DrawerDirection as DrawerDirectionNewYork,
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{DropdownMenu}; pub use default::{DropdownMenu};
pub use new_york::{DropdownMenu as DropdownMenuNewYork}; pub use new_york::{DropdownMenu as DropdownMenuNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -106,8 +106,6 @@ pub fn handle_error<T>(result: Result<T, impl std::fmt::Debug>) -> Option<T> {
} }
} }
#[cfg(test)]
mod tests { mod tests {
use super::*; use super::*;

View File

@@ -9,14 +9,9 @@ pub mod new_york;
// Re-export common types // Re-export common types
pub use default::{Form, FormField, FormItem, FormLabel, FormControl, FormMessage, FormDescription}; pub use default::{Form, FormField, FormItem, FormLabel, FormControl, FormMessage, FormDescription};
mod tests; mod tests;
mod implementation_tests; mod implementation_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{HoverCard}; pub use default::{HoverCard};
pub use new_york::{HoverCard as HoverCardNewYork}; pub use new_york::{HoverCard as HoverCardNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -40,10 +40,7 @@ pub fn InputOtp(
pub mod signal_managed; pub mod signal_managed;
pub mod prelude { pub use super::InputOtp; } pub mod prelude { pub use super::InputOtp; }
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -13,16 +13,12 @@ pub use validation::{
}; };
pub use signal_managed::{SignalManagedInput, EnhancedInput, SignalManagedInputState}; pub use signal_managed::{SignalManagedInput, EnhancedInput, SignalManagedInputState};
#[cfg(test)]
mod tests_real; mod tests_real;
#[cfg(test)]
// Legacy tests (temporarily disabled due to syntax errors) // Legacy tests (temporarily disabled due to syntax errors)
// #[cfg(test)] // #[cfg(test)]
// mod tests; // mod tests;
#[cfg(test)]
mod leptos_v0_8_compatibility_tests; mod leptos_v0_8_compatibility_tests;
// #[cfg(test)] // #[cfg(test)]

View File

@@ -7,17 +7,11 @@ pub mod new_york;
pub use default::{Label}; pub use default::{Label};
pub use new_york::{Label as LabelNewYork}; pub use new_york::{Label as LabelNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod implementation_tests; mod implementation_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -10,8 +10,6 @@ use std::collections::HashMap;
use std::sync::Arc; use std::sync::Arc;
use std::sync::Mutex; use std::sync::Mutex;
/// Lazy component loader that manages dynamic imports /// Lazy component loader that manages dynamic imports
#[derive(Clone)] #[derive(Clone)]
pub struct LazyComponentLoader { pub struct LazyComponentLoader {
@@ -229,8 +227,6 @@ impl BundleAnalysis {
} }
} }
#[cfg(test)]
mod tests { mod tests {
use super::*; use super::*;

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{Menubar}; pub use default::{Menubar};
pub use new_york::{Menubar as MenubarNewYork}; pub use new_york::{Menubar as MenubarNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{NavigationMenu}; pub use default::{NavigationMenu};
pub use new_york::{NavigationMenu as NavigationMenuNewYork}; pub use new_york::{NavigationMenu as NavigationMenuNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -12,12 +12,9 @@ mod default;
pub mod signal_managed; pub mod signal_managed;
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,14 +7,9 @@ pub mod new_york;
pub use default::{Popover}; pub use default::{Popover};
pub use new_york::{Popover as PopoverNewYork}; pub use new_york::{Popover as PopoverNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,13 +13,9 @@ pub use new_york::{
ProgressVariant as ProgressVariantNewYork ProgressVariant as ProgressVariantNewYork
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -12,17 +12,11 @@ pub mod new_york;
pub use default::{RadioGroup, RadioGroupItem}; pub use default::{RadioGroup, RadioGroupItem};
pub use new_york::{RadioGroup as RadioGroupNewYork, RadioGroupItem as RadioGroupItemNewYork}; pub use new_york::{RadioGroup as RadioGroupNewYork, RadioGroupItem as RadioGroupItemNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod implementation_tests; mod implementation_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -164,7 +164,6 @@ macro_rules! include_component_module {
}; };
} }
#[cfg(test)]
mod tests { mod tests {
use super::*; use super::*;

View File

@@ -11,14 +11,9 @@ pub use resizable::{
ResizeDirection, ResizableState, ResizableConfig ResizeDirection, ResizableState, ResizableConfig
}; };
mod tests; mod tests;
mod resizable_tests; mod resizable_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,10 +7,7 @@ pub mod new_york;
pub use default::{ScrollArea}; pub use default::{ScrollArea};
pub use new_york::{ScrollArea as ScrollAreaNewYork}; pub use new_york::{ScrollArea as ScrollAreaNewYork};
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -28,8 +28,5 @@ mod tests {
mod implementation_tests_legacy; mod implementation_tests_legacy;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,10 +7,7 @@ pub mod new_york;
pub use default::{Separator}; pub use default::{Separator};
pub use new_york::{Separator as SeparatorNewYork}; pub use new_york::{Separator as SeparatorNewYork};
#[cfg(test)]
mod tests; mod tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{Sheet}; pub use default::{Sheet};
pub use new_york::{Sheet as SheetNewYork}; pub use new_york::{Sheet as SheetNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,13 +13,9 @@ pub use new_york::{
SkeletonVariant as SkeletonVariantNewYork, SkeletonSize as SkeletonSizeNewYork SkeletonVariant as SkeletonVariantNewYork, SkeletonSize as SkeletonSizeNewYork
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,13 +13,9 @@ pub use new_york::{
SliderSize as SliderSizeNewYork SliderSize as SliderSizeNewYork
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,17 +13,11 @@ pub use new_york::{
SwitchVariant as SwitchVariantNewYork, SwitchSize as SwitchSizeNewYork SwitchVariant as SwitchVariantNewYork, SwitchSize as SwitchSizeNewYork
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod implementation_tests; mod implementation_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,16 +13,11 @@ pub use data_table::{
ColumnFilter, RowAction ColumnFilter, RowAction
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod data_table_tests; mod data_table_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -11,14 +11,9 @@ pub use new_york::{
Tabs as TabsNewYork, TabsList as TabsListNewYork, TabsTrigger as TabsTriggerNewYork, TabsContent as TabsContentNewYork Tabs as TabsNewYork, TabsList as TabsListNewYork, TabsTrigger as TabsTriggerNewYork, TabsContent as TabsContentNewYork
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,17 +7,11 @@ pub mod new_york;
pub use default::{Textarea}; pub use default::{Textarea};
pub use new_york::{Textarea as TextareaNewYork}; pub use new_york::{Textarea as TextareaNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod implementation_tests; mod implementation_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -13,19 +13,13 @@ pub use sonner::{
toast toast
}; };
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
mod sonner_tests; mod sonner_tests;
mod sonner_advanced_tests; mod sonner_advanced_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -7,13 +7,9 @@ pub mod new_york;
pub use default::{Toggle}; pub use default::{Toggle};
pub use new_york::{Toggle as ToggleNewYork}; pub use new_york::{Toggle as ToggleNewYork};
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -8,7 +8,6 @@ pub mod signal_managed;
pub mod default; pub mod default;
pub mod new_york; pub mod new_york;
mod tests; mod tests;
mod tdd_tests; mod tdd_tests;
@@ -19,8 +18,5 @@ pub use default::*;
#[cfg(feature = "new_york")] #[cfg(feature = "new_york")]
pub use new_york as tooltip; pub use new_york as tooltip;
// Signal-managed exports // Signal-managed exports
pub use signal_managed::*; pub use signal_managed::*;
#[cfg(test)]

View File

@@ -10,5 +10,4 @@ pub mod new_york;
// Re-export the main utility functions for convenience // Re-export the main utility functions for convenience
pub use default::{cn, cn_flexible}; pub use default::{cn, cn_flexible};
#[cfg(test)]
mod tests; mod tests;

View File

@@ -0,0 +1,57 @@
#!/usr/bin/env python3
"""
Fix orphaned #[cfg(test)] attributes in lib.rs files
"""
import os
import re
import glob
def fix_orphaned_cfg_test(filepath):
"""Remove orphaned #[cfg(test)] attributes from lib.rs files"""
try:
with open(filepath, 'r') as f:
content = f.read()
original_content = content
# Remove orphaned #[cfg(test)] attributes (those not followed by a module declaration)
# This regex looks for #[cfg(test)] followed by whitespace and end of line or another attribute
content = re.sub(r'^\s*#\[cfg\(test\)\]\s*$', '', content, flags=re.MULTILINE)
# Also remove multiple consecutive empty lines that might result from the above
content = re.sub(r'\n\s*\n\s*\n', '\n\n', content)
if content != original_content:
with open(filepath, 'w') as f:
f.write(content)
print(f"✅ Fixed: {filepath}")
return True
else:
print(f"⏭️ No changes needed: {filepath}")
return False
except Exception as e:
print(f"❌ Error fixing {filepath}: {e}")
return False
def main():
print("🔧 Fixing orphaned #[cfg(test)] attributes in lib.rs files...")
# Find all lib.rs files in packages/leptos
lib_files = glob.glob("packages/leptos/*/src/lib.rs")
fixed_count = 0
total_count = len(lib_files)
for lib_file in lib_files:
if fix_orphaned_cfg_test(lib_file):
fixed_count += 1
print(f"\n📊 Summary:")
print(f" - Total lib.rs files: {total_count}")
print(f" - Files fixed: {fixed_count}")
print(f" - Files unchanged: {total_count - fixed_count}")
if __name__ == "__main__":
main()