Project

General

Profile

action #10524

Updated by okurz over 5 years ago

## Motivation 
 Prevent sneaky errors confusing users by ignoring that values are unitialized. 

 ## Acceptance criteria 
 * **AC1:** Static code checks or unit tests in the tests distribution prevent the use of uninitialized values 

 ## Observation observation 
 https://openqa.suse.de/tests/195260/file/autoinst-log.txt 

 ``` 
 Use of uninitialized value $1 in numeric eq (==) at tests/console/zypper_up.pm line 23. 
 ... 
 Use of uninitialized value $1 in numeric eq (==) at tests/console/zypper_up.pm line 28. 

 ``` 

 ## steps to reproduce 
 probably rerun these tests 

 ## problem 
 There is no check for uninitialized values in tests which could cause unforeseen results. 

 ## suggestion 
 Prevent use of potentially uninitialized values by static code checks in tests distribution.

Back