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 post token test and add .vscode gitignore #37

Conversation

y2468101216
Copy link
Collaborator

@y2468101216 y2468101216 commented Feb 5, 2021

📝 相關的 issue / Related Issues

⛏ 變更內容 / Details of Changes

  • 增加 route_token_test.go
  • 增加 gitignore .vscode

完成了 route_token_test.go 的測試部分,使用 mock

route_token_test.go Outdated Show resolved Hide resolved
@teyuchang
Copy link
Contributor

Could you please rebase onto the latest development branch?
We have changed the project layout in this PR #23

@y2468101216
Copy link
Collaborator Author

Could you please rebase onto the latest development branch?
We have changed the project layout in this PR #23

ok,I will rebase on Monday.

@codecov-io
Copy link

Codecov Report

Merging #37 (fcfb82e) into development (2963132) will increase coverage by 3.45%.
The diff coverage is 33.33%.

Impacted file tree graph

@@              Coverage Diff               @@
##           development     #37      +/-   ##
==============================================
+ Coverage         6.23%   9.69%   +3.45%     
==============================================
  Files               19      20       +1     
  Lines              577     588      +11     
==============================================
+ Hits                36      57      +21     
+ Misses             538     526      -12     
- Partials             3       5       +2     
Impacted Files Coverage Δ
internal/mock/user_record.go 27.27% <27.27%> (ø)
internal/mock/usecase.go 46.66% <100.00%> (+6.66%) ⬆️
internal/delivery/http/route.go 11.36% <0.00%> (+3.40%) ⬆️
internal/delivery/http/route_token.go 48.57% <0.00%> (+40.00%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2963132...fcfb82e. Read the comment docs.

@y2468101216
Copy link
Collaborator Author

Could you please rebase onto the latest development branch?
We have changed the project layout in this PR #23

reabse done. if I need to squash, please tell me

internal/delivery/http/route_users_test.go Outdated Show resolved Hide resolved
internal/mock/user_record.go Outdated Show resolved Hide resolved
gn01842919 and others added 2 commits February 17, 2021 00:21
…nfomation

Implement issue Ptt-official-app#58 [主線] [PTT] 實作個人看板的使用者資訊 (repository)
@y2468101216 y2468101216 mentioned this pull request Feb 17, 2021
5 tasks
@y2468101216 y2468101216 changed the base branch from development to development_enable_lint February 17, 2021 06:33
@PichuChen
Copy link
Member

LGTM

@y2468101216 y2468101216 self-assigned this Feb 18, 2021
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.

None yet

8 participants