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

MQTT test case fails randomly #347

Closed
localvar opened this issue Nov 2, 2021 · 0 comments · Fixed by #353
Closed

MQTT test case fails randomly #347

localvar opened this issue Nov 2, 2021 · 0 comments · Fixed by #353
Assignees
@samutamm
Labels
bug Something isn't working

Comments

@localvar
Copy link
Collaborator

localvar commented Nov 2, 2021

Describe the bug
MQTT test case fails randomly

To Reproduce

  1. switch to directory pkg/object/mqttproxy
  2. execute go test for several times
  3. test cases failed randomly
@localvar localvar added the bug Something isn't working label Nov 2, 2021
samutamm added a commit that referenced this issue Nov 4, 2021
@samutamm
remove goroutine from a test (close #347)
cdb269c
samutamm pushed a commit that referenced this issue Nov 4, 2021
remove goroutine from a test (close #347)
9d26982
@samutamm samutamm mentioned this issue Nov 4, 2021
Merged
@localvar localvar closed this as completed Nov 6, 2021
localvar added a commit that referenced this issue Nov 8, 2021
@samutamm @localvar @suchen-sci
remove goroutine from a test (close #347) (#353)
1afeb8f 
* remove goroutine from a test (close #347)

* modify http.Server.start() in mqtt_test to wait until ready

* Update pkg/object/mqttproxy/mqtt_test.go

Co-authored-by: Bomin Zhang <[email protected]>

* Fix typo in url address

Co-authored-by: SU Chen <[email protected]>

* fix formatting

Co-authored-by: Bomin Zhang <[email protected]>
Co-authored-by: SU Chen <[email protected]>
zhao-kun pushed a commit that referenced this issue Nov 8, 2021
@samutamm @localvar
@suchen-sci @zhao-kun
remove goroutine from a test (close #347) (#353)
ff8d391 
* remove goroutine from a test (close #347)

* modify http.Server.start() in mqtt_test to wait until ready

* Update pkg/object/mqttproxy/mqtt_test.go

Co-authored-by: Bomin Zhang <[email protected]>

* Fix typo in url address

Co-authored-by: SU Chen <[email protected]>

* fix formatting

Co-authored-by: Bomin Zhang <[email protected]>
Co-authored-by: SU Chen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samutamm samutamm

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove goroutine from a test (close #347) easegress-io/easegress
2 participants
@localvar @samutamm