From 48eb16797c9547e4236fdd2ddeb28e1b901aaf08 Mon Sep 17 00:00:00 2001 From: wzshiming Date: Thu, 4 Feb 2021 18:10:55 +0800 Subject: [PATCH] Speed up pkg/volume/util/operationexecutor unit tests --- .../operation_executor_test.go | 22 +++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/pkg/volume/util/operationexecutor/operation_executor_test.go b/pkg/volume/util/operationexecutor/operation_executor_test.go index c94249b6d30..a65ec69b71f 100644 --- a/pkg/volume/util/operationexecutor/operation_executor_test.go +++ b/pkg/volume/util/operationexecutor/operation_executor_test.go @@ -77,6 +77,8 @@ func TestOperationExecutor_MountVolume_ConcurrentMountForNonAttachableAndNonDevi } func TestOperationExecutor_MountVolume_ConcurrentMountForAttachablePlugins(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToMount := make([]VolumeToMount, numVolumesToAttach) @@ -102,6 +104,8 @@ func TestOperationExecutor_MountVolume_ConcurrentMountForAttachablePlugins(t *te } func TestOperationExecutor_MountVolume_ConcurrentMountForDeviceMountablePlugins(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToMount := make([]VolumeToMount, numVolumesToAttach) @@ -161,6 +165,8 @@ func TestOperationExecutor_UnmountVolume_ConcurrentUnmountForAllPlugins(t *testi } func TestOperationExecutor_UnmountDeviceConcurrently(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() attachedVolumes := make([]AttachedVolume, numDevicesToUnmount) @@ -182,6 +188,8 @@ func TestOperationExecutor_UnmountDeviceConcurrently(t *testing.T) { } func TestOperationExecutor_AttachSingleNodeVolumeConcurrentlyToSameNode(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToAttach := make([]VolumeToAttach, numVolumesToAttach) @@ -210,6 +218,8 @@ func TestOperationExecutor_AttachSingleNodeVolumeConcurrentlyToSameNode(t *testi } func TestOperationExecutor_AttachMultiNodeVolumeConcurrentlyToSameNode(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToAttach := make([]VolumeToAttach, numVolumesToAttach) @@ -238,6 +248,8 @@ func TestOperationExecutor_AttachMultiNodeVolumeConcurrentlyToSameNode(t *testin } func TestOperationExecutor_AttachSingleNodeVolumeConcurrentlyToDifferentNodes(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToAttach := make([]VolumeToAttach, numVolumesToAttach) @@ -294,6 +306,8 @@ func TestOperationExecutor_AttachMultiNodeVolumeConcurrentlyToDifferentNodes(t * } func TestOperationExecutor_DetachSingleNodeVolumeConcurrentlyFromSameNode(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() attachedVolumes := make([]AttachedVolume, numVolumesToDetach) @@ -322,6 +336,8 @@ func TestOperationExecutor_DetachSingleNodeVolumeConcurrentlyFromSameNode(t *tes } func TestOperationExecutor_DetachMultiNodeVolumeConcurrentlyFromSameNode(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() attachedVolumes := make([]AttachedVolume, numVolumesToDetach) @@ -411,6 +427,8 @@ func TestOperationExecutor_VerifyVolumesAreAttachedConcurrentlyOnDifferentNodes( } func TestOperationExecutor_VerifyControllerAttachedVolumeConcurrently(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToMount := make([]VolumeToMount, numVolumesToVerifyControllerAttached) @@ -460,6 +478,8 @@ func TestOperationExecutor_MountVolume_ConcurrentMountForNonAttachablePlugins_Vo } func TestOperationExecutor_MountVolume_ConcurrentMountForAttachablePlugins_VolumeMode_Block(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() volumesToMount := make([]VolumeToMount, numVolumesToAttach) @@ -527,6 +547,8 @@ func TestOperationExecutor_UnmountVolume_ConcurrentUnmountForAllPlugins_VolumeMo } func TestOperationExecutor_UnmountDeviceConcurrently_VolumeMode_Block(t *testing.T) { + t.Parallel() + // Arrange ch, quit, oe := setup() attachedVolumes := make([]AttachedVolume, numDevicesToUnmap)