Visual tests - correct code to pass and remove xfails

[CyclingNinja]

No description provided.

[astrofrog]

pre-commit.ci autofix

[astrofrog]

This looks good to me, though it looks like there might be a genuine failure in the line style test. Once this is fixed, the hashes for the figures should be updated for the two tests which will allow the CI to pass.

[CyclingNinja]

pre-commit.ci autofix

[astrofrog]

Looking good, I think you just need to update the hashes now!

[codecov]

Codecov Report

Attention: Patch coverage is 18.46154% with 53 lines in your changes missing coverage. Please review.

Project coverage is 84.41%. Comparing base (f997e7d) to head (cdcd961).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
glue_jupyter/bqplot/scatter/tests/test_visual.py	0.00%	29 Missing ⚠️
glue_jupyter/bqplot/scatter/layer_artist.py	29.41%	24 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #479      +/-   ##
==========================================
- Coverage   85.07%   84.41%   -0.67%     
==========================================
  Files          91       91              
  Lines        5407     5447      +40     
==========================================
- Hits         4600     4598       -2     
- Misses        807      849      +42     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[dhomeier]

Seems test_remove is failing now (due to changed number of tests?)
And the two failing image comparisons look like it’s just a change in colourmaps, so just update the hashes again?