Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Ignoring flaky test #255

Merged
merged 2 commits into from
Oct 15, 2020
Merged

Ignoring flaky test #255

merged 2 commits into from
Oct 15, 2020

Conversation

saratvemulapalli
Copy link
Contributor

@saratvemulapalli saratvemulapalli commented Oct 14, 2020

*Issue #254 *
Ignoring this test for now as it is flaky in github runs.
Will take care of it as part of the backlog.
Created a tracking issue #254

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@codecov
Copy link

codecov bot commented Oct 14, 2020

Codecov Report

Merging #255 into master will increase coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #255      +/-   ##
============================================
+ Coverage     73.01%   73.04%   +0.02%     
- Complexity     1461     1462       +1     
============================================
  Files           164      164              
  Lines          6834     6834              
  Branches        527      527              
============================================
+ Hits           4990     4992       +2     
+ Misses         1594     1592       -2     
  Partials        250      250              
Flag Coverage Δ Complexity Δ
#cli 79.27% <ø> (ø) 0.00 <ø> (ø)
#plugin 72.31% <ø> (+0.03%) 1462.00 <ø> (+1.00)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...ransport/SearchAnomalyDetectorTransportAction.java 77.77% <0.00%> (-22.23%) 2.00% <0.00%> (ø%)
...ransport/DeleteAnomalyDetectorTransportAction.java 59.03% <0.00%> (+4.81%) 16.00% <0.00%> (+1.00%)

@saratvemulapalli saratvemulapalli merged commit ba7ee59 into master Oct 15, 2020
@saratvemulapalli saratvemulapalli deleted the flaky-test branch October 15, 2020 17:33
@ohltyler ohltyler added the infra label Oct 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants