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

fix: revert deletion of tests auto-run after pull_request changes #64

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

bugITwhisperer
Copy link
Collaborator

@bugITwhisperer bugITwhisperer commented Nov 28, 2023

Resolves #19

Continuation of PR #63

which was merged by accident

Scope of changes

  • revert deletion of running on pr changes (must have been fixed in another PR, comment by @kat-kan)
  • skip running tests if the changes in PR include only documentation changes (only .md files are modified)
  • add CI badge w/ current main testing branch status
  • remove running tests on every push to tests branch (to be discussed; it was initially a part of PR but can be reverted, comment by @kat-kan)

PR steps completed:

  • The PR is associated with the relevant ISSUE
  • all tests have PASS status
  • Commit messages follow the kawqa-gad-playwright project's git commit message format

@bugITwhisperer bugITwhisperer changed the title fix: revert pr deletion fix: revert deletion of tests auto-run after pull_request changes Nov 28, 2023
@bugITwhisperer bugITwhisperer self-assigned this Nov 28, 2023
@bugITwhisperer bugITwhisperer added the chore: configuration Project configuration label Nov 29, 2023
@kat-kan
Copy link
Owner

kat-kan commented Jan 20, 2024

It's not visible in PR #63 yet somehow most of the tests are currently skipped on the main branch. Is it something that is necessary?

kat-kan
kat-kan previously approved these changes Jan 20, 2024
adamcegielka
adamcegielka previously approved these changes Jan 22, 2024
@bugITwhisperer
Copy link
Collaborator Author

@kat-kan : they should not be skipped - I've just removed the skip condition from the test files

@bugITwhisperer
Copy link
Collaborator Author

bugITwhisperer commented Feb 3, 2024

@kat-kan , @adamcegielka , @KarolinaZakrzewska , @Justyna-KO : could you please review the changes?

Also, please note that I marked 2 tests as fixme from the tests\api\articles-post.spec.ts suite since they are failing on CI:

  • Returns 401 Unauthorized after sending article JSON with missing userId
  • Returns 201 Created after creating article

Justyna-KO
Justyna-KO previously approved these changes Feb 4, 2024
adamcegielka
adamcegielka previously approved these changes Feb 5, 2024
@kat-kan kat-kan self-assigned this Jul 5, 2024
# Conflicts:
#	.github/workflows/pw-tests.yml
#	tests/api/articles/articles-post.spec.ts
@kat-kan
Copy link
Owner

kat-kan commented Jul 7, 2024

@Justyna-KO @Slawomir-DKl I updated this branch and PR description, the history of branch is overwhelming but eventually the changes not so much :) please review!

Copy link
Collaborator

@Slawomir-DKl Slawomir-DKl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have one remark only

.github/workflows/pw-tests.yml Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore: configuration Project configuration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[GitHub Project] auto-trigger tests run on CI before merging changes to the base branch (main)
5 participants