chore: remove component: dataframe to toolset

[edwinjosechittilappilly]

This pull request removes the unit tests for the DataFrameToToolsetComponent from the codebase. The test file test_dataframe_toolset.py is deleted, which means automated testing for this component is no longer present. No changes were made to the implementation of the component itself.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Walkthrough

Two component classes in the processing module were updated to include a new public legacy = True attribute: DataFrameToToolsetComponent and DynamicCreateDataComponent. No logic or behavioral changes were introduced.

Changes

Cohort / File(s)	Summary
Component Legacy Attributes src/lfx/src/lfx/components/processing/dataframe_to_toolset.py, src/lfx/src/lfx/components/processing/dynamic_create_data.py	Added public class attribute legacy = True to DataFrameToToolsetComponent and DynamicCreateDataComponent to signal legacy behavior status.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

This change involves adding a simple class attribute to two files with no logic modifications or behavioral changes.

Suggested labels

size:XS, lgtm

Suggested reviewers

• Yukiyukiyeah
• erichare

Pre-merge checks and finishing touches

❌ Failed checks (1 error, 4 warnings)

Check name	Status	Explanation	Resolution
Test Coverage For New Implementations	❌ Error	PR adds legacy attribute to two components without corresponding test coverage verifying the attribute is set correctly or that components remain functional.	Add test files verifying the legacy attribute is correctly set on both components and that they remain functional after being marked as legacy.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.
Test Quality And Coverage	⚠️ Warning	PR adds legacy attribute to components but lacks tests verifying this attribute, inconsistent with established codebase patterns.	Add test_component_legacy_status methods to test files verifying legacy=True for both modified components.
Test File Naming And Structure	⚠️ Warning	Pull request adds legacy = True attribute to components but existing tests lack assertions verifying this new attribute.	Add assert component.legacy is True assertions to test_component_properties methods in both test files to verify the legacy attribute.
Title check	⚠️ Warning	The PR title states 'remove component: dataframe to toolset', but the actual changes add a legacy=True attribute to mark components as legacy rather than removing them.	Update the title to accurately reflect the changes, such as 'chore: Mark processing components as legacy' to match the actual modifications and PR objectives.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Excessive Mock Usage Warning	✅ Passed	Test files for modified components contain no mocks and use real instances to verify actual behavior, demonstrating excellent test design practices.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[github-actions]

Frontend Unit Test Coverage Report

Coverage Summary

Lines	Statements	Branches	Functions
	15.29% (4188/27381)	8.49% (1778/20935)	9.6% (579/6031)

Unit Test Results

Tests	Skipped	Failures	Errors	Time
1638	0 💤	0 ❌	0 🔥	21.422s ⏱️

[codecov]

Codecov Report

❌ Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (release-1.7.0@4df89f1). Learn more about missing BASE report.

Files with missing lines	Patch %	Lines
...rc/lfx/src/lfx/base/embeddings/embeddings_class.py	0.00%	21 Missing ⚠️

Additional details and impacted files

@@               Coverage Diff                @@
##             release-1.7.0   #10643   +/-   ##
================================================
  Coverage                 ?   32.52%           
================================================
  Files                    ?     1367           
  Lines                    ?    63359           
  Branches                 ?     9368           
================================================
  Hits                     ?    20608           
  Misses                   ?    41717           
  Partials                 ?     1034           

Flag	Coverage Δ
backend	51.62% <ø> (?)
lfx	39.98% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...rc/lfx/src/lfx/base/embeddings/embeddings_class.py	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[edwinjosechittilappilly]

@carlosrcoelho @Cristhianzl can you take a look into this PR and merge it?