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

dressca の Cypress が実行できない #743

Closed
1nu1taichi opened this issue Jan 16, 2024 · 1 comment · Fixed by #888
Closed

dressca の Cypress が実行できない #743

1nu1taichi opened this issue Jan 16, 2024 · 1 comment · Fixed by #888
Assignees
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある 不具合 現在リリース中のバージョンに不具合・バグがある
Milestone

Comments

@1nu1taichi
Copy link
Contributor

1nu1taichi commented Jan 16, 2024

概要

viteの更新により package.json に "type":"module" を追加すると発生するため、それらの更新に紐づけて行う #664

対処法

  • cypress.config.ts → cypress.config.js にする
  • require を使わないようにする(plugin の利用を消す?)

完了条件

npm run test:e2e スクリプトが実行できる

@1nu1taichi 1nu1taichi added 不具合 現在リリース中のバージョンに不具合・バグがある target: Dressca サンプルアプリケーションDresscaに関係がある labels Jan 16, 2024
@tsuna-can-se tsuna-can-se added this to the v0.7 milestone Jan 25, 2024
@tsuna-can-se tsuna-can-se modified the milestones: v0.7, v0.8 Feb 22, 2024
@1nu1taichi
Copy link
Contributor Author

原因

Vite 5 に更新すると、モジュール形式がESMに対応していくため package.json に "type" : "module" を追加する
この時 cypress がESM未対応のためエラーが発生する

対応

cypress-io/cypress#28347
Vite 5 への対応も取り組み中っぽい、、?

ひとまずVite 5 でも動かせるように一時的な修正案

config ファイルの形式を .ts から .js にする

cypress.config.ts とすると以下のエラーが出る

exports is not defined in ES module scope

cypress.config.js では抑えられる
ある程度 vite 5 対応が進み次第 cypress.config.ts に戻したい

plugins の require を削除する

plugins の導入で setupNodeEvents で plugins/index.ts を require している。
ESM では require を使えないため削除する
また plugins の導入方法については Cypress 10 で方式が変更されており、plugins フォルダが不要となった
現段階で plugins は使用していないため削除して、必要に迫られた際に対応したい

import { defineConfig } from 'cypress';

export default defineConfig({
  e2e: {
-   // We've imported your old cypress plugins here.
-   // You may want to clean this up later by importing these.
-   setupNodeEvents(on, config) {
-     return require('./cypress/plugins/index.ts').default(on, config)
-   },
    baseUrl: 'http://localhost:5050',
  },
});

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある 不具合 現在リリース中のバージョンに不具合・バグがある
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants