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

bugITwhisperer commented 1 year ago

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:

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

kat-kan commented 10 months ago

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?

bugITwhisperer commented 10 months ago

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

bugITwhisperer commented 10 months ago

@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

kat-kan commented 4 months ago

@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!

kat-kan / kawqa-gad-playwright