Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix: remove console.log statements from e2e_extract test file (#1234)#3700

Open
guangyang1206 wants to merge 1 commit into
firecrawl:mainfirecrawl/firecrawl:mainfrom
guangyang1206:fix-remove-console-logs-testsguangyang1206/firecrawl:fix-remove-console-logs-testsCopy head branch name to clipboard
Open

fix: remove console.log statements from e2e_extract test file (#1234)#3700
guangyang1206 wants to merge 1 commit into
firecrawl:mainfirecrawl/firecrawl:mainfrom
guangyang1206:fix-remove-console-logs-testsguangyang1206/firecrawl:fix-remove-console-logs-testsCopy head branch name to clipboard

Conversation

@guangyang1206
Copy link
Copy Markdown

@guangyang1206 guangyang1206 commented Jun 3, 2026

PR Title: fix: remove console.log statements from e2e_extract test file

Summary

This PR removes debug console.log statements from the apps/api/src/__tests__/e2e_extract/index.test.ts file. These debug statements were likely left over from development and should be removed for cleaner test output.

Changes Made

  • Removed 8 console.log statements from the e2e_extract test file
  • Cleaned up test output by removing unnecessary debug statements
  • Improved test code maintainability

Testing

  • Verified that all tests still pass after removing the console.log statements
  • The test logic remains unchanged, only debug output was removed

Screenshots

N/A (test code cleanup)

Related Issues

Fixes: #1234 (good first issue - code cleanup)

Type of Change

  • Code cleanup (non-breaking change, no functional impact)
  • Bug fix
  • New feature
  • Breaking change

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works (if applicable)
  • New and existing unit tests pass locally with my changes

Additional Notes

This is a simple cleanup PR that removes debug statements from test files. This improves the development experience by providing cleaner test output and makes the codebase more maintainable.


Summary by cubic

Removed 8 debug console.log statements from apps/api/src/tests/e2e_extract/index.test.ts to clean up test output. No logic changes; tests still pass.

Written for commit ab1f196. Summary will update on new commits.

Review in cubic

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Re-trigger cubic

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.