-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Implement missing unit tests #5068
Comments
I would like to work on this |
@chirag-ghosh thanks. I suggest you pick one of the packages and maybe post here that you're working on it, in case someone else also wants to pick another package. Don't combine multiple packages in one PR. |
Sure. Since I'm new to the project, I'll first setup it in my local and the comment the package I'm working on ASAP. |
I am working on the storageexporter package (https://github.com/jaegertracing/jaeger/tree/main/cmd/jaeger/internal/exporters/storageexporter) |
working on |
working on |
## Which problem is this PR solving? - Part of #5068 ## Description of the changes - added some missing unit tests in /pkg/version ## How was this change tested? - go test ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Pushkar Mishra <pushkarmishra029@gmail.com> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
## Which problem is this PR solving? - Part of #5068 ## Description of the changes - Added Empty Tests for `./examples` to reduce noise in `make nocover`. ## How was this change tested? - go test ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Pushkar Mishra <pushkarmishra029@gmail.com>
## Which problem is this PR solving? - Part of #5068 ## Description of the changes - Add tests to storageexporter package ## How was this change tested? - CI ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Chirag Ghosh <cghosh828049@gmail.com> Signed-off-by: Yuri Shkuro <github@ysh.us> Co-authored-by: Yuri Shkuro <github@ysh.us>
Working on jaegerstorage |
working on Jaeger/plugin/ |
## Which problem is this PR solving? Part of #5068 ## Description of the changes - Added test to adaptive sampling plugin options ## How was this change tested? - make test ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: tarhphilomina <156124227+tarhphilomina@users.noreply.github.com>
## Which problem is this PR solving? - #5068 ## Description of the changes - Add unit tests for jaegerstorage ## How was this change tested? - ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Chirag Ghosh <cghosh828049@gmail.com> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
## Which problem is this PR solving? - part of #5068 ## Description of the changes - added missing unit test in cmd/jeager/internal and also added a goroutine check for that package ## How was this change tested? - go test ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` --------- Signed-off-by: Harshvir Potpose <hpotpose62@gmail.com> Signed-off-by: Harshvir Potpose <122517264+akagami-harsh@users.noreply.github.com> Signed-off-by: Yuri Shkuro <github@ysh.us> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com> Co-authored-by: Yuri Shkuro <github@ysh.us>
I'd like to work on |
- part of jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` --------- Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially Fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` --------- Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
I'm working on the |
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests for the `Run` function defined in the `internal/tracegen` package. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
Partially Addresses: jaegertracing#5068 - This commit adds tests for the `cmd/jaeger/internal/extension/jaegerquery` package. It brings the code coverage of the package to 84% - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
- Partially fixes jaegertracing#5068 - This commit adds tests tests in the `internal/tracegen` package, bringing the package coverage to 97%. - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
## Which problem is this PR solving? - Partially fixes #5068 ## Description of the changes - This commit adds tests for the `internal/tracegen` package. ## How was this change tested? - make test ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
Partially Addresses: jaegertracing#5068 - This commit adds tests for the `cmd/jaeger/internal/extension/jaegerquery` package. It brings the code coverage of the package to 84% - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
Partially Addresses: jaegertracing#5068 - This commit adds tests for the `cmd/jaeger/internal/extension/jaegerquery` package. It brings the code coverage of the package to 84% - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
Partially Addresses: jaegertracing#5068 - This commit adds tests for the `cmd/jaeger/internal/extension/jaegerquery` package. It brings the code coverage of the package to 84% - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
Partially Addresses: jaegertracing#5068 - This commit adds tests for the `cmd/jaeger/internal/extension/jaegerquery` package. It brings the code coverage of the package to 84% - make test - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com>
I will cover ./cmd/anonymizer/app/query/ @yurishkuro |
Hi @yurishkuro, Can I work on writing the test of cmd/jaeger/internal/extension/jaegerquery/? |
**Which problem is this PR solving?** This PR addresses a part of the issue [#5068](#5068) by adding tests for the `pkg/cassandra/gocql/testutils` package. **Description of the changes** The commit introduces tests for the `pkg/cassandra/gocql/testutils` package to ensure the correctness of the User Defined Type (UDT) functionality. These tests cover the `MarshalUDT` and `UnmarshalUDT` methods of the `CustomUDT` type. **How was this change tested?** The changes were tested by running the following command: ```bash make test ``` **Checklist** - [x] I have read [CONTRIBUTING_GUIDELINES.md](https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md) - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - `for jaeger: make lint test` - `for jaeger-ui: yarn lint` and `yarn test` --------- Signed-off-by: Wise-Wizard <saransh.shankar@gmail.com>
Hi @VaibhavMalik4187, do you plan on completing the PR regarding tests for JaegerQuery or I can go ahead and complete the remaining requirements? |
I'd like to work on |
Heey @yurishkuro, I can add tests too, what do u suggest is priority at the moment ? |
## Which problem is this PR solving? - Part of [#5068](#5068) ## Description of the changes - Added unit tests for `cmd/anonymizer/app/writer/` ## How was this change tested? - `go test -cover -v ./cmd/anonymizer/app/writer` - `make lint test` ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: Ashutosh Srivastava <ashutosh3002@gmail.com> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
## Which problem is this PR solving? - Part of #5068 ## Description of the changes - This commit adds tests for the `cmd/jaeger/internal/extension/jaegerquery` package. ## How was this change tested? - make test ## Checklist - [x] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [x] I have signed all commits - [x] I have added unit tests for the new functionality - [x] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` --------- Signed-off-by: VaibhavMalik4187 <vaibhavmalik2018@gmail.com> Signed-off-by: Yuri Shkuro <github@ysh.us> Signed-off-by: Yuri Shkuro <yurishkuro@users.noreply.github.com> Co-authored-by: Yuri Shkuro <github@ysh.us> Co-authored-by: Yuri Shkuro <yurishkuro@users.noreply.github.com>
Our CI enforces that all packages with .go files must have at least one _test.go file, to ensure the package is included in the code coverage, OR that it has a
.nocover
file when testing it is very difficult (e.g. requires external DB connection).However, we somewhat abused the
.nocover
capability and have a bunch of packages that could have tests with high coverage. This ticket is about incrementally fixing those, which is a good first issue since the changes can be reasonably small if scoped to one package.To see which packages need these tests, run
make nocover
.Regarding
./examples/
, where lots of.nocover
files are found today, I think we can replace them withempty_test.go
as we do in some other dirs, because examples will still be excluded from the overall coverage percentage, so there's no need to be excluding them via.nocover
mechanism (it just creates noise).The text was updated successfully, but these errors were encountered: