9128527407
83%
master: 72%

Ran 17 May 2024 12:58PM UTC

Jobs 1

Files 555

Run time 1min

Badge

Embed ▾

Committed 17 May 2024 12:52PM UTC coverage: 83.204% (+0.02%) from 83.186%

Build # 9128527407

Build Type

push

github

Committed by

bassosimone

Commit Message

fix(cli/rm/rm.go): using panic is not user friendly (#1608)

I manually tested the cli and concluded that it's too easy to reach the
panic caused by runtimex.Try0 over there and it would expose panics to
users too frequently.

Let's just convert those into normal error checks. Here is what we would
previously get:

```
% ./ooniprobe rm
? Are you sure you wish to delete the result #0  [Use arrows to move, type to filter]
  true
> false
panic: Try0: interrupt

[...]
```

So, here I just pressed ^D and got a panic. Ouch!

After the diff, we have this:

```
% ./ooniprobe rm
? Are you sure you wish to delete the result #0 false
   ⨯ failure in main command   error=canceled by user
```

and:

```
% ./ooniprobe rm --all
? Are you sure you wish to delete ALL results false
   ⨯ failure in main command   error=canceled by user
```

It seems much better, isn't it?

Part of the QA for https://github.com/ooni/probe/issues/2722

Run Details

26959 of 32401 relevant lines covered (83.2%)

53.03 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	0 - 9128527407.1	17 May 2024 12:58PM UTC	0	83.2	GitHub Action Run

Source Files on build 9128527407

Detailed source file information is not available for this build.

ooni / probe-cli / 9128527407
83%
master: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9128527407

ooni / probe-cli / 9128527407 83% master: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 9128527407

ooni / probe-cli / 9128527407
83%
master: 72%

README BADGES
x