[GSOC] fix test_convergence_plot_command_line

[Aaryan-Dadu]

📝 Description

Type: 🪲 bugfix

fixes #3417

• previously for the environment only is_notebook was set to False, but under the VS Code test interface allows_widget_display still evaluated to True.
• mocking allows_widget_display to False to correct this bug, now the test passes on both CLI and vscode test interface

📌 Resources

🚦 Testing

How did you test these changes?

• Testing pipeline
• Other method (describe)
• My changes can't be tested (explain why)

☑️ Checklist

• I requested two reviewers for this pull request
• I updated the documentation according to my changes
• I built the documentation by applying the build_docs label

Note: If you are not allowed to perform any of these actions, ping (@) a contributor.

AI Usage

None

[andrewfullard]

Nice fix, works for me

[tardis-bot]

*beep* *bop*
Hi human,
I ran ruff on the latest commit (8354e74).
Here are the outputs produced.
Results can also be downloaded as artifacts here.
Summarised output:

Details

3	F811	[*] redefined-while-unused
1	I001	[*] unsorted-imports
Found 4 errors.
[*] 4 fixable with the `--fix` option.

Complete output(might be large):

Details

tardis/visualization/tools/tests/test_convergence_plot.py:3:1: I001 [*] Import block is un-sorted or un-formatted
tardis/visualization/tools/tests/test_convergence_plot.py:14:25: F811 [*] Redefinition of unused `defaultdict` from line 3
tardis/visualization/tools/tests/test_convergence_plot.py:15:32: F811 [*] Redefinition of unused `go` from line 6
tardis/visualization/tools/tests/test_convergence_plot.py:16:30: F811 [*] Redefinition of unused `u` from line 8
Found 4 errors.
[*] 4 fixable with the `--fix` option.