You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The restful action layer is covered by integration tests, so they are excluded in the UT. The transport action layer are still missing some coverage. I am adding more unit tests for the following classes in transport layer for AD.
Also, these 2 task runners also needs more tests. We can address them in the next.
// #241
'org.opensearch.ad.task.ADBatchTaskRunner',
'org.opensearch.ad.task.ADTaskManager',
break down this issue into several sub-issues in 492 and 493. The classes that will be removed out of the jacocoExclusions are identified in the above two issues. The rest layer classes are tested in integration tests so we will keep them in the jacocoExclusions list.
Describe the bug
The
jacocoExclusions
list is quite long, https://github.com/opensearch-project/anomaly-detection/blob/main/build.gradle#L478. Maybe we're just ignoring coverage thresholds, or they are too high? In either case those thresholds are likely not useful.Either fix the TODOs and reduce the list, or remove/reduce the coverage thresholds.
The text was updated successfully, but these errors were encountered: