ooni / probe-cli / 6472224044 / 1
80%
master: 72%

LAST BUILD BRANCH: refactor/no-forks
DEFAULT BRANCH: master
Ran
Files 515
Run time 11s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 79.61% (+0.01%) from 79.598%
6472224044.1

push

github

web-flow 
fix: anomaly with android_dns_cache_no_data and inconsistent dns (#1211)

## Checklist

- [x] I have read the [contribution
guidelines](https://github.com/ooni/probe-cli/blob/master/CONTRIBUTING.md)
- [x] reference issue for this pull request:
https://github.com/ooni/probe/issues/2499
- [x] if you changed anything related to how experiments work and you
need to reflect these changes in the ooni/spec repository, please link
to the related ooni/spec pull request: N/A
- [x] if you changed code inside an experiment, make sure you bump its
version number

## Summary

This diff changes the `summary.go` algorithm of Web Connectivity v0.4 to
handle `android_dns_cache_no_data` as an anomaly when the DNS is
inconsistent. We continue handling `dns_nxdomain_error` as an anomaly
when the DNS is inconsistent, as demonstrated by the fact that the
corresponding netem test is still passing.

This diff also bumps the version number to v0.4.3. Version v0.4.2 did
not handle this case, which caused measurements to be marked as failed
as documented by https://github.com/ooni/probe/issues/2499.

This diff is also related to https://github.com/ooni/probe/issues/2029,
in the sense that it is slightly improving our analysis results when the
is an NXDOMAIN error (even if it's masked by Android's DNS cache
behavior).

While there, add empty lines to improve the code readability.

23629 of 29681 relevant lines covered (79.61%)

39.03 hits per line

Source Files on job 0 - 6472224044.1