Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating Tests and Fixing Related Bugs #2319

Merged
merged 16 commits into from
Jun 25, 2024
Merged

Updating Tests and Fixing Related Bugs #2319

merged 16 commits into from
Jun 25, 2024

Conversation

Cristhianzl
Copy link
Collaborator

This pull request includes updates to our test suite and fixes several bugs identified during testing. The key changes and improvements are as follows:

πŸ› (tableNodeCellRender): fix templateValue check to use Object.keys
✨ (textAreaComponent): add Case component for conditional rendering
♻️ (editNodeModal): remove commented-out useEffect code

βœ… (basicExamples.spec): update test selectors and increase timeout values
βœ… (filterEdge.spec.ts): update test IDs to match new naming conventions
βœ… (inputListComponent.spec.ts, intComponent.spec.ts): update test IDs and streamline test steps for consistency and clarity
βœ… (keyPairListComponent.spec.ts): update test ID for model element
βœ… (keyPairListComponent.spec.ts): add steps to test editing model options and saving changes

✨ (textAreaComponent): add Case component for conditional rendering
♻️ (editNodeModal): remove commented-out useEffect code
βœ… (basicExamples.spec): update test selectors and increase timeout values

βœ… (filterEdge.spec.ts): update test IDs to match new naming conventions

βœ… (inputListComponent.spec.ts, intComponent.spec.ts): update test IDs and streamline test steps for consistency and clarity

βœ… (keyPairListComponent.spec.ts): update test ID for model element
βœ… (keyPairListComponent.spec.ts): add steps to test editing model options and saving changes
@Cristhianzl Cristhianzl self-assigned this Jun 24, 2024
Copy link

vercel bot commented Jun 24, 2024

The latest updates on your projects. Learn more about Vercel for Git β†—οΈŽ

Name Status Preview Comments Updated (UTC)
langflow βœ… Ready (Inspect) Visit Preview πŸ’¬ Add feedback Jun 25, 2024 4:25pm

@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jun 24, 2024
@dosubot dosubot bot added bug Something isn't working enhancement New feature or request labels Jun 24, 2024
…mponents

- Update test selectors for better accuracy
- Add keyboard interaction in chatInputOutputUser.spec.ts
- Improve file handling in chat image upload test
- Refine prompt modal component tests for better validation
- Enhance twoEdges.spec.ts with additional view controls
…o improve stability

βœ… (textInputOutput.spec.ts): add waitFor visibility checks to ensure elements are visible before interaction
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Jun 25, 2024
βœ… (tests): add waitForSelector to ensure elements are loaded before interaction
…peed up tests

βœ… (chatInputOutputUser.spec.ts): increase timeout for AI response to 100000ms
βœ… (chatInputOutputUser.spec.ts): correct file path for image upload test
βœ… (deleteComponentFlows.spec.ts): change waitFor to target last checkbox-component
βœ… (store.spec.ts): increase timeout for share button to 100000ms
@ogabrielluiz ogabrielluiz merged commit 4b62e87 into dev Jun 25, 2024
11 of 18 checks passed
@ogabrielluiz ogabrielluiz deleted the cz/tests-fix branch June 25, 2024 16:23
ogabrielluiz added a commit that referenced this pull request Jun 25, 2024
* πŸ› (tableNodeCellRender): fix templateValue check to use Object.keys
✨ (textAreaComponent): add Case component for conditional rendering
♻️ (editNodeModal): remove commented-out useEffect code
βœ… (basicExamples.spec): update test selectors and increase timeout values

βœ… (filterEdge.spec.ts): update test IDs to match new naming conventions

βœ… (inputListComponent.spec.ts, intComponent.spec.ts): update test IDs and streamline test steps for consistency and clarity

βœ… (keyPairListComponent.spec.ts): update test ID for model element
βœ… (keyPairListComponent.spec.ts): add steps to test editing model options and saving changes

* βœ… (tests): update end-to-end tests for chat input/output and modal components

- Update test selectors for better accuracy
- Add keyboard interaction in chatInputOutputUser.spec.ts
- Improve file handling in chat image upload test
- Refine prompt modal component tests for better validation
- Enhance twoEdges.spec.ts with additional view controls

* ♻️ (tests): refactor repeated click actions into reusable function in e2e tests

* βœ… (tests): replace waitForTimeout with waitForSelector in end-to-end tests

* βœ… (typescript_test.yml): add --debug flag to Playwright test command for better debugging

* βœ… (typescript_test.yml): enable trace option in Playwright tests for better debugging

* βœ… (typescript_test.yml): reduce Playwright test workers from 2 to 1 to improve stability
βœ… (textInputOutput.spec.ts): add waitFor visibility checks to ensure elements are visible before interaction

* βœ… (tests): update paths for test assets to correct locations
βœ… (tests): add waitForSelector to ensure elements are loaded before interaction

* βœ… (typescript_test.yml): increase Playwright workers from 1 to 2 to speed up tests
βœ… (chatInputOutputUser.spec.ts): increase timeout for AI response to 100000ms
βœ… (chatInputOutputUser.spec.ts): correct file path for image upload test
βœ… (deleteComponentFlows.spec.ts): change waitFor to target last checkbox-component
βœ… (store.spec.ts): increase timeout for share button to 100000ms

* πŸ› (flows.py): ensure flow names are unique by appending a number if necessary

* Apply Ruff formatting

---------

Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]>
Co-authored-by: Cristhianzl <[email protected]>
ogabrielluiz added a commit that referenced this pull request Jun 25, 2024
* πŸ› (tableNodeCellRender): fix templateValue check to use Object.keys
✨ (textAreaComponent): add Case component for conditional rendering
♻️ (editNodeModal): remove commented-out useEffect code
βœ… (basicExamples.spec): update test selectors and increase timeout values

βœ… (filterEdge.spec.ts): update test IDs to match new naming conventions

βœ… (inputListComponent.spec.ts, intComponent.spec.ts): update test IDs and streamline test steps for consistency and clarity

βœ… (keyPairListComponent.spec.ts): update test ID for model element
βœ… (keyPairListComponent.spec.ts): add steps to test editing model options and saving changes

* βœ… (tests): update end-to-end tests for chat input/output and modal components

- Update test selectors for better accuracy
- Add keyboard interaction in chatInputOutputUser.spec.ts
- Improve file handling in chat image upload test
- Refine prompt modal component tests for better validation
- Enhance twoEdges.spec.ts with additional view controls

* ♻️ (tests): refactor repeated click actions into reusable function in e2e tests

* βœ… (tests): replace waitForTimeout with waitForSelector in end-to-end tests

* βœ… (typescript_test.yml): add --debug flag to Playwright test command for better debugging

* βœ… (typescript_test.yml): enable trace option in Playwright tests for better debugging

* βœ… (typescript_test.yml): reduce Playwright test workers from 2 to 1 to improve stability
βœ… (textInputOutput.spec.ts): add waitFor visibility checks to ensure elements are visible before interaction

* βœ… (tests): update paths for test assets to correct locations
βœ… (tests): add waitForSelector to ensure elements are loaded before interaction

* βœ… (typescript_test.yml): increase Playwright workers from 1 to 2 to speed up tests
βœ… (chatInputOutputUser.spec.ts): increase timeout for AI response to 100000ms
βœ… (chatInputOutputUser.spec.ts): correct file path for image upload test
βœ… (deleteComponentFlows.spec.ts): change waitFor to target last checkbox-component
βœ… (store.spec.ts): increase timeout for share button to 100000ms

* πŸ› (flows.py): ensure flow names are unique by appending a number if necessary

* Apply Ruff formatting

---------

Co-authored-by: Gabriel Luiz Freitas Almeida <[email protected]>
Co-authored-by: Cristhianzl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request size:XL This PR changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants