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

Add cross-platform spec test to MNTR #2919

Merged
merged 71 commits into from
Aug 4, 2017
Merged

Add cross-platform spec test to MNTR #2919

merged 71 commits into from
Aug 4, 2017

Conversation

Arkatufus
Copy link
Contributor

No description provided.

Arkatufus and others added 30 commits July 6, 2017 12:06
@Arkatufus
Port SocketChannel for core. Add TAP to APM async model converter ext…
b2b022a 
…ension
@Arkatufus
Convert Net45 code convention to Core (mainly Assembly related)
0306ac6
@Arkatufus
Add missing methods and classes that are no longer available in Core
ab52532
@Arkatufus
Fix run shutdown spec so that it runs properly
e85bb90
@Arkatufus
Remove Faker dependency by moving all related methods
41220b8
@Arkatufus
Add core frameworks to projects csproj that are required by MultiNode…
4f42a03 
…TestRunner.Shared
@Arkatufus
Add AKKAIO constant to Akka because it is needed by NodeTestRunner
8da5406
@Arkatufus
Port SocketChannel for core. Add TAP to APM async model converter ext…
9f1154d 
…ension
@Arkatufus
Convert Net45 code convention to Core (mainly Assembly related)
d7d0415
@Arkatufus
Add missing methods and classes that are no longer available in Core
47b02cf
@Arkatufus
Fix run shutdown spec so that it runs properly
a281741
@Arkatufus
Remove Faker dependency by moving all related methods
5c83351
@Arkatufus
Add core frameworks to projects csproj that are required by MultiNode…
48ee5fc 
…TestRunner.Shared
@Arkatufus
Add AKKAIO constant to Akka because it is needed by NodeTestRunner
a90d3d5
@Arkatufus
Change buildIncremental.fsx to include Akka.MultiNodeTestRunner.Share…
d875d3c 
…d.Tests
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
338f38e
@Arkatufus
Combine compiler conditions that can be merged for both platform
d921313
@Arkatufus
TestCaseDiscoveryMessage.TestAssembly.Assembly have different behavio…
acae241 
…ur between Net452 and NetCore1.1
@Arkatufus
Turn on Akka.Remote.Testkit.Tests NetCore build because Akka.Remote.T…
7716940 
…estKit NetCore build.
@Arkatufus
Remove code optimization, not needed in tests
9eaf62a
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
7799f1d
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
9aeb38b
@Arkatufus
Move SpecPass, SpecFail, and Discovery class to Akka.MultiNodeTestRun…
fdfe655 
…ner.Shared
@Arkatufus
Remove Akka.MultiNodeTestRunner and Akka.NodeTestRunner reference fro…
d5eb8c5 
…m Akka.MultiNodeTestRunner.Shared.Test
@Arkatufus
Rollback unnescessary NetCore implementation
87e3ccc
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
edc0795
@Danthar
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
25c9862
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
ffabcfa
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
f01744a
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
826558a
Arkatufus and others added 10 commits July 29, 2017 17:51
@Arkatufus
Fix erroneous json and html target file name.
20eddde
@Arkatufus
Change Discovery.Tests to List<List<NodeTest>> because of possible ke…
1f37baf 
…y collision and key was never used anywhere.
@Arkatufus
Change sharding multinode test structure to follow proper test file c…
30e3689 
…onfiguration
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
68d770d
@heynickc
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
c09f019
@Arkatufus
Revert MNTR.Discovery.Tests back to dictionary, needed for unit test
6dee7d4
@Arkatufus
Add initial cross-platform spec support
b8c948b 
- Change build script for Net Core MNTR, copy node runner net and net core binaries to mntr/runner
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
92437da
@Arkatufus
Revert "Add initial cross-platform spec support"
1114d26 
This reverts commit b8c948b.
@Arkatufus
Initial commit
d9d7704
@Arkatufus
Copy link
Contributor Author

Very simple cross platform spec test for multi node test runner.
All it does is alternate the node test runner binary used to run each node role in each spec between Net and Net Core binaries

@Arkatufus
Copy link
Contributor Author

Note that cross platform test support is only available on Net Core build.

Arkatufus and others added 12 commits August 3, 2017 14:02
@Arkatufus
Merge branch 'v1.3' into Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828
f7ea438
@Arkatufus
Merge branch 'v1.3' into MNTR_cross-platform_spec
a598537
@heynickc
20 second wait for Controller actor terminate
e50e68c
@Arkatufus
Merge pull request #1 from heynickc/arkatafus-controllerspec
a6d2931 
20 second wait for Controller actor Terminate
@Arkatufus
Merge remote-tracking branch 'upstream/v1.3' into MNTR_cross-platform…
f2a3515 
…_spec

# Conflicts:
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingFailureSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingGracefulShutdownSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingLeavingSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingSpec.cs
@Arkatufus
Merge remote-tracking branch 'upstream/v1.3' into Fix_Akka.MultiTestN…
7c5751f 
…odeRunner.Shared.Tests_#2828

# Conflicts:
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingFailureSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingGracefulShutdownSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingLeavingSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingSpec.cs
@Arkatufus
Finalize 1.3 merging
84dfd8c 
- Change Cluster.Sharding.Tests.Multinode to be compatible with the new MNTR
- Remove Net Core build target on Cluster.Sharding.Tests.Multinode
@Arkatufus
Merge branch 'Fix_Akka.MultiTestNodeRunner.Shared.Tests_#2828' into M…
3a495cd 
…NTR_cross-platform_spec

# Conflicts:
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingFailureSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingGracefulShutdownSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingLeavingSpec.cs
#	src/contrib/cluster/Akka.Cluster.Sharding.Tests.MultiNode/ClusterShardingSpec.cs
@Arkatufus
Merge branch 'v1.3' into MNTR_cross-platform_spec
96f354b
@Arkatufus
Merge branch 'v1.3' into MNTR_cross-platform_spec
dd3cb70
@Arkatufus
Merge branch 'v1.3' into MNTR_cross-platform_spec
631d199 
# Conflicts:
#	src/core/Akka.MultiNodeTestRunner/Program.cs
#	src/core/Akka.NodeTestRunner/Program.cs
@Aaronontheweb
Merge branch 'v1.3' into MNTR_cross-platform_spec
74174dc
@Aaronontheweb
Copy link
Member

@Arkatufus just so I understand correctly, all this PR provides right now are some switches for enabling X-plat multi-node tests. Not any actual x-plat multi-node tests themselves, correct?

@Aaronontheweb
Copy link
Member

Derp, nevermind:

Very simple cross platform spec test for multi node test runner.
All it does is alternate the node test runner binary used to run each node role in each spec between Net and Net Core binaries

@Aaronontheweb Aaronontheweb merged commit 8c2691d into akkadotnet:v1.3 Aug 4, 2017
@Aaronontheweb Aaronontheweb added this to the 1.3.0 milestone Aug 4, 2017
@Arkatufus Arkatufus deleted the MNTR_cross-platform_spec branch April 22, 2022 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.0
Development

Successfully merging this pull request may close these issues.
4 participants
@Arkatufus @Aaronontheweb @Danthar @heynickc