SecUseCooldownComponent - fix bug with 0 seconds

- Ref #776

Author: ShaneBeee

src/main/java/com/shanebeestudios/skbee/elements/itemcomponent/sections/SecUseCooldownComponent.java

@@ -76,6 +76,8 @@ public boolean init(Expression<?>[] exprs, int matchedPattern, Kleenean isDelaye
             Timespan timespan = this.seconds.getSingle(event);
             if (timespan != null) seconds = (float) timespan.getAs(Timespan.TimePeriod.MILLISECOND) / 1000;
         }
+        // Minecraft requires > 0, so lowest we'll go is 1 tick
+        seconds = Math.clamp(seconds, 0.05f, Float.MAX_VALUE);
 
         UseCooldown.Builder builder = UseCooldown.useCooldown(seconds);
 

src/test/scripts/general/item-component.sk

@@ -283,6 +283,12 @@ test "SkBee - Item Component test" when running minecraft "1.21.3":
 		group: "blah:special_apple"
 	assert {_i} has item component minecraft:use_cooldown with "The apple should have a use cooldown component"
 
+	set {_i} to 1 of stick
+	apply use cooldown to {_i}:
+		seconds: 0 seconds # Testing a previous bug
+		group: "sovde_found_this_bug_so_he_gets_a_custom_group"
+	assert {_i} has item component minecraft:use_cooldown with "The apple should have a use cooldown component"
+
 	# Use Remainder
 	set {_i} to 1 of honey bottle
 	assert use remainder component of {_i} is glass bottle with "Default use remainder of a honey bottle should be a glass bottle"