Added tests to buffs and badinputtracker packages

[Jasrags]

Description

Added generated tests to buffs and badinputtracker packages.

Changes

Provide a bullet point list of noteworthy changes in this Pull Request:

• github.com/GoMudEngine/GoMud/internal/buffs 0.306s coverage: 25.8% of statements
• github.com/GoMudEngine/GoMud/internal/badinputtracker 0.351s coverage: 100.0% of statements

[Copilot]

Pull Request Overview

This PR introduces refactored logic in the buffs package and adds comprehensive unit tests for the badinputtracker package, improving test coverage for bad-input tracking.

• Simplified clamping of RoundInterval in BuffSpec.GetValue and corrected a typo in the SearchBuffs comment.
• Generated and added extensive tests for TrackBadCommand, GetBadCommands, and Clear in badinputtracker.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File	Description
internal/buffs/buffspec.go	Replace manual clamping with max helper; fix comment typo
internal/badinputtracker/badinputtracker_test.go	Add generated tests covering all bad-input tracking behaviors

Comments suppressed due to low confidence (3)

internal/buffs/buffspec.go:98

• The refactored GetValue logic (particularly negative RoundInterval handling) isn't covered by existing tests. Consider adding unit tests for edge cases like RoundInterval > 5 and RoundInterval == 5.

func (b *BuffSpec) GetValue() int {

internal/buffs/buffspec.go:151

• The new SearchBuffs function lacks unit tests. Consider adding tests for trimming, case-insensitivity, and partial matches to validate its behavior.

func SearchBuffs(searchTerm string) []int {

internal/buffs/buffspec.go:99

• The max function may be undefined in this context. Ensure you have a helper for integer max or inline equivalent logic to prevent an undefined identifier error.

freqVal := max(5-b.RoundInterval, 0)