Skip to content

Do not use solidus_frontend to test extension test app generator#5888

Merged
tvdeyen merged 1 commit intosolidusio:mainfrom
tvdeyen:remove-solidus_frontend-extension-test
Oct 29, 2024
Merged

Do not use solidus_frontend to test extension test app generator#5888
tvdeyen merged 1 commit intosolidusio:mainfrom
tvdeyen:remove-solidus_frontend-extension-test

Conversation

@tvdeyen
Copy link
Copy Markdown
Member

@tvdeyen tvdeyen commented Oct 29, 2024

solidus_frontend is EOL since 2024-10-21 using it to test the extension:test_app rake task is not working since it is not compatible with latest Rails, Ruby and Solidus.

Since we already have a test for the extension test app generator we can check for it's dummy app existence instead.

Extracted from #5843

solidus_frontend is [EOL since 2024-10-21](https://github.com/solidusio/solidus_frontend?tab=readme-ov-file#-warning)
using it to test the extension:test_app rake task is not working since
it is not compatible with latest Rails, Ruby and Solidus.

Since we already have a test for the extension test app generator we
can check for it's dummy app existence instead.
@tvdeyen tvdeyen requested a review from a team as a code owner October 29, 2024 16:50
@github-actions github-actions Bot added the changelog:repository Changes to the repository not within any gem label Oct 29, 2024
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.57%. Comparing base (b36be11) to head (cb8ef4e).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5888   +/-   ##
=======================================
  Coverage   89.57%   89.57%           
=======================================
  Files         782      782           
  Lines       17965    17965           
=======================================
  Hits        16092    16092           
  Misses       1873     1873           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tvdeyen tvdeyen merged commit 0edec31 into solidusio:main Oct 29, 2024
@tvdeyen tvdeyen deleted the remove-solidus_frontend-extension-test branch October 29, 2024 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:repository Changes to the repository not within any gem

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants