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

LINK-1891 | Fix failing tests #330

Merged
merged 3 commits into from
Feb 6, 2024
Merged

LINK-1891 | Fix failing tests #330

merged 3 commits into from
Feb 6, 2024

Conversation

jorilindell
Copy link
Contributor

@jorilindell jorilindell commented Feb 6, 2024

Description

Fix failing test by is caused by the missing prop in the test file.
Fix failing e2e tests.
Add e2e tests for the footer links which still are missing tests

Closes

LINK-1891

@jorilindell jorilindell requested review from harriris-vincit and a team February 6, 2024 08:56
@terovirtanen
Copy link
Contributor

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr330-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is failed for https://linkedcomponents-pr330-ui.dev.hel.ninja 😿💢💥💥

1 similar comment
@terovirtanen
Copy link
Contributor

e2e tests result is failed for https://linkedcomponents-pr330-ui.dev.hel.ninja 😿💢💥💥

@jorilindell
Copy link
Contributor Author

e2e tests result is failed for https://linkedcomponents-pr330-ui.dev.hel.ninja 😿💢💥💥

Nice to have e2e-tests as a part of the review pipeline. I fixed failing e2e-tests and added also some new one for the new footer links. Hopefully the pipeline will pass after these changes.

@terovirtanen
Copy link
Contributor

LINKEDCOMPONENTS-UI branch is deployed to platta: https://linkedcomponents-pr330-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

e2e tests result is success for https://linkedcomponents-pr330-ui.dev.hel.ninja 😆🎉🎉🎉

Copy link

sonarcloud bot commented Feb 6, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

0.0% Coverage on New Code (required ≥ 65%)

See analysis details on SonarCloud

@jorilindell jorilindell changed the title add: add missing prop to a test file LINK-1891 | Fix failing tests Feb 6, 2024
@jorilindell jorilindell merged commit 7c8e302 into main Feb 6, 2024
18 of 19 checks passed
@jorilindell jorilindell deleted the fix/add-missing-prop branch February 6, 2024 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants