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

docs(cypress): update file structure #7016

Open
wants to merge 4 commits into
base: refactor-add-memory-cache
Choose a base branch
from

Conversation

pixincreate
Copy link
Member

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

This PR updates Cypress documentation to deal with recent file structure change.

  • cypress-tests folder structure to reduce confusion and chaos in within e2e directory
    • previous:

       .
       └── cypress
           ├── e2e
           │   ├── PaymentMethodListTest
           │   ├── PaymentMethodListUtils
           │   ├── PaymentTest
           │   ├── PaymentUtils
           │   ├── PayoutTest
           │   ├── PayoutUtils
           │   ├── RoutingTest
           │   └── RoutingUtils
           ├── fixtures
           ├── support
           └── utils
    • present:

       .
       └── cypress
           ├── e2e
           │   ├── configs
           │   │   ├── Payment
           │   │   ├── PaymentMethodList
           │   │   ├── Payout
           │   │   └── Routing
           │   └── spec
           │       ├── Misc
           │       ├── Payment
           │       ├── PaymentMethodList
           │       ├── Payout
           │       └── Routing
           ├── fixtures
           ├── support
           └── utils

closes #6955

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

Just trying to be up-to-date with the latest changes.

How did you test it?

NA

Checklist

  • I formatted the code npm run format
  • I addressed lints thrown by npm run lint
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@pixincreate pixincreate added C-doc Category: Documentation updates S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Jan 9, 2025
@pixincreate pixincreate added this to the December 2024 Release milestone Jan 9, 2025
@pixincreate pixincreate self-assigned this Jan 9, 2025
@pixincreate pixincreate requested a review from a team as a code owner January 9, 2025 12:16
Copy link

semanticdiff-com bot commented Jan 9, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  cypress-tests/README.md Unsupported file format
  cypress-tests/cypress/support/e2e.js  0% smaller

Comment on lines 19 to 20
- [Getting Started](#getting-started)
- [Prerequisites](#prerequisites)
- [Installation](#installation)
- [Running Tests](#running-tests)
- [Development Mode (Interactive)](#development-mode-interactive)
- [CI Mode (Headless)](#ci-mode-headless)
- [Execute tests against multiple connectors or in parallel](#execute-tests-against-multiple-connectors-or-in-parallel)
- [Prerequisites](#prerequisites)
Copy link
Member

Choose a reason for hiding this comment

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

Why are "Getting Started", "Prerequisites", "Installation", etc. all h2, and "Getting Started" has no data?

@pixincreate pixincreate linked an issue Jan 10, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-doc Category: Documentation updates S-waiting-on-review Status: This PR has been implemented and needs to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] update cypress docs
2 participants