Add vitest to GitHub actions #9162
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Failing tests are expected per #9083 (comment). Successful run can be seen in https://github.com/pantheon-systems/documentation/actions/runs/10307574827/job/28533128469?pr=9161 |
stevector
reviewed
Aug 8, 2024
stevector
reviewed
Aug 8, 2024
stevector
approved these changes
Aug 8, 2024
|
I left it open ended in case we wanted to use it for other types of tests, but since we're running on specific paths, that's probably fine. |
rachelwhitton
added a commit
that referenced
this pull request
Oct 24, 2024
* mdx and import ordering * Update index.js * whitespace fixes * BackstopJS init for local VRT (#8989) * removing potentially unneeded component * Revert "removing potentially unneeded component" This reverts commit d95bca8. * removing releaseHeadlines from MdxWrapper * adding more pages to backstop * adding npx to instructions * mdx customshortcodes * Revert "BackstopJS init for local VRT (#8989)" This reverts commit 5b9f380. * starting on a new sidebar * getting sidebar somewhat functional * more sidebar nav playing * Certification multilevel nav * Getting certification working with multilevel menu * more sidebar experimenting * sidebar nav graphql-ing * adding guides to omni nav * adding subtitles * adding stuff * more nav * link nesting * more link mucking * still working * still working * removing commented out code * cleanup * starting on omni component * trying function in separate file * functioning again * submenus * wrangling the guides * getting decoupled guides * more decouple guiding * more simplification * splitting up * segmenting out guide function * delete unused files * guide! * whitespace * more re-arranging * no errors * remove unused stuff * no errors with query coming from outside index.js * more tweaks * adding OmniSidebarNavWithFallback.js * fallback is working in guide * whitespace * using fallback * more cleanup * getting rid of separate OmniSidebarNavWithFallback * more cleanup * link fix * screenshot testing nav * update prettier * running prettier * switching prettier standard based on internal discussion * adding eslint * adding eslint * Revert "adding eslint" This reverts commit f84fc28. * Revert "adding eslint" This reverts commit 7705f80. * Revert "running prettier" This reverts commit e4f05fb. * small prettier formatting * formatting * manual spacing * more manual * manual formatting * prettier with manual fix * prettier fixes * simplify regex * prettier comment * Revert "prettier comment" This reverts commit 7f95a83. * going back to work regex * I do not even need the regex * adding vitest * adding unit tests samples * moving helper functions to their own file * more testing * query in its own file * more unit testing * one more unit test * fix * removing helper functions that were too small * more moving helper functions * moving deck chairs * removing a too small function. And removing sample unit test * deleting separate sidebarNav component * moving test fixture * simplyfing a function * simplify data structures * running prettier * checking for search js before adding it * cleanup * adding some doc blocks and minor refactor * docs and cleanup * more comments * more cleanup * more comments more tests * adding all the front-end sites guides * removing extra nav alert in the middle of decouple overview pages * running prettier * going back to old prettier config files. Redo this change under #9082 * starting to add a styling file for omnisidebarnav * starting on styling * getting external link indicator working * remove extra arrow in link * removing console.log * Add vitest to GitHub actions (#9162) * Add vitest to GHA * Make a clean install * switching from jsx to React.createElement() * Update and rename test.yml to vitest.yml --------- Co-authored-by: Steve Persch <[email protected]> Co-authored-by: Steve Persch <[email protected]> * moving icon to back and setting to small * testing readme * fixing link * copy fix * changing addsearch back * addsearch back * adding vite.config.ts * Updating import order * update tests readme * Apply Prettier formatting * removing comment * vitest fix, fingers crossed * adding rollup * package merges * re-adding vitest stuff --------- Co-authored-by: Rachel Whitton <[email protected]> Co-authored-by: Chris Reynolds <[email protected]> Co-authored-by: Pantheon Bot <[email protected]>
rachelwhitton
added a commit
that referenced
this pull request
Dec 18, 2024
* moving deck chairs * removing a too small function. And removing sample unit test * deleting separate sidebarNav component * moving test fixture * simplyfing a function * simplify data structures * running prettier * checking for search js before adding it * cleanup * adding some doc blocks and minor refactor * docs and cleanup * more comments * more cleanup * more comments more tests * adding all the front-end sites guides * removing extra nav alert in the middle of decouple overview pages * running prettier * going back to old prettier config files. Redo this change under #9082 * reporting files * reporting tests * more reporting tools * tests for reporting * more tests and console logging * moving reporting function * more testing of reporting * reporting * getting percentages calculated * adding contributors for exceptions examples * starting to unwind logic error that had percentages reversed * fixing logic * more logic fixing * getting close * 8%! * removing unused stuff * adding to menu * more menus * 55% * migrating and upgrading guides * more guides in menus * 199! * updating exceptions * Adding menu to video template * putting omni items in doc template * starting on DNS * putting everything in a menu * more sidebars * moving simpleLink to helper * breaking out file for submenu * breaking out menus * two more * breaking out menus * assigning menus * a few more assignments * getting doc template with sidebar * dns menus * lots of menu moving * Drupal and WordPress separate menus * moving menus * adding about section * everything in a top level menu * build tools menu link * comment about nested docroot * starting to add a styling file for omnisidebarnav * starting on styling * getting external link indicator working * remove extra arrow in link * removing console.log * workflows() * getting rendering of default docs pages working * code organizing * adding comment * adding comments * reporting on status codes * Moving submenus around * more * Adding label * Add vitest to GitHub actions (#9162) * Add vitest to GHA * Make a clean install * switching from jsx to React.createElement() * Update and rename test.yml to vitest.yml --------- Co-authored-by: Steve Persch <[email protected]> Co-authored-by: Steve Persch <[email protected]> * moving icon to back and setting to small * testing readme * fixing link * copy fix * changing addsearch back * addsearch back * adding vite.config.ts * some menu restructuring * menu reorgs * fixing page title * drupal notes and DNS titles * Refining menus with Rachel and Chris. And adding some JS todos * more menu tweaks with Rachel and Chris * fixup terminus template and submenu * whitespace to trigger build * more menu tweaking * terminus menu * adding a comment to a md file * working out a top-level nav * Coworking on submenus - integrations * manually bringing over files from main before merging * manually bringing over prettier config change from main * running prettier on some of the files in omniSidebarNav * Apply Prettier formatting * renaming dnsProviders * working on the migrate section * Apply Prettier formatting * Migration submenu edits * Apply Prettier formatting * Combine migration and getstarted submenus * Debug migration nested submenu * Apply Prettier formatting * Fix link to getstarted submenu * test pre-push hook * test pre-push hook * add prettier pre-push hook to .github/hooks * add git hooks section to readme * actually add the changed files to git * White spaces * Apply Prettier formatting * Apply Prettier formatting * Support submenu * Apply Prettier formatting * Move troubleshooting menu from webinfra * Apply Prettier formatting * Edits to support submenu * Apply Prettier formatting * Edits to accountmanagement submenu * Apply Prettier formatting * add comments * Apply Prettier formatting * Coworking session submenu updates * Apply Prettier formatting * finish workflows section * Apply Prettier formatting * web infra submenu * Apply Prettier formatting * more submenu work * Apply Prettier formatting * Add notes for merge blockers * Apply Prettier formatting * certification menu organizing * Apply Prettier formatting * updating links in header * Apply Prettier formatting * clean up a few links * Cleaning up education section * Apply Prettier formatting * cleaning up support menu * prettier * more reordering in troubleshooting * prettier * getting header functioning with menu of links * header cleanup * top level menu fixes * Apply Prettier formatting * moving a guide * Apply Prettier formatting * reviewing pages to delete * prettier * removing unused import * moving pages to delete * bring back search bar * Apply Prettier formatting * removing popover code * Apply Prettier formatting * removing popover code * Apply Prettier formatting * fixing import order * Apply Prettier formatting * landing page layout * landing page layout * Apply Prettier formatting * some comment cleanup * whitespace * cleaning up the drupal upgrade and migration stuff * moving around links * Apply Prettier formatting * menu updates * prettier * updating all-written-paths.txt * more menu updates * prettier * some cleanup * redistributing landing pages * prettier * Adding search to landing page * adding new pages to menu * prettier * Apply Prettier formatting * Deleting the unassignedPages submenu * redistributing some pages that were previously in "pages to delete" * For easier review, reorder based on work done in pr #9335 * Apply Prettier formatting * Finish pagestodelete submenu * Apply Prettier formatting * Delete all-written-paths.txt * Update source/content/guides/drush/04-drupal-commandline.md * removing todos that are done * Update src/components/omniSidebarNav/submenus/support.js * moving some reporting out of index * prettier * Apply suggestions from code review * Apply Prettier formatting * Terminus menu fix * Apply Prettier formatting --------- Co-authored-by: Chris Reynolds <[email protected]> Co-authored-by: Rachel Whitton <[email protected]> Co-authored-by: Pantheon Bot <[email protected]> Co-authored-by: Chris Reynolds <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #9083
Replaces #9161
Summary
Adds a GitHub actions workflow to run
npx vitest src/componentson PRs that change files insrc/(can be altered to limit that scope tosrc/components/or if the workflow itself is updated.Also allows for manual triggering (
workflow_dispatch) in the Actions tab.