test: Expand test coverage to background.ts and refactor sendToBackground method

[tora-pan]

Split original branch into to separate branches in order to be able to merge logic and test changes separately.

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (main@d7c4c8f). Click here to learn what that means.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1449   +/-   ##
=======================================
  Coverage        ?   80.16%           
=======================================
  Files           ?        7           
  Lines           ?     2999           
  Branches        ?      186           
=======================================
  Hits            ?     2404           
  Misses          ?      590           
  Partials        ?        5           

Impacted Files	Coverage Δ
extension/rikaichan.ts	87.70% <ø> (ø)
extension/background.ts	97.40% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[melink14]

Looks like it still contains the changes from the other PR. :(

Also, I'm not sure why the title was changed as it was since you already have a PR which is about fixing the bug!

As I said in the chat, you can try again or let me know if you want to work through it together!

[tora-pan]

@melink14 Do I need to add these tests changes to their own branch? Will that clean things up?

[melink14]

@melink14 Do I need to add these tests changes to their own branch? Will that clean things up?

Yes, in this one I would just make a new branch off of main and then copy just the test file into it. (we can also wait til we land the first change if you want to reuse the helper methods you made)