fix sanity tests

pull/23904/head
Jan Tattermusch 5 years ago
parent 80252ec4a6
commit e36ee8ef19
  1. 3
      tools/distrib/check_copyright.py
  2. 3
      tools/distrib/check_include_guards.py
  3. 2
      tools/dockerfile/grpc_clang_format/clang_format_all_the_things.sh
  4. 4
      tools/run_tests/sanity/check_port_platform.py

@ -151,7 +151,8 @@ for filename in filename_list:
if filename in _EXEMPT:
continue
# Skip check for upb generated code.
if filename.endswith('.upb.h') or filename.endswith('.upb.c'):
if (filename.endswith('.upb.h') or filename.endswith('.upb.c') or
filename.endswith('.upbdefs.h') or filename.endswith('.upbdefs.c')):
continue
ext = os.path.splitext(filename)[1]
base = os.path.basename(filename)

@ -187,7 +187,8 @@ validator = GuardValidator()
for filename in filename_list:
# Skip check for upb generated code.
if filename.endswith('.upb.h') or filename.endswith('.upb.c'):
if (filename.endswith('.upb.h') or filename.endswith('.upb.c') or
filename.endswith('.upbdefs.h') or filename.endswith('.upbdefs.c')):
continue
ok = ok and validator.check(filename, args.fix)

@ -29,7 +29,7 @@ for dir in $DIRS
do
for glob in $GLOB
do
files="$files `find ${CLANG_FORMAT_ROOT}/$dir -name $glob -and -not -name '*.generated.*' -and -not -name '*.upb.h' -and -not -name '*.upb.c' -and -not -name '*.pb.h' -and -not -name '*.pb.c' -and -not -name '*.pb.cc' -and -not -name '*.pbobjc.h' -and -not -name '*.pbobjc.m' -and -not -name '*.pbrpc.h' -and -not -name '*.pbrpc.m' -and -not -name end2end_tests.cc -and -not -name end2end_nosec_tests.cc -and -not -name public_headers_must_be_c89.c -and -not -name grpc_shadow_boringssl.h`"
files="$files `find ${CLANG_FORMAT_ROOT}/$dir -name $glob -and -not -name '*.generated.*' -and -not -name '*.upb.h' -and -not -name '*.upb.c' -and -not -name '*.upbdefs.h' -and -not -name '*.upbdefs.c' -and -not -name '*.pb.h' -and -not -name '*.pb.c' -and -not -name '*.pb.cc' -and -not -name '*.pbobjc.h' -and -not -name '*.pbobjc.m' -and -not -name '*.pbrpc.h' -and -not -name '*.pbrpc.m' -and -not -name end2end_tests.cc -and -not -name end2end_nosec_tests.cc -and -not -name public_headers_must_be_c89.c -and -not -name grpc_shadow_boringssl.h`"
done
done

@ -36,7 +36,9 @@ def check_port_platform_inclusion(directory_root):
if filename.endswith('.pb.h') or filename.endswith('.pb.c'):
continue
# Skip check for upb generated code.
if filename.endswith('.upb.h') or filename.endswith('.upb.c'):
if (filename.endswith('.upb.h') or filename.endswith('.upb.c') or
filename.endswith('.upbdefs.h') or
filename.endswith('.upbdefs.c')):
continue
with open(path) as f:
all_lines_in_file = f.readlines()

Loading…
Cancel
Save