Skip to content

Commit 4de30a0

Browse files
NettozxNettozx
committed
fix shadow var and format issues
1 parent c0ebf70 commit 4de30a0

2 files changed

Lines changed: 54 additions & 53 deletions

File tree

‎test/cli/sarif_test.py‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,7 @@ def run_sarif_check(code, extra_args=None):
123123
return sarif_data
124124
except json.JSONDecodeError as e:
125125
pytest.fail(f"Failed to parse SARIF JSON: {e}\nOutput: {stderr}")
126+
return None
126127

127128

128129
def test_sarif_basic_structure():

‎test/testsarifreport.cpp‎

Lines changed: 53 additions & 53 deletions
Original file line numberDiff line numberDiff line change
@@ -101,11 +101,11 @@ class TestSarifReport : public TestFixture
101101
const picojson::array& runs = root.at("runs").get<picojson::array>();
102102
ASSERT_EQUALS(1U, runs.size());
103103

104-
const picojson::object& run = runs[0].get<picojson::object>();
105-
const picojson::array& results = run.at("results").get<picojson::array>();
104+
const picojson::object& cur_run = runs[0].get<picojson::object>();
105+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
106106
ASSERT_EQUALS(0U, results.size());
107107

108-
const picojson::object& tool = run.at("tool").get<picojson::object>();
108+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
109109
const picojson::object& driver = tool.at("driver").get<picojson::object>();
110110
const picojson::array& rules = driver.at("rules").get<picojson::array>();
111111
ASSERT_EQUALS(0U, rules.size());
@@ -122,10 +122,10 @@ class TestSarifReport : public TestFixture
122122

123123
const picojson::object& root = json.get<picojson::object>();
124124
const picojson::array& runs = root.at("runs").get<picojson::array>();
125-
const picojson::object& run = runs[0].get<picojson::object>();
125+
const picojson::object& cur_run = runs[0].get<picojson::object>();
126126

127127
// Check results
128-
const picojson::array& results = run.at("results").get<picojson::array>();
128+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
129129
ASSERT_EQUALS(1U, results.size());
130130

131131
const picojson::object& result = results[0].get<picojson::object>();
@@ -136,7 +136,7 @@ class TestSarifReport : public TestFixture
136136
ASSERT_EQUALS("Null pointer dereference", message.at("text").get<std::string>());
137137

138138
// Check rules
139-
const picojson::object& tool = run.at("tool").get<picojson::object>();
139+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
140140
const picojson::object& driver = tool.at("driver").get<picojson::object>();
141141
const picojson::array& rules = driver.at("rules").get<picojson::array>();
142142
ASSERT_EQUALS(1U, rules.size());
@@ -158,12 +158,12 @@ class TestSarifReport : public TestFixture
158158

159159
const picojson::object& root = json.get<picojson::object>();
160160
const picojson::array& runs = root.at("runs").get<picojson::array>();
161-
const picojson::object& run = runs[0].get<picojson::object>();
161+
const picojson::object& cur_run = runs[0].get<picojson::object>();
162162

163-
const picojson::array& results = run.at("results").get<picojson::array>();
163+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
164164
ASSERT_EQUALS(3U, results.size());
165165

166-
const picojson::object& tool = run.at("tool").get<picojson::object>();
166+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
167167
const picojson::object& driver = tool.at("driver").get<picojson::object>();
168168
const picojson::array& rules = driver.at("rules").get<picojson::array>();
169169
ASSERT_EQUALS(3U, rules.size());
@@ -184,13 +184,13 @@ class TestSarifReport : public TestFixture
184184

185185
const picojson::object& root = json.get<picojson::object>();
186186
const picojson::array& runs = root.at("runs").get<picojson::array>();
187-
const picojson::object& run = runs[0].get<picojson::object>();
187+
const picojson::object& cur_run = runs[0].get<picojson::object>();
188188

189189
// Should have no results (GitHub doesn't support findings without locations)
190-
const picojson::array& results = run.at("results").get<picojson::array>();
190+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
191191
ASSERT_EQUALS(0U, results.size());
192192

193-
const picojson::object& tool = run.at("tool").get<picojson::object>();
193+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
194194
const picojson::object& driver = tool.at("driver").get<picojson::object>();
195195
const picojson::array& rules = driver.at("rules").get<picojson::array>();
196196
ASSERT_EQUALS(0U, rules.size());
@@ -218,8 +218,8 @@ class TestSarifReport : public TestFixture
218218

219219
const picojson::object& root = json.get<picojson::object>();
220220
const picojson::array& runs = root.at("runs").get<picojson::array>();
221-
const picojson::object& run = runs[0].get<picojson::object>();
222-
const picojson::array& results = run.at("results").get<picojson::array>();
221+
const picojson::object& cur_run = runs[0].get<picojson::object>();
222+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
223223

224224
ASSERT_EQUALS(1U, results.size());
225225

@@ -252,9 +252,9 @@ class TestSarifReport : public TestFixture
252252

253253
const picojson::object& root = json.get<picojson::object>();
254254
const picojson::array& runs = root.at("runs").get<picojson::array>();
255-
const picojson::object& run = runs[0].get<picojson::object>();
255+
const picojson::object& cur_run = runs[0].get<picojson::object>();
256256

257-
const picojson::array& results = run.at("results").get<picojson::array>();
257+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
258258
ASSERT_EQUALS(6U, results.size());
259259

260260
// Check severity mappings
@@ -282,8 +282,8 @@ class TestSarifReport : public TestFixture
282282

283283
const picojson::object& root = json.get<picojson::object>();
284284
const picojson::array& runs = root.at("runs").get<picojson::array>();
285-
const picojson::object& run = runs[0].get<picojson::object>();
286-
const picojson::object& tool = run.at("tool").get<picojson::object>();
285+
const picojson::object& cur_run = runs[0].get<picojson::object>();
286+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
287287
const picojson::object& driver = tool.at("driver").get<picojson::object>();
288288
const picojson::array& rules = driver.at("rules").get<picojson::array>();
289289

@@ -327,8 +327,8 @@ class TestSarifReport : public TestFixture
327327

328328
const picojson::object& root = json.get<picojson::object>();
329329
const picojson::array& runs = root.at("runs").get<picojson::array>();
330-
const picojson::object& run = runs[0].get<picojson::object>();
331-
const picojson::object& tool = run.at("tool").get<picojson::object>();
330+
const picojson::object& cur_run = runs[0].get<picojson::object>();
331+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
332332
const picojson::object& driver = tool.at("driver").get<picojson::object>();
333333
const picojson::array& rules = driver.at("rules").get<picojson::array>();
334334

@@ -361,8 +361,8 @@ class TestSarifReport : public TestFixture
361361

362362
const picojson::object& root = json.get<picojson::object>();
363363
const picojson::array& runs = root.at("runs").get<picojson::array>();
364-
const picojson::object& run = runs[0].get<picojson::object>();
365-
const picojson::object& tool = run.at("tool").get<picojson::object>();
364+
const picojson::object& cur_run = runs[0].get<picojson::object>();
365+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
366366
const picojson::object& driver = tool.at("driver").get<picojson::object>();
367367
const picojson::array& rules = driver.at("rules").get<picojson::array>();
368368

@@ -392,16 +392,16 @@ class TestSarifReport : public TestFixture
392392
report.addFinding(
393393
createErrorMessage("test1", Severity::error, "Conclusive", "test.cpp", 10, 5, 0, Certainty::normal));
394394
report.addFinding(createErrorMessage(
395-
"test2", Severity::error, "Inconclusive", "test.cpp", 20, 5, 0, Certainty::inconclusive));
395+
"test2", Severity::error, "Inconclusive", "test.cpp", 20, 5, 0, Certainty::inconclusive));
396396

397397
std::string sarif = report.serialize("Cppcheck");
398398
picojson::value json;
399399
ASSERT(parseAndValidateJson(sarif, json));
400400

401401
const picojson::object& root = json.get<picojson::object>();
402402
const picojson::array& runs = root.at("runs").get<picojson::array>();
403-
const picojson::object& run = runs[0].get<picojson::object>();
404-
const picojson::object& tool = run.at("tool").get<picojson::object>();
403+
const picojson::object& cur_run = runs[0].get<picojson::object>();
404+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
405405
const picojson::object& driver = tool.at("driver").get<picojson::object>();
406406
const picojson::array& rules = driver.at("rules").get<picojson::array>();
407407

@@ -430,8 +430,8 @@ class TestSarifReport : public TestFixture
430430

431431
const picojson::object& root = json.get<picojson::object>();
432432
const picojson::array& runs = root.at("runs").get<picojson::array>();
433-
const picojson::object& run = runs[0].get<picojson::object>();
434-
const picojson::array& results = run.at("results").get<picojson::array>();
433+
const picojson::object& cur_run = runs[0].get<picojson::object>();
434+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
435435

436436
ASSERT_EQUALS(1U, results.size());
437437

@@ -452,8 +452,8 @@ class TestSarifReport : public TestFixture
452452

453453
const picojson::object& root = json.get<picojson::object>();
454454
const picojson::array& runs = root.at("runs").get<picojson::array>();
455-
const picojson::object& run = runs[0].get<picojson::object>();
456-
const picojson::object& tool = run.at("tool").get<picojson::object>();
455+
const picojson::object& cur_run = runs[0].get<picojson::object>();
456+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
457457
const picojson::object& driver = tool.at("driver").get<picojson::object>();
458458
const picojson::array& rules = driver.at("rules").get<picojson::array>();
459459

@@ -489,8 +489,8 @@ class TestSarifReport : public TestFixture
489489

490490
const picojson::object& root = json.get<picojson::object>();
491491
const picojson::array& runs = root.at("runs").get<picojson::array>();
492-
const picojson::object& run = runs[0].get<picojson::object>();
493-
const picojson::array& results = run.at("results").get<picojson::array>();
492+
const picojson::object& cur_run = runs[0].get<picojson::object>();
493+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
494494

495495
ASSERT_EQUALS(2U, results.size());
496496

@@ -541,13 +541,13 @@ class TestSarifReport : public TestFixture
541541

542542
const picojson::object& root = json.get<picojson::object>();
543543
const picojson::array& runs = root.at("runs").get<picojson::array>();
544-
const picojson::object& run = runs[0].get<picojson::object>();
544+
const picojson::object& cur_run = runs[0].get<picojson::object>();
545545

546546
// Should have 3 results but only 1 rule
547-
const picojson::array& results = run.at("results").get<picojson::array>();
547+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
548548
ASSERT_EQUALS(3U, results.size());
549549

550-
const picojson::object& tool = run.at("tool").get<picojson::object>();
550+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
551551
const picojson::object& driver = tool.at("driver").get<picojson::object>();
552552
const picojson::array& rules = driver.at("rules").get<picojson::array>();
553553
ASSERT_EQUALS(1U, rules.size());
@@ -568,8 +568,8 @@ class TestSarifReport : public TestFixture
568568

569569
const picojson::object& root = json.get<picojson::object>();
570570
const picojson::array& runs = root.at("runs").get<picojson::array>();
571-
const picojson::object& run = runs[0].get<picojson::object>();
572-
const picojson::object& tool = run.at("tool").get<picojson::object>();
571+
const picojson::object& cur_run = runs[0].get<picojson::object>();
572+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
573573
const picojson::object& driver = tool.at("driver").get<picojson::object>();
574574

575575
// Should use "Cppcheck" as default when custom name doesn't parse
@@ -587,8 +587,8 @@ class TestSarifReport : public TestFixture
587587

588588
const picojson::object& root = json.get<picojson::object>();
589589
const picojson::array& runs = root.at("runs").get<picojson::array>();
590-
const picojson::object& run = runs[0].get<picojson::object>();
591-
const picojson::object& tool = run.at("tool").get<picojson::object>();
590+
const picojson::object& cur_run = runs[0].get<picojson::object>();
591+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
592592
const picojson::object& driver = tool.at("driver").get<picojson::object>();
593593

594594
// Should have a semantic version
@@ -627,8 +627,8 @@ class TestSarifReport : public TestFixture
627627

628628
const picojson::object& root = json.get<picojson::object>();
629629
const picojson::array& runs = root.at("runs").get<picojson::array>();
630-
const picojson::object& run = runs[0].get<picojson::object>();
631-
const picojson::object& tool = run.at("tool").get<picojson::object>();
630+
const picojson::object& cur_run = runs[0].get<picojson::object>();
631+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
632632
const picojson::object& driver = tool.at("driver").get<picojson::object>();
633633
const picojson::array& rules = driver.at("rules").get<picojson::array>();
634634

@@ -678,8 +678,8 @@ class TestSarifReport : public TestFixture
678678

679679
const picojson::object& root = json.get<picojson::object>();
680680
const picojson::array& runs = root.at("runs").get<picojson::array>();
681-
const picojson::object& run = runs[0].get<picojson::object>();
682-
const picojson::object& tool = run.at("tool").get<picojson::object>();
681+
const picojson::object& cur_run = runs[0].get<picojson::object>();
682+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
683683
const picojson::object& driver = tool.at("driver").get<picojson::object>();
684684

685685
const std::string& version = driver.at("semanticVersion").get<std::string>();
@@ -700,8 +700,8 @@ class TestSarifReport : public TestFixture
700700

701701
const picojson::object& root = json.get<picojson::object>();
702702
const picojson::array& runs = root.at("runs").get<picojson::array>();
703-
const picojson::object& run = runs[0].get<picojson::object>();
704-
const picojson::object& tool = run.at("tool").get<picojson::object>();
703+
const picojson::object& cur_run = runs[0].get<picojson::object>();
704+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
705705
const picojson::object& driver = tool.at("driver").get<picojson::object>();
706706

707707
// Should have a name (either parsed or default)
@@ -730,8 +730,8 @@ class TestSarifReport : public TestFixture
730730

731731
const picojson::object& root = json.get<picojson::object>();
732732
const picojson::array& runs = root.at("runs").get<picojson::array>();
733-
const picojson::object& run = runs[0].get<picojson::object>();
734-
const picojson::array& results = run.at("results").get<picojson::array>();
733+
const picojson::object& cur_run = runs[0].get<picojson::object>();
734+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
735735

736736
ASSERT_EQUALS(2U, results.size());
737737

@@ -789,8 +789,8 @@ class TestSarifReport : public TestFixture
789789

790790
const picojson::object& root = json.get<picojson::object>();
791791
const picojson::array& runs = root.at("runs").get<picojson::array>();
792-
const picojson::object& run = runs[0].get<picojson::object>();
793-
const picojson::array& results = run.at("results").get<picojson::array>();
792+
const picojson::object& cur_run = runs[0].get<picojson::object>();
793+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
794794

795795
ASSERT_EQUALS(3U, results.size());
796796

@@ -824,8 +824,8 @@ class TestSarifReport : public TestFixture
824824

825825
const picojson::object& root = json.get<picojson::object>();
826826
const picojson::array& runs = root.at("runs").get<picojson::array>();
827-
const picojson::object& run = runs[0].get<picojson::object>();
828-
const picojson::object& tool = run.at("tool").get<picojson::object>();
827+
const picojson::object& cur_run = runs[0].get<picojson::object>();
828+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
829829
const picojson::object& driver = tool.at("driver").get<picojson::object>();
830830
const picojson::array& rules = driver.at("rules").get<picojson::array>();
831831

@@ -869,14 +869,14 @@ class TestSarifReport : public TestFixture
869869

870870
const picojson::object& root = json.get<picojson::object>();
871871
const picojson::array& runs = root.at("runs").get<picojson::array>();
872-
const picojson::object& run = runs[0].get<picojson::object>();
872+
const picojson::object& cur_run = runs[0].get<picojson::object>();
873873

874874
// Should only have results for findings with locations
875-
const picojson::array& results = run.at("results").get<picojson::array>();
875+
const picojson::array& results = cur_run.at("results").get<picojson::array>();
876876
ASSERT_EQUALS(3U, results.size());
877877

878878
// Should only have rules for findings with locations
879-
const picojson::object& tool = run.at("tool").get<picojson::object>();
879+
const picojson::object& tool = cur_run.at("tool").get<picojson::object>();
880880
const picojson::object& driver = tool.at("driver").get<picojson::object>();
881881
const picojson::array& rules = driver.at("rules").get<picojson::array>();
882882
ASSERT_EQUALS(3U, rules.size());

0 commit comments

Comments
 (0)