[tor-commits] [tor/master] practracker: Test for bad ".c" includes in the unit tests

nickm at torproject.org nickm at torproject.org
Wed Dec 4 17:15:37 UTC 2019


commit 19324ff0bae96059155a5b16301528d682b78078
Author: teor <teor at torproject.org>
Date:   Sat Nov 16 21:45:52 2019 +1000

    practracker: Test for bad ".c" includes in the unit tests
    
    Part of 32522.
---
 scripts/maint/practracker/testdata/.may_include               | 1 +
 scripts/maint/practracker/testdata/a.c                        | 3 +++
 scripts/maint/practracker/testdata/ex0-expected.txt           | 6 +++---
 scripts/maint/practracker/testdata/ex1-overbroad-expected.txt | 2 +-
 scripts/maint/practracker/testdata/ex1.txt                    | 6 +++---
 5 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/scripts/maint/practracker/testdata/.may_include b/scripts/maint/practracker/testdata/.may_include
index 40bf8155d..8542a3580 100644
--- a/scripts/maint/practracker/testdata/.may_include
+++ b/scripts/maint/practracker/testdata/.may_include
@@ -1,3 +1,4 @@
 !advisory
 
 permitted.h
+ext/good.c
diff --git a/scripts/maint/practracker/testdata/a.c b/scripts/maint/practracker/testdata/a.c
index 1939773f5..3c338ab40 100644
--- a/scripts/maint/practracker/testdata/a.c
+++ b/scripts/maint/practracker/testdata/a.c
@@ -5,6 +5,9 @@
 
 # include "permitted.h"
 
+#include "ext/good.c"
+#include "bad.c"
+
 int
 i_am_a_function(void)
 {
diff --git a/scripts/maint/practracker/testdata/ex0-expected.txt b/scripts/maint/practracker/testdata/ex0-expected.txt
index 5f3d9e5ae..502005edb 100644
--- a/scripts/maint/practracker/testdata/ex0-expected.txt
+++ b/scripts/maint/practracker/testdata/ex0-expected.txt
@@ -1,8 +1,8 @@
-problem file-size a.c 38
-problem include-count a.c 4
+problem file-size a.c 41
+problem include-count a.c 6
 problem function-size a.c:i_am_a_function() 9
 problem function-size a.c:another_function() 12
-problem dependency-violation a.c 3
+problem dependency-violation a.c 4
 problem file-size b.c 15
 problem function-size b.c:foo() 4
 problem function-size b.c:bar() 5
diff --git a/scripts/maint/practracker/testdata/ex1-overbroad-expected.txt b/scripts/maint/practracker/testdata/ex1-overbroad-expected.txt
index f69c608f4..b757202da 100644
--- a/scripts/maint/practracker/testdata/ex1-overbroad-expected.txt
+++ b/scripts/maint/practracker/testdata/ex1-overbroad-expected.txt
@@ -1,2 +1,2 @@
-problem file-size a.c 40 -> 38
+problem file-size a.c 45 -> 41
 problem file-size z.c 100 -> 0
diff --git a/scripts/maint/practracker/testdata/ex1.txt b/scripts/maint/practracker/testdata/ex1.txt
index c698005d0..af8de0329 100644
--- a/scripts/maint/practracker/testdata/ex1.txt
+++ b/scripts/maint/practracker/testdata/ex1.txt
@@ -1,6 +1,6 @@
 
-problem file-size a.c 40
-problem include-count a.c 4
+problem file-size a.c 45
+problem include-count a.c 6
 # this problem will produce an error
 problem function-size a.c:i_am_a_function() 8
 # this problem will produce a warning
@@ -12,7 +12,7 @@ problem file-size b.c 15
 problem file-size z.c 100
 
 problem function-size b.c:bar() 5
-problem dependency-violation a.c 3
+problem dependency-violation a.c 4
 problem dependency-violation header.h 3
 problem file-size header.h 8
 problem include-count header.h 4





More information about the tor-commits mailing list