@@ -130,19 +130,45 @@ func TestValidateFields(t *testing.T) {
130130 wantErr : true ,
131131 },
132132 {
133- name : "Missing multiple fields " ,
133+ name : "Missing Script " ,
134134 fields : Fields {
135- ServiceID : "test" , Script : "test" ,
135+ ServiceID : "test" , TagPrefix : "test" , Interval : "test" , User : "test" , Group : "test" ,
136+ },
137+ wantErr : true ,
138+ },
139+ {
140+ name : "Empty Script" ,
141+ fields : Fields {
142+ ServiceID : "test" , Script : "" , TagPrefix : "test" , Interval : "test" , User : "test" , Group : "test" ,
143+ },
144+ wantErr : true ,
145+ },
146+ {
147+ name : "Missing multiple fields including Script" ,
148+ fields : Fields {
149+ ServiceID : "test" , TagPrefix : "test" ,
136150 },
137151 wantErr : true ,
138152 },
139153 }
140154
141155 for _ , tt := range tests {
142156 t .Run (tt .name , func (t * testing.T ) {
143- if err := validateFields (& tt .fields ); (err != nil ) != tt .wantErr {
157+ err := validateFields (& tt .fields )
158+ if (err != nil ) != tt .wantErr {
144159 t .Errorf ("validateFields() error = %v, wantErr %v" , err , tt .wantErr )
145160 }
161+ if tt .wantErr {
162+ if err == nil {
163+ t .Errorf ("validateFields() expected error, got nil" )
164+ } else {
165+ if tt .name == "Missing Script" || tt .name == "Empty Script" {
166+ if ! strings .Contains (err .Error (), "Script" ) {
167+ t .Errorf ("validateFields() error does not mention 'Script': %v" , err )
168+ }
169+ }
170+ }
171+ }
146172 })
147173 }
148174}
0 commit comments