Skip to content

Conversation

stalek71
Copy link
Contributor

@stalek71 stalek71 commented Feb 6, 2019

PNCounter

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

namespace Hazelcast.Client.Test
{
[TestFixture]
public class ClientPNCounterTest : MultiMemberBaseTest
Copy link
Contributor

@asimarslan asimarslan Feb 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need a clean up here :

[TestFixture]
    public class ClientPNCounterTest : SingleMemberBaseTest
    {
        private ClientPNCounterProxy _pnCounter;

        [SetUp]
        public void Setup()
        {
            _pnCounter = Client.GetPNCounter(TestSupport.RandomString()) as ClientPNCounterProxy;
        }

        [TearDown]
        public void TearDown()
        {
        }

        [Test]
        public void Reset_Succeeded()
        {
            _pnCounter.Reset();
        }

...

We still need this

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.


namespace Hazelcast.Client.Test
{
public class VectorClockTest : SingleMemberBaseTest
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing [TestFixture]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

}

[TearDown]
public static void Destroy()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

older Nunit(pre 2.5) mandate static tear-down method but it's not required anymore. Non-static one would be better in new implementations

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

public class VectorClockTest : SingleMemberBaseTest
{
internal static VectorClock _inst;
internal const string name = "ClientPNCounterTest";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dead code

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

{
public class VectorClockTest : SingleMemberBaseTest
{
internal static VectorClock _inst;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can be private non-static with Teardown method change

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

<ItemGroup>
<EmbeddedResource Include="Resources\*" />
</ItemGroup>
<ItemGroup>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need these as above <EmbeddedResource Include="Resources\*" /> adds everthing

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@asimarslan asimarslan requested a review from ihsandemir February 7, 2019 12:51
@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests FAILed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@stalek71
Copy link
Contributor Author

stalek71 commented Feb 8, 2019

Test failed on timeout. The test run from Visual Studio passed conditions without any problems

@stalek71
Copy link
Contributor Author

stalek71 commented Feb 8, 2019

verify

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows Net Core Tests PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows .Net Framework Tests PASSed.

@stalek71 stalek71 merged commit e9bc514 into hazelcast:master Feb 12, 2019
@asimarslan asimarslan added this to the 3.11 milestone Mar 8, 2019
@stalek71 stalek71 deleted the PNCounterPR branch March 14, 2019 14:36
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.

4 participants