ooni / probe-cli / 6147920723

72%

Build Type

push

github

Committed by web-flow

Commit Message

refactor(measurexlite): move OperationLogger to logx (#1246)

## 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/2531
- [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: N/A

## Description

I am starting to look into how to support beacons into the codebase as
explained by https://github.com/ooni/probe/issues/2531.

The first issue that I stumbled upon is that there is excessive coupling
between packages at the lower-levels, which we can reduce with easy
refactoring, and which would help reasoning on what to do to address the
underlying issue.

This diff is the first minor step towards cleaning up the code a bit: we
are moving a functionality that pertains to logging from measurexlite to
logx (which is where it should be).

While there, I noticed I needed to run `go mod tidy` to update the
`go.mod` and `go.sum` files, which were outdated.

Run Details

23 of 25 new or added lines in 22 files covered. (92.0%)

8 existing lines in 3 files now uncovered.

22329 of 27563 relevant lines covered (81.01%)

30.58 hits per line