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

Failing test: BrokerTests Broker_Send_Many #896

Open
roller opened this issue Apr 11, 2019 · 0 comments · May be fixed by #897
Open

Failing test: BrokerTests Broker_Send_Many #896

roller opened this issue Apr 11, 2019 · 0 comments · May be fixed by #897

Comments

@roller
Copy link

roller commented Apr 11, 2019

What version of PushSharp are you using?

master f68bb1e

Describe your issue:

  1. Test Failure : PushSharp.Tests.BrokerTests.Broker_Send_Many
    Expected: 1000
    But was: 976

What are the steps required to reproduce this issue?

Running with an older nunit, I'm not sure that matters.

NUnit version 2.5.10.11092
Copyright (C) 2002-2009 Charlie Poole.
Copyright (C) 2002-2004 James W. Newkirk, Michael C. Two, Alexei A. Vorontsov.
Copyright (C) 2000-2002 Philip Craig.
All Rights Reserved.

Runtime Environment -
OS Version: Microsoft Windows NT 6.2.9200.0
CLR Version: 2.0.50727.8825 ( Net 2.0 )
... snip ...

roller added a commit to roller/PushSharp that referenced this issue Apr 11, 2019
The test appears to only run a single worker, but the increment count
values themselves still may need synchronization with the original test
thread.

Fixes Redth#896
@roller roller linked a pull request Apr 11, 2019 that will close this issue
roller pushed a commit to RJOBrien/PushSharp that referenced this issue Apr 26, 2019
The test appears to only run a single worker, but the increment count
values themselves still may need synchronization with the original test
thread.

Fixes Redth#896
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 a pull request may close this issue.

1 participant