Increase test verbosity

[overhacked]

Description

Change config["verbose"] from 1 to 2 so that DEBUG-level logging from plugin import tasks and event hooks are emitted. (See beets.plugins._set_log_level_and_params.)

config["verbose"] == 2 does not appear to affect logging anywhere else in the codebase, while config["verbose"] == 3 would enable output of messages logged via BeetsLogger.extra_debug().

To Do

• Documentation
• Changelog
• Tests

[github-actions]

Thank you for the PR! The changelog has not been updated, so here is a friendly reminder to check if you need to add an entry.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.97%. Comparing base (acc1235) to head (9cc1c04).
⚠️ Report is 3 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6828   +/-   ##
=======================================
  Coverage   74.97%   74.97%           
=======================================
  Files         163      163           
  Lines       21043    21043           
  Branches     3312     3312           
=======================================
  Hits        15777    15777           
  Misses       4514     4514           
  Partials      752      752           

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[snejus]

extra_debug is currently primarily used in lastgenre plugin for development - is there any value in enabling this logging level here?

[overhacked]

extra_debug is currently primarily used in lastgenre plugin for development - is there any value in enabling this logging level here?

I was wondering that myself, if maximum debug logging is the right way to go for tests, and I figured you would have a better sense of that. IMO, if extra_debug() is intended to be a restriction on user-facing console log spam and not "trace-level" logging then we should enable it for tests. If you agree, I'll re-push with verbose=3, or go ahead and amend and merge if you like! Thx!

[snejus]

Thinking about the future, this is actually a good idea - verbosity=2 is what we'd like to have indeed!

[semohr]

extra_debug is currently primarily used in lastgenre plugin for development - is there any value in enabling this logging level here?

I was wondering that myself, if maximum debug logging is the right way to go for tests, and I figured you would have a better sense of that. IMO, if extra_debug() is intended to be a restriction on user-facing console log spam and not "trace-level" logging then we should enable it for tests. If you agree, I'll re-push with verbose=3, or go ahead and amend and merge if you like! Thx!

Im working towards removing this extra debug level at some point 🤣 Feel free to have a look at #6553 and #6773