Ignore code coverage for method executed non-deterministically in tests

[msridhar]

Fixes #828 (hopefully)

There is some non-determinism in whether com.uber.nullaway.dataflow.AccessPath.IteratorContentsKey#equals runs or not during tests (see the code comment). The non-determinism will be hard for us to fix, so instead, add an annotation that tells JaCoCo to ignore coverage of this method (which is low risk as it's an equals() method that I don't expect will change).

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (b64d1c1) 86.76% compared to head (a89927d) 86.78%.

Additional details and impacted files

@@             Coverage Diff              @@
##             master     #838      +/-   ##
============================================
+ Coverage     86.76%   86.78%   +0.01%     
  Complexity     1877     1877              
============================================
  Files            74       74              
  Lines          6197     6191       -6     
  Branches       1206     1204       -2     
============================================
- Hits           5377     5373       -4     
+ Misses          408      407       -1     
+ Partials        412      411       -1     

Files	Coverage Δ
...in/java/com/uber/nullaway/dataflow/AccessPath.java	90.76% <ø> (+0.76%)	⬆️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lazaroclapp]

👍