search

ch-robinson / dotnet-avro

An Avro implementation for .NET
https://engineering.chrobinson.com/dotnet-avro/
MIT License
135 stars 51 forks source link

Fix broken serde build checks in `registry-test` verb #223

Closed dstelljes closed 2 years ago

dstelljes commented 2 years ago

registry-test relies on reflective method invocations that are no longer correct after signature changes in 8.0.0:

Unhandled exception. System.Reflection.TargetParameterCountException: Parameter count mismatch.
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at System.Reflection.MethodBase.Invoke(Object obj, Object[] parameters)
   at Chr.Avro.Cli.TestSchemaVerb.Run() in /_/src/Chr.Avro.Cli/Cli/TestSchemaVerb.cs:line 62
   at Chr.Avro.Cli.Verb.Execute() in /_/src/Chr.Avro.Cli/Cli/Verb.cs:line 12   at Chr.Avro.Cli.Program.<Main>(String[] args)

This change switches to the BuildExpression methods which produce the same results.