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

test: remove extra symbols to avoid 404 pages redirection #229

Conversation

olehbairak
Copy link
Contributor

What ❔

Removing extra symbols that prevents page from loading.
After the test cases were imported from Allure project - some of the pages didn't load because of the different symbols.

Why ❔

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.

@olehbairak olehbairak added documentation Improvements or additions to documentation automation writing/fixing/maintaining auto-tests/e2e-tests labels Apr 12, 2024
@olehbairak olehbairak requested a review from pcheremu April 12, 2024 15:49
@olehbairak olehbairak self-assigned this Apr 12, 2024
Copy link

API E2E Test Results

207 tests  ±0   207 ✅ ±0   19s ⏱️ -1s
 14 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 5c749c8. ± Comparison against base commit 8580045.

Copy link

Visit the preview URL for this PR (updated for commit 5c749c8):

(expires Fri, 19 Apr 2024 15:54:01 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: e508f9012944951194447cb8885950b451a24403

Copy link

Unit Test Results

    4 files  ±0    263 suites  ±0   11m 20s ⏱️ +22s
2 081 tests ±0  2 080 ✅ ±0  1 💤 ±0  0 ❌ ±0 
2 289 runs  ±0  2 288 ✅ ±0  1 💤 ±0  0 ❌ ±0 

Results for commit 5c749c8. ± Comparison against base commit 8580045.

@olehbairak olehbairak marked this pull request as ready for review April 12, 2024 16:00
@pcheremu pcheremu merged commit 42fabcb into QA-613-add-docusaurus-docs-project Apr 12, 2024
27 checks passed
@pcheremu pcheremu deleted the oleh-bairak-qa-786-docusaurus-remove-symbols-that-prevent-loading-pages branch April 12, 2024 17:17
pcheremu pushed a commit that referenced this pull request Apr 15, 2024
# What ❔

Removing extra symbols that prevents page from loading.
After the test cases were imported from Allure project - some of the
pages didn't load because of the different symbols.

## Why ❔

<!-- Why are these changes done? What goal do they contribute to? What
are the principles behind them? -->
<!-- Example: PR templates ensure PR reviewers, observers, and future
iterators are in context about the evolution of repos. -->

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [ ] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ ] Tests for the changes have been added / updated.
- [ ] Documentation comments have been added / updated.
pcheremu pushed a commit that referenced this pull request May 2, 2024
# What ❔

Removing extra symbols that prevents page from loading.
After the test cases were imported from Allure project - some of the
pages didn't load because of the different symbols.

## Why ❔

<!-- Why are these changes done? What goal do they contribute to? What
are the principles behind them? -->
<!-- Example: PR templates ensure PR reviewers, observers, and future
iterators are in context about the evolution of repos. -->

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [ ] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ ] Tests for the changes have been added / updated.
- [ ] Documentation comments have been added / updated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation writing/fixing/maintaining auto-tests/e2e-tests documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants