Skip to content

Commit ec5dc94

Browse files
committed
Improved test to check separately different default behaviors
1 parent ada91d9 commit ec5dc94

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

test/framework/github.py

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1329,12 +1329,14 @@ def test_github_create_test_report(self):
13291329
'REMOVEME': 'test-abc',
13301330
'ReMoVeMe2': 'TeSt-xyz',
13311331

1332-
'AWS_ACCESS_KEY': 'AKIAIOSFODNN7EXAMPLE',
1333-
'AWS_SECRET_KEY': 'wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY',
1334-
'JWT': '.'.join([JWT_HDR, JWT_PLD, JWT_SIG]),
1335-
'GH_TOKEN': 'ghp_123456789_ABCDEFGHIJKlmnopqrstuvwxyz',
1336-
'SLACK_TOKEN': 'xoxb-1234567890-1234567890123-ABCDEFabcdef',
1337-
1332+
# Test default removal based on variable value
1333+
'TOTALLYPUBLICVAR1': 'AKIAIOSFODNN7EXAMPLE', # AWS_ACCESS_KEY
1334+
'TOTALLYPUBLICVAR2': 'wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY', # AWS_SECRET_KEY
1335+
'TOTALLYPUBLICVAR3': '.'.join([JWT_HDR, JWT_PLD, JWT_SIG]), # JWT
1336+
'TOTALLYPUBLICVAR4': 'ghp_123456789_ABCDEFGHIJKlmnopqrstuvwxyz', # GH_TOKEN
1337+
'TOTALLYPUBLICVAR5': 'xoxb-1234567890-1234567890123-ABCDEFabcdef', # SLACK_TOKEN
1338+
1339+
# Test default removal based on variable name
13381340
'API_SOMETHING': '1234567890',
13391341
'MY_PASSWORD': '1234567890',
13401342
'ABC_TOKEN': '1234567890',
@@ -1374,11 +1376,11 @@ def test_github_create_test_report(self):
13741376
]
13751377
# Test that known token regexes for ENV vars are excluded by default
13761378
exclude_patterns2 = [
1377-
'AWS_ACCESS_KEY',
1378-
'AWS_SECRET_KEY',
1379-
'JWT',
1380-
'GH_TOKEN',
1381-
'SLACK_TOKEN',
1379+
'TOTALLYPUBLICVAR1',
1380+
'TOTALLYPUBLICVAR2',
1381+
'TOTALLYPUBLICVAR3',
1382+
'TOTALLYPUBLICVAR4',
1383+
'TOTALLYPUBLICVAR5',
13821384

13831385
'API_SOMETHING',
13841386
'MY_PASSWORD',

0 commit comments

Comments
 (0)