From 9a9a66e71557a5b19c96677eb26f97067c0a06d6 Mon Sep 17 00:00:00 2001 From: Jon Skeet Date: Wed, 25 Oct 2017 08:03:50 +0100 Subject: Run C# codegen when testing it This will cause failures if shared protos change in an incompatible way. --- tests.sh | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'tests.sh') diff --git a/tests.sh b/tests.sh index 6733e835..50c45fa9 100755 --- a/tests.sh +++ b/tests.sh @@ -87,9 +87,7 @@ build_cpp_distcheck() { } build_csharp() { - # Just for the conformance tests. We don't currently - # need to really build protoc, but it's simplest to keep with the - # conventions of the other builds. + # Required for conformance tests and to regenerate protos. internal_build_cpp NUGET=/usr/local/bin/nuget.exe @@ -104,6 +102,10 @@ build_csharp() { (cd dotnettmp; dotnet new > /dev/null) rm -rf dotnettmp + # Check that the protos haven't broken C# codegen. + # TODO(jonskeet): Fail if regenerating creates any changes. + csharp/generate_protos.sh + csharp/buildall.sh cd conformance && make test_csharp && cd .. -- cgit v1.2.3