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 make command to test locally #40

Merged
merged 1 commit into from
Apr 17, 2024
Merged

add make command to test locally #40

merged 1 commit into from
Apr 17, 2024

Conversation

WindzCUHK
Copy link
Contributor

Description

add make command to test locally

  • make test
  • make coverage

Type of change

  • Bug fix
  • New feature
  • Refactoring (no functional changes, no api changes)
  • Non-code changes (update documentation, pipeline, etc.)

Flags

  • Breaks backward compatibility
  • Requires a documentation update
  • Has untestable code

Checklist

  • Followed the guidelines in the CONTRIBUTING document
  • Added prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip] in the PR title if necessary
  • Tested and linted the code
  • Commented the code
  • Made corresponding changes to the documentation
  • Passed all pipeline checking

Checklist for maintainer

  • Use Squash and merge
  • Double-confirm the merge message has prefix [skip ci]/[ci skip]/[no ci]/[skip actions]/[actions skip]
  • Delete the branch after merge

Signed-off-by: wfan <wfan@lycorp.co.jp>
@mlajkim
Copy link
Contributor

mlajkim commented Apr 16, 2024

I get the following error when I run make test:

PASS
ok  	github.com/AthenZ/authorization-proxy/v4/usecase	(cached)
FAIL
make: *** [test] Error 1

@mlajkim
Copy link
Contributor

mlajkim commented Apr 16, 2024

Wonder if there is any sort of documentation in the main README.md where I can use it to set it up and properly run make test

Copy link
Contributor

@mlajkim mlajkim left a comment

Choose a reason for hiding this comment

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

OK

@mlajkim mlajkim merged commit bb645d3 into master Apr 17, 2024
2 checks passed
@mlajkim mlajkim deleted the test-locally branch April 17, 2024 07:33
t4niwa pushed a commit that referenced this pull request Jul 19, 2024
Signed-off-by: wfan <wfan@lycorp.co.jp>
Signed-off-by: taniwa <taniwa@lycorp.co.jp>
t4niwa pushed a commit that referenced this pull request Jul 19, 2024
Signed-off-by: wfan <wfan@lycorp.co.jp>
Signed-off-by: taniwa <taniwa@lycorp.co.jp>
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.

2 participants