Skip to content

test/live-tests#5

Merged
DYung26 merged 7 commits into
mainfrom
test/live-tests
Oct 31, 2025
Merged

test/live-tests#5
DYung26 merged 7 commits into
mainfrom
test/live-tests

Conversation

@DYung26
Copy link
Copy Markdown
Owner

@DYung26 DYung26 commented Oct 31, 2025

This pull request adds new live tests for the plotsense package and refactors the plot generator unit and integration test suite for clarity and consistency. The most important changes are grouped below:

New Live Tests

  • Added live_test_explanations.py to test the explainer function for generating explanations of matplotlib plots using selected LLM models and API keys.
  • Added live_test_plotgen.py to test the plotgen function with various plot types and custom plot functions using sample DataFrames.
  • Added live_test_suggestions.py to test the recommender function for recommending visualizations from a DataFrame using multiple LLM providers.

Test Suite Refactoring

  • Renamed test/test_plotgen.py to test/unit/test_plotgen.py and updated imports to use BasicPlotGenerator and SmartPlotGenerator instead of the previous PlotGenerator.
  • Updated fixtures and all test classes to use BasicPlotGenerator for unit, integration, end-to-end, error handling, performance, and edge case tests, improving clarity and aligning with new class naming. [1] [2] [3] [4] [5] [6] [7]
  • Updated test assertions to compare SmartPlotGenerator plot functions against BasicPlotGenerator instead of the old PlotGenerator for consistency.

… `PlotExplainer` and explainer `API` coverage
…icPlotGenerator`, `SmartPlotGenerator`, and `plotgen` integration tests
…`VisualizationRecommender` initialization, parsing, and `LLM` integration
…est_suggestions` into `test/unit` for clearer test structure organization
@DYung26 DYung26 changed the title test/live tests test/live-tests Oct 31, 2025
@DYung26 DYung26 merged commit 61af8ad into main Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant