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.

Sign up for GitHub

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

test: Add Go tests #440

Merged
merged 15 commits into from
Oct 23, 2023
Merged

test: Add Go tests #440

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
b8ffb5c
TSI-2123 add basic golang test cases
hahmed-dev Oct 20, 2023
497aac7
TSI-2123 add go test runner
hahmed-dev Oct 20, 2023
025025f
TSI-2123 add go test runner
hahmed-dev Oct 20, 2023
f1f06a5
TSI-2123 add go test runner
hahmed-dev Oct 20, 2023
da064d6
Test with only openapi generator command
hahmed-dev Oct 20, 2023
d7285a5
Test with only openapi generator command
hahmed-dev Oct 20, 2023
3a7e685
Test with goimport commands
hahmed-dev Oct 20, 2023
43d2cde
Test incorrect assertions
hahmed-dev Oct 20, 2023
0e7a4c2
Revert incorrect assertion
hahmed-dev Oct 20, 2023
29ae575
Comment out linting for Go
hahmed-dev Oct 20, 2023
ed456d8
readd goimport installation
hahmed-dev Oct 20, 2023
f7330e1
Change test runner to match the build and deploy versions
hahmed-dev Oct 20, 2023
ec065c3
Trigger make file after go setup
hahmed-dev Oct 20, 2023
24ec8c4
Assertion for serialization check added
hahmed-dev Oct 20, 2023
ef62b54
Revert gitignore changes
hahmed-dev Oct 20, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 20 additions & 0 deletions .github/workflows/test-go.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
name: Run Go Tests
on: [push]
jobs:
test:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- name: Build
run: |
npm install
npm run generate.go
- name: Setup Go
uses: actions/setup-go@v4
with:
go-version: '1.21.0'
working-directory: ./clients/go
- name: Run Tests
run: |
cd ./clients/go/test
go test
15 changes: 13 additions & 2 deletions .gitignore
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
clients/go/

clients/python/*
!clients/python/test/test_locales_api.py
!clients/python/test/test_uploads_api.py
Expand Down Expand Up @@ -56,6 +54,19 @@ clients/php/test/Api/*
!clients/php/test/Api/LocalesApiTest.php
!clients/php/test/Api/UploadsApiTest.php

clients/go/*.go
clients/go/README.md
Copy link
Collaborator

Choose a reason for hiding this comment

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

We've discussed this here #427 (comment) and decided not to go this individual exclusion route. so, feel free to leave this file as is, and only git add -f files that need to be added.

clients/go/git_push.sh
clients/go/go.mod
clients/go/go.sum
clients/go/.gitignore
clients/go/.openapi-generator
clients/go/api/*
clients/go/docs
clients/go/test/*
!clients/go/test/api_locales_test.go
!clients/go/test/api_uploads_test.go

clients/cli/cmd/api_*.go
clients/cli/phrase-cli

Expand Down
157 changes: 157 additions & 0 deletions clients/go/test/api_locales_test.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

77 changes: 77 additions & 0 deletions clients/go/test/api_uploads_test.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading