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

Fix mqtt tests using mock Client and mock Token #791

Merged
merged 6 commits into from
Jul 5, 2024

Conversation

vipul-rawat
Copy link
Member

fixes #790

@aryanmehrotra aryanmehrotra merged commit 63ebe9e into development Jul 5, 2024
11 checks passed
@aryanmehrotra aryanmehrotra deleted the fix-mqtt-tests branch July 5, 2024 07:36
This was referenced Jul 8, 2024
aryanmehrotra added a commit that referenced this pull request Jul 8, 2024
* Add support for profilling endpoints (#774)

* Bump cloud.google.com/go/pubsub from 1.39.0 to 1.40.0 (#775)

* Bump google.golang.org/api from 0.186.0 to 0.187.0 (#776)

* Fix MQTT implementation issue  (#771)

* add request timeout for health check calls (#777)

* add request timeout for health check calls,
add timeout field for custom health config

* use context without cancel and fix typos

* fix panic due to context cancel and fix inaffectual ctx assignment

* fix dependent integration test in example (#784)

Co-authored-by: Srijan Rastogi <44723623+srijan-27@users.noreply.github.com>

* Enhance Mongo Driver (#779)

* Add support for retry of failing downstream calls (#778)

* Fix mqtt tests using mock Client and mock Token (#791)

* improve ci (#788)

* ci: bump golangci-lint to latest version

fix .golangci.yml to use the non-deprecated identifiers for:
- err113 (instead of err113)
- mnd (instead of gomnd)

Also, fix //nolint because of these renamings

* chore: fix golangci-lint errors

With go 1.22, min and max are now built-in commands.

gocritic reports variable shadowing them.

* ci: enable mirror linter and fix reported errors

* ci: enable errorllint and fix reported errors

* ci: enable thelper and fix reported errors

* ci: enable usestdlibvars and fix reported errors

* ci: enable revive.bool-literal-in-expr and fix reported errors

* ci: enable revive.comment-spacing and fix reported errors

* ci: enable revive.bare-return and fix reported errors

* ci: enable revive.early-return and fix reported errors

* ci: enable revive.defer and fix reported errors

* ci: enable recent gci linter options and fix reported errors (#798)

* ci: improve dependabot (#792)

* Bump paambaati/codeclimate-action from 5.0.0 to 8.0.0 (#804)

* Bump go.opentelemetry.io/otel from 1.27.0 to 1.28.0 (#813)

* Bump docker/login-action from 1 to 3 (#802)

* Bump go.opentelemetry.io/otel/sdk from 1.27.0 to 1.28.0 (#805)

* Bump actions/setup-go from 4 to 5 (#806)

* Bump go.opentelemetry.io/otel/exporters/zipkin from 1.27.0 to 1.28.0 (#814)

* Bump github.com/XSAM/otelsql from 0.31.0 to 0.32.0 (#810)

* Bump google.golang.org/grpc from 1.64.0 to 1.65.0 (#808)

* Bump go.opentelemetry.io/otel/exporters/prometheus from 0.49.0 to 0.50.0 (#816)

* Bump docker/setup-buildx-action from 2 to 3 (#812)

* Bump golang.org/x/term from 0.21.0 to 0.22.0 (#817)

* Bump docker/build-push-action from 4 to 6 (#811)

* Bump google-github-actions/auth from 1 to 2 (#807)

* Add external DB healthcheck to healthHandler (#780)

* ci: enable ls-lint via GHA and fix reported errors (#799)

* Bump actions/download-artifact from 3 to 4 (#809)

* Update version.go
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.

Mqtt tests in pipeline fails ofter due to disconnection with public mqtt broker
3 participants