Skip to content

Commit db7f09f

Browse files
authored
Fix yaml and toml tests in test_config (#1253)
* Fix yaml and toml tests in test_config * Related to #1159 * Add test_config back to updated CI Two of the tests in test_config were failing, presumably because they'd gotten out of date when we changed the default for extraction. These tests were not being run in CI (thanks to @Molkree for noticing that) so they didn't get fixed in a timely manner. Signed-off-by: Terri Oda <[email protected]>
1 parent 6248aef commit db7f09f

File tree

2 files changed

+1
-5
lines changed

2 files changed

+1
-5
lines changed

.github/workflows/pythonapp.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,6 @@ jobs:
137137
--ignore=test/test_cli.py
138138
--ignore=test/test_cvedb.py
139139
--ignore=test/test_requirements.py
140-
--ignore=test/test_config.py
141140
- name: Run synchronous tests
142141
run: >
143142
pytest -v
@@ -202,7 +201,6 @@ jobs:
202201
--ignore=test/test_cli.py
203202
--ignore=test/test_cvedb.py
204203
--ignore=test/test_requirements.py
205-
--ignore=test/test_config.py
206204
- name: Run synchronous tests
207205
run: >
208206
pytest -v --cov --cov-append --cov-report=xml
@@ -263,7 +261,6 @@ jobs:
263261
--ignore=test/test_cvedb.py
264262
--ignore=test/test_requirements.py
265263
--ignore=test/test_helper_script.py
266-
--ignore=test/test_config.py
267264
- name: Run synchronous tests
268265
run: >
269266
pytest -v

test/test_config.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,7 @@ class TestConfig:
1515
"cvss": 0,
1616
"directory": "test/assets",
1717
"disable_version_check": False,
18-
"exclude": [],
19-
"extract": False,
18+
"extract": True,
2019
"format": "console",
2120
"input_file": "test/csv/triage.csv",
2221
"log_level": "debug",

0 commit comments

Comments
 (0)