yitsushi / totp-cli / 8455455856

29%

master: 20%

Build Type

push

github

Committed by yitsushi

Commit Message

refactor: cleanup, mostly tests

In preparation for more work, related to #112, #37 and #8.

- Migrated all tests to use testify's `Suite` instead of direct `assert`
  calls.
- Simplified how the credentials file location is determined, now it's
  defined in one simple function.

Changes I may revert: I hate string concatenation, it's just ugly in my
eyes, so I may just add that linter to disabled and revert to `Sprintf`. I
know it's more efficient, but not on this scale.

Closes #8

References:
- https://github.com/yitsushi/totp-cli/issues/8
- https://github.com/yitsushi/totp-cli/issues/37
- https://github.com/yitsushi/totp-cli/issues/112

Run Details

15 of 53 new or added lines in 8 files covered. (28.3%)

5 existing lines in 3 files now uncovered.

299 of 1030 relevant lines covered (29.03%)

2.11 hits per line