Skip to content

Fix Magento 2 Integration Tests - #64

Merged
tuyennn merged 1 commit into
masterfrom
tuyennn-workflow-patch-remove-mage243
Apr 16, 2026
Merged

Fix Magento 2 Integration Tests#64
tuyennn merged 1 commit into
masterfrom
tuyennn-workflow-patch-remove-mage243

Conversation

@tuyennn

@tuyennn tuyennn commented Apr 15, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Fix Magento 2 Integration Tests caused by Ubuntu specific version

Summary by CodeRabbit

  • Documentation

    • Updated search engine compatibility documentation to include OpenSearch alongside ElasticSearch
    • Expanded Magento version compatibility matrix with dedicated Magento 2.4.8 configuration and refined version-specific requirements

  • Chores

    • Relaxed minimum PHP version requirement from 8.1 to 7.4

@coderabbitai

coderabbitai Bot commented Apr 15, 2026 •
edited
Loading

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 98cdf494-7eb6-466b-8dfd-96eccd8120c9

📥 Commits

Reviewing files that changed from the base of the PR and between 8a4e433 and 27c1669.

📒 Files selected for processing (3)
  • .github/workflows/integration.yml
  • README.md
  • composer.json
📝 Walkthrough

Walkthrough

CI workflow updated to trigger only on pull_request, added Magento 2.4.8 integration test job, and standardized existing jobs to ubuntu-22.04. README updated to document OpenSearch support alongside Elasticsearch and new Magento 2.4.8 compatibility details. PHP requirement lowered from 8.1.0 to 7.4 in composer.json.

Changes

Cohort / File(s) Summary
CI/Testing Configuration
.github/workflows/integration.yml		 Workflow trigger changed from push and pull_request to pull_request only. New Magento 2.4.8 test job added with MySQL 8.0 and Elasticsearch 8.1. Existing test jobs for versions 2.4.7, 2.4.6, 2.4.4, 2.4.3 updated to use ubuntu-22.04 instead of ubuntu-latest and configured with block_insecure: false.
Documentation Updates
README.md		 Search engine support expanded from "Elastic Search" to "ElasticSearch or OpenSearch". Compatibility matrix refactored to split Magento 2.4.6 range and add new Magento 2.4.8 row with OpenSearch 3.x and ElasticSuite >=2.11.13 support.
Dependency Management
composer.json		 PHP platform requirement lowered from >=8.1.0 to >=7.4 in the require section.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Hoppy hops for Magento's leap,
To 2.4.8, compatibility so deep,
OpenSearch joins the compatibility choir,
Broader PHP support takes flight ever higher,
Testing grows strong with ubuntu's care,
This code change makes us all declare: ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch tuyennn-workflow-patch-remove-mage243

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

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

@what-the-diff

what-the-diff Bot commented Apr 15, 2026

Copy link
Copy Markdown

PR Summary

  • Removes Redundant mage24 Job Configuration
    This PR eliminates the unnecessary configuration for the mage24 job in our GitHub Action workflow, making the workflow simpler and easier to manage. This includes the removal of associated MySQL and Elasticsearch services, thereby reducing dependencies and potential sources of errors. Additionally, it no longer runs Magento 2 integration tests for version 2.4.3, as they are no longer necessary.

@tuyennn tuyennn changed the title Delete Magento 2 Integration Tests for v2.4.3 Fix Magento 2 Integration Tests Apr 16, 2026
@tuyennn
tuyennn force-pushed the tuyennn-workflow-patch-remove-mage243 branch from 675a39c to 9d04120 Compare April 16, 2026 02:37
@tuyennn
tuyennn force-pushed the tuyennn-workflow-patch-remove-mage243 branch from 9d04120 to 27c1669 Compare April 16, 2026 02:37
@tuyennn
tuyennn marked this pull request as ready for review April 16, 2026 02:41
@tuyennn
tuyennn merged commit b709236 into master Apr 16, 2026
7 of 8 checks passed
@tuyennn
tuyennn deleted the tuyennn-workflow-patch-remove-mage243 branch April 16, 2026 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@tuyennn