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

Jump to bottom

Create unit tests #7

Closed
adrcunha opened this issue Jul 16, 2018 · 2 comments
Closed

Create unit tests #7

adrcunha opened this issue Jul 16, 2018 · 2 comments
Assignees
@adrcunha

Comments

@adrcunha
Copy link
Contributor

No description provided.

@adrcunha
Copy link
Contributor Author

We should add at least a bogus test to exercise the libraries and presubmit script and avoid issues like #135.

@adrcunha
Copy link
Contributor Author

/assign adrcunha

@adrcunha adrcunha mentioned this issue Sep 21, 2018
Merged
knative-prow-robot pushed a commit that referenced this issue Sep 25, 2018
@adrcunha @knative-prow-robot
Create a smoke test for test-infra (#138)
afc9980 
* Create a smoke test for test-infra

This will ensure the test infrastructure is not broken.

Fixes #7.

Bonus: remove the `dummy.go` file.

* Restrict unit tests to `/test`, as it's the only directory with tests
Cynocracy pushed a commit to Cynocracy/test-infra that referenced this issue Jul 22, 2020
Add reviewers to OWNERS. (knative#7)
2915511
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@adrcunha adrcunha

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@adrcunha