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

feat(project): Add GitHub Actions testing workflow #289

Merged
merged 6 commits into from
May 3, 2020

Conversation

andygrunwald
Copy link
Owner

Description

GitHub actions is a workflow engine.
This testing workflow will

  • keep everything inside GitHub (one platform)
  • reduce dependency to an external service (TravisCI)
  • introduce stricter testing (next to unit tests, staticcheck, fmt, vet)

After this PR is merged, we can deactivate TravisCI

Checklist

GitHub actions is a workflow engine.
This testing workflow will
- keep everything inside GitHub (one platform)
- reduce dependency to an external service (TravisCI)
- introduce stricter testing (next to unit tests, staticcheck, fmt, vet)
@andygrunwald
Copy link
Owner Author

The tests are failing due to #283

* master:
  feat(IssueService): allow empty JQL (#268)
  style: Fix staticcheck (static analysis) errors for this library (#283)
  feat(project): Add workflow to greet new contributors (#288)
  feat(project): Add cronjob to check for stale issues (#287)
  feat(issues): Add GetEditMeta on issue
* master:
  fix(issue): IssueService.Search() with a not empty JQL triggers 400 bad request (#292)
… error message checking

We check the error string in TestIssueService_GetEditMeta_Fail.
On different operting systems, this error message is different. See

- Linux: TestIssueService_GetEditMeta_Fail: metaissue_test.go:456: Error Get "http://127.0.0.1:65328/rest/api/2/issue/PROJ-9001/editmeta": dial tcp 127.0.0.1:65328: connect: connection refused
- Windows: Error Get "http://127.0.0.1:50122/rest/api/2/issue/PROJ-9001/editmeta": dial tcp 127.0.0.1:50122: connectex: No connection could be made because the target machine actively refused it.

Now we check the error type instead of the error message
@andygrunwald andygrunwald merged commit 80c0282 into master May 3, 2020
@andygrunwald andygrunwald deleted the github-actions-testing branch May 3, 2020 13:45
andygrunwald added a commit that referenced this pull request May 12, 2020
* master:
  fix(tests): Fix TestIssueService_PostAttachment unit test
  style: Fix typos
  style: Make code go fmt conform
  chore(test): Remove unit testing log output for success cases (#293)
  feat(project): Add GitHub Actions testing workflow (#289)
  feat(context): Add support for context package
  fix(issue): IssueService.Search() with a not empty JQL triggers 400 bad request (#292)
  feat(IssueService): allow empty JQL (#268)
  style: Fix staticcheck (static analysis) errors for this library (#283)
  feat(project): Add workflow to greet new contributors (#288)
  feat(project): Add cronjob to check for stale issues (#287)
  feat(issues): Add GetEditMeta on issue
  feat: Add Names support on Issue struct (#278)
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.

1 participant