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

Add footer tests #563

Closed
wants to merge 1 commit into from
Closed

Add footer tests #563

wants to merge 1 commit into from

Conversation

HonkingGoose
Copy link
Collaborator

@HonkingGoose HonkingGoose commented Nov 17, 2021

Superseded by PR #575.


Changes

  • Check page shows link to Mario's homepage
  • Check page shows link to Octoclairvoyant's GitHub repository
  • Check page shows Vercel badge

Context

This PR adds some more tests to check if we're actually displaying the wanted items in our footer. Right now it uses find by role, but it's not limited to checking the <footer> section, so it's really checking if the page shows the item, not if the footer specifically contains the items.

I need some help to get the Vercel badge test working properly. I cannot get it to accurately find the src to test that it uses the correct SVG image. I also cannot test the href that comes with the Vercel badge.

Tests

I've checked my work by:

  • Adding new tests or adjusting existing tests
  • Testing manually

@HonkingGoose
Copy link
Collaborator Author

By the way I'm making this PR from a fork, not from within a branch that's already on the main repository. Gitpod doesn't allow me to push to any organization branch. I'll make a private discussion to debug this together. 😉

@Belco90
Copy link
Member

Belco90 commented Nov 20, 2021

Can we move this to a branch within the repo now that we solved the gitpod issue?

@HonkingGoose HonkingGoose deleted the tests/test-footer-more branch November 20, 2021 19:09
@HonkingGoose HonkingGoose removed the request for review from Belco90 November 20, 2021 19:10
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.

2 participants