Skip to content

ci: integration test with Action Text #1235

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

Merged
merged 2 commits into from
May 13, 2025

Conversation

flavorjones
Copy link
Member

No description provided.

@flavorjones flavorjones force-pushed the flavorjones/rails-integration-tests branch 3 times, most recently from ea5516d to 5945063 Compare May 13, 2025 21:35
@flavorjones flavorjones force-pushed the flavorjones/rails-integration-tests branch from 5945063 to 5c8ef0a Compare May 13, 2025 21:41
@flavorjones flavorjones merged commit 55ef9be into main May 13, 2025
6 checks passed
@flavorjones flavorjones deleted the flavorjones/rails-integration-tests branch May 13, 2025 21:44
- name: Action Text tests
run: |
cd rails/actiontext
bundle exec rake test test:system
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is exciting! Nice work.

Were there any lessons learned in this process that could be applied to rails/rails? From what I recall, buildkite isn't currently executing the System Test suite for Action Text as part of CI runs.

Copy link
Member Author

Choose a reason for hiding this comment

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

@seanpdoyle I don't think I have anything to add to those threads, which seem related at various points to Buildkite, container networking, karma browser coverage, and selenium configuration. I don't know much about any of those things!

By default, Action Text's system tests will use headless chrome, which is installed on the Github Actions runner so it Just Worked™ for me for this limited use case.

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.

2 participants