[Fix] Guid? == null comparison

[jogibear9988]

Can you create a new nuget?

[StefH]

Unit tests fail. Can you take a look at your code?

[jogibear9988]

I thought, maybe we should work on

int FindBestMethod(IEnumerable methods, Expression[] args, out MethodBase method)

the different created arrays seem not verry perfomant to me

[codecov]

Codecov Report

Merging #85 into master will increase coverage by 0.09%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #85      +/-   ##
==========================================
+ Coverage   83.97%   84.06%   +0.09%     
==========================================
  Files          25       25              
  Lines        3082     3087       +5     
  Branches      448      449       +1     
==========================================
+ Hits         2588     2595       +7     
+ Misses        365      364       -1     
+ Partials      129      128       -1

Impacted Files	Coverage Δ
src/System.Linq.Dynamic.Core/ExpressionParser.cs	81.14% <100%> (+0.19%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1dd9f0a...8a72624. Read the comment docs.

[jogibear9988]

fixed...

[jogibear9988]

would be nice if it could be merged, and you could create a new nuget

[jogibear9988]

can you create a new nuget?

[StefH]

I did merge, and added some extra tests.

However it some tests fails? Not related to guid I think, maybe I missed something...
See:
https://github.com/StefH/System.Linq.Dynamic.Core/blob/master/test/System.Linq.Dynamic.Core.Tests/ExpressionTests.cs#L583

[jogibear9988]

I fixed your test, you need to use

  new object[] {null}

but there was also a error then in the lib, because the expression is not the same as the cached null expression!