PR#206: [tests] Check for TERM in environment before setting - gssproxy

gssproxy

#206 [tests] Check for TERM in environment before setting

Merged 6 years ago by cipherboy. Opened 6 years ago by cipherboy.

cipherboy/gssproxy fix_testsuite into master

[tests] Check for TERM in environment before setting

Alexander Scheel • 6 years ago

e175a80

proxy/tests/runtests.py

file modified

+3 -1

		`@@ -62,7 +62,9 @@`
		`keysenv = setup_keys(testdir, kdcenv)`

		`gssapienv = setup_gssapi_env(testdir, kdcenv)`
		`- gssapienv['TERM'] = os.environ['TERM']`
		`+`
		`+ if 'TERM' in os.environ:`
		`+ gssapienv['TERM'] = os.environ['TERM']`

		`gssproxylog = os.path.join(testdir, 'gssproxy.log')`

cipherboy commented 6 years ago

The presence of the TERM environment variable was not
checked before referencing, causing KeyError to be
raised when building on environments without TERM.

Fixes Jenkins build.

rebased

6 years ago

cipherboy commented 6 years ago

Commit a672e71 fixes this pull-request

Pull-Request has been merged by ascheel@redhat.com

6 years ago

cipherboy commented 6 years ago

Commit f7cd0ff fixes this pull-request

Pull-Request has been merged by ascheel@redhat.com

6 years ago

cipherboy commented 6 years ago