search

FWDekker / intellij-randomness

IntelliJ plugin for inserting random numbers, UUIDs, names, IP addresses, and much more
https://plugins.jetbrains.com/plugin/9836-randomness
MIT License
45 stars 7 forks source link

New error reporter #523

Closed FWDekker closed 8 months ago

FWDekker commented 8 months ago

Fixes #455.

The new error reporter is inspired by Patrick Scheibe's error reporter, though significant changes have been made:

  1. Rewritten in Kotlin
  2. Completely different class structure
  3. Fetches up-to-date token from repo
  4. Includes attachments
  5. Allows multiple stacktraces
  6. Ensures synchronization when multiple errors are reported simultaneously

Test coverage is, unfortunately, extremely poor because this is hard to test, and to be honest I don't really feel like adding entire mock structures everywhere to test code that is mostly quite obvious from reading it.

codecov[bot] commented 8 months ago

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (2274a0e) 85.50% compared to head (d206663) 85.50%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/FWDekker/intellij-randomness/pull/523/graphs/tree.svg?width=650&height=150&src=pr&token=HXVFGMwtwv&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Florine+W.+Dekker)](https://app.codecov.io/gh/FWDekker/intellij-randomness/pull/523?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Florine+W.+Dekker) ```diff @@ Coverage Diff @@ ## main #523 +/- ## ======================================= Coverage 85.50% 85.50% ======================================= Files 51 51 Lines 2035 2035 Branches 343 343 ======================================= Hits 1740 1740 Misses 224 224 Partials 71 71 ```