starlingx/config
Code Issues Proposed changes

Enable Unused Variable

Browse Source 
Flake8 currently ignores:
F841: local variable is assigned to but never used

Pylint currently ignores:
W0612: Unused variable warning

Enable them for more thorough testing of code

Depends-On: I79eed868c5b42e2545ccf0c11dbbbea4336b9b19
Change-Id: I4c55186f939675cc2c3dc871ad03699d95c6091b
Story: 2004515
Task: 29303
Signed-off-by: Eric Barrett <eric.barrett@windriver.com>
This commit is contained in:
Eric Barrett 2019-06-10 10:37:34 -04:00 committed by Scott Little
parent ee342d3a7b
commit 55a26be559
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
tsconfig/tsconfig/tsconfig/tests/test_basics.py
View File

@ -114,7 +114,7 @@ vswitch_type=ovs-dpdk
# 1st: /etc/build.info
# 2nd: /etc/platform/platform.conf
mock_open.return_value = io.StringIO(self.mock_malformed_build)
from tsconfig import tsconfig
from tsconfig import tsconfig # pylint: disable=unused-variable
mock_logging_exception.assert_called_once()
# This tests the behaviour when the platform.conf is missing
@ -129,7 +129,7 @@ vswitch_type=ovs-dpdk
# 1st: /etc/build.info
# 2nd: /etc/platform/platform.conf
mock_open.return_value = io.StringIO(self.mock_19_01_build)
from tsconfig import tsconfig
from tsconfig import tsconfig # pylint: disable=unused-variable
mock_logging_exception.assert_called_once()
# This tests the behaviour when the platform.conf is empty
@ -145,7 +145,7 @@ vswitch_type=ovs-dpdk
# 2nd: /etc/platform/platform.conf
mock_open.side_effect = [io.StringIO(self.mock_19_01_build),
io.StringIO(self.mock_platform_conf_empty)]
from tsconfig import tsconfig
from tsconfig import tsconfig # pylint: disable=unused-variable
mock_logging_exception.assert_called_once()
# This tests the behaviour when the platform.conf has the minimal entries