Skip to content

chore: move lazy and standalone tests to subdirectories - #27881

Merged
liamdebeasi merged 4 commits into
FW-4612from
4612-subdirs
Aug 1, 2023
Merged

chore: move lazy and standalone tests to subdirectories#27881
liamdebeasi merged 4 commits into
FW-4612from
4612-subdirs

Conversation

@liamdebeasi

@liamdebeasi liamdebeasi commented Jul 28, 2023

Copy link
Copy Markdown
Contributor

Issue number: N/A


What is the current behavior?

During development I found it difficult to determine where to place standalone vs lazy tests. We currently have a single directory full of all the components. As these tests apps grow (and as we have more standalone test) I think it would help to have separate directories for each.

What is the new behavior?

  • All tests that use lazy loaded components are now in the lazy directory. This applies to both the cypress tests as well as the Angular components.
  • All tests that use standalone components are now in the standalone directory. This applies to both the cypress tests as well as the Angular components.
  • Renamed the standalone directory to routing-standalone so it's clear that this is testing lazy loaded Ionic components and routing with an Angular standalone component.

Does this introduce a breaking change?

  • Yes
  • No

Other information

@bolt-new-by-stackblitz

Copy link
Copy Markdown

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@liamdebeasi liamdebeasi changed the title 4612 subdirs chore: move lazy and standalone tests to subdirectories Jul 28, 2023
@github-actions github-actions Bot added the package: angular @ionic/angular package label Jul 28, 2023
@liamdebeasi
liamdebeasi requested a review from sean-perkins July 28, 2023 14:44
@liamdebeasi
liamdebeasi marked this pull request as ready for review July 28, 2023 15:00
@liamdebeasi
liamdebeasi merged commit 291b131 into FW-4612 Aug 1, 2023
@liamdebeasi
liamdebeasi deleted the 4612-subdirs branch August 1, 2023 19:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package: angular @ionic/angular package

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants