fix(11-01): update hardcoded count assertions to reflect 35 traffic classes
- Rename TestNewBankHas14Layers to TestNewBankHasAllLayers, use len(classify.AllClasses()) - Update config_test.go: replace 4 hardcoded 14 counts with len(classify.AllClasses()) - Update TestPrintConfigContainsAllClasses: replace hardcoded list with AllClasses() loop - Update classifier_test.go: change TestAllClassesCount want from 32 to 35
This commit is contained in:
+3
-3
@@ -7,10 +7,10 @@ import (
|
||||
"github.com/netsynth/netsynth/classify"
|
||||
)
|
||||
|
||||
func TestNewBankHas14Layers(t *testing.T) {
|
||||
func TestNewBankHasAllLayers(t *testing.T) {
|
||||
b := NewBank(1.0, ClassFreqConfigs)
|
||||
if len(b.layers) != 14 {
|
||||
t.Errorf("NewBank() has %d layers, want 14", len(b.layers))
|
||||
if len(b.layers) != len(classify.AllClasses()) {
|
||||
t.Errorf("NewBank() has %d layers, want %d", len(b.layers), len(classify.AllClasses()))
|
||||
}
|
||||
// Verify each class has exactly one layer
|
||||
for _, class := range classify.AllClasses() {
|
||||
|
||||
Reference in New Issue
Block a user