-
Notifications
You must be signed in to change notification settings - Fork 38
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
Feat/toc companions walls cypress test#393 #455
Merged
jouwdan
merged 63 commits into
tutors-sdk:development
from
20041540MichaelKelly:feat/toc-companions-walls-cypress-test#393
Aug 26, 2023
Merged
Feat/toc companions walls cypress test#393 #455
jouwdan
merged 63 commits into
tutors-sdk:development
from
20041540MichaelKelly:feat/toc-companions-walls-cypress-test#393
Aug 26, 2023
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
…tion of reference-manual and reference-course to a new directory called doc. Also Cypress is after being moved into the tutors-publish-html to perform static local testing
…the /app directory and refactored out the static html generator into its new directory of doc/reference-course. directions to run the tests: Cypress Testing: Static html Generator: cd /doc/reference-course npx tutors-publish-html <= This will generate a html folder This html folder is our static html Generator that we will be testing locally Inside the project directory of doc/reference-course run this cmd => npx cypress open This will open the Cypress Testing UI Important! comment back in line 27 if you want the test to run as it should. This line was commented out because the video link was missing. Dynamic Reader: cd /app run this cmd => npx cypress open This will open the Cypress Testing UI
… the most up todate version
✅ Deploy Preview for tutors-dev canceled.
|
… and included in the Cypress test as automation
…t_blank on a website and the link and checking for the correct href in the non website companions
3 tasks
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.
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This is a feat, a refactor of the cypress tests
Cypress Testing:
Static html Generator:
cd /doc/reference-course
npx tutors-publish-html <= This will generate a html folder
This html folder is our static html Generator that we will be testing locally
Inside the project directory of
doc/reference-course
run this cmd => npx cypress open
This will open the Cypress Testing UI
Important! comment back in line 27 if you want the test to run as it should.
This line was commented out because the video link was missing.
What commits does this PR relate to?
pr: feat/toc-companions-walls-cypress-test#393
Thank you for your contribution
We hope you stay around and connect with our growing community!