From 162e6d9c25bb7688e8ddd262015b06f7e8ec1f8f Mon Sep 17 00:00:00 2001
From: Jean Mainguy <9969006+jhandguy@users.noreply.github.com>
Date: Tue, 3 Oct 2023 16:40:09 +0200
Subject: [PATCH] chore: remove sleeps in k6 load tests

---
 dynamo/k6/script.js  | 8 +-------
 gateway/k6/script.js | 8 +-------
 s3/k6/script.js      | 8 +-------
 sql/k6/script.js     | 5 ++---
 4 files changed, 5 insertions(+), 24 deletions(-)

diff --git a/dynamo/k6/script.js b/dynamo/k6/script.js
index 20b1f5b..c250146 100644
--- a/dynamo/k6/script.js
+++ b/dynamo/k6/script.js
@@ -1,5 +1,5 @@
 import grpc from 'k6/net/grpc';
-import {check, sleep} from 'k6';
+import {check} from 'k6';
 import {randomString, uuidv4} from 'https://jslib.k6.io/k6-utils/1.4.0/index.js';
 
 export const options = {
@@ -50,18 +50,12 @@ export default function () {
         'create message is valid': (r) => r.message.item.id === id && r.message.item.content === content,
     });
 
-    sleep(0.5);
-
     check(client.invoke('item.ItemService/GetItem', {id: id}, params), {
         'get status is OK': (r) => r.status === grpc.StatusOK,
         'get message is valid': (r) => r.message.item.id === id && r.message.item.content === content,
     });
 
-    sleep(0.5);
-
     check(client.invoke('item.ItemService/DeleteItem', {id: id}, params), {
         'delete status is OK': (r) => r.status === grpc.StatusOK,
     });
-
-    sleep(1);
 }
diff --git a/gateway/k6/script.js b/gateway/k6/script.js
index efd8a84..b0d758a 100644
--- a/gateway/k6/script.js
+++ b/gateway/k6/script.js
@@ -1,5 +1,5 @@
 import http from 'k6/http';
-import {check, sleep} from 'k6';
+import {check} from 'k6';
 import {randomString, uuidv4} from 'https://jslib.k6.io/k6-utils/1.4.0/index.js';
 
 export const options = {
@@ -44,18 +44,12 @@ export default function () {
         'post response body is valid': (r) => r.json().id === id && r.json().content === content,
     });
 
-    sleep(0.5);
-
     check(http.get(`${url}/message/${id}`, params), {
         'get response status is 200': (r) => r.status === 200,
         'get response body is valid': (r) => r.json().id === id && r.json().content === content,
     });
 
-    sleep(0.5);
-
     check(http.del(`${url}/message/${id}`, null, params), {
         'delete response status is 200': (r) => r.status === 200,
     });
-
-    sleep(1);
 }
diff --git a/s3/k6/script.js b/s3/k6/script.js
index b1ac700..1ebd2b1 100644
--- a/s3/k6/script.js
+++ b/s3/k6/script.js
@@ -1,5 +1,5 @@
 import grpc from 'k6/net/grpc';
-import {check, sleep} from 'k6';
+import {check} from 'k6';
 import {randomString, uuidv4} from 'https://jslib.k6.io/k6-utils/1.4.0/index.js';
 
 export const options = {
@@ -50,18 +50,12 @@ export default function () {
         'create message is valid': (r) => r.message.object.id === id && r.message.object.content === content,
     });
 
-    sleep(0.5);
-
     check(client.invoke('object.ObjectService/GetObject', {id: id}, params), {
         'get status is OK': (r) => r.status === grpc.StatusOK,
         'get message is valid': (r) => r.message.object.id === id && r.message.object.content === content,
     });
 
-    sleep(0.5);
-
     check(client.invoke('object.ObjectService/DeleteObject', {id: id}, params), {
         'delete status is OK': (r) => r.status === grpc.StatusOK,
     });
-
-    sleep(1);
 }
diff --git a/sql/k6/script.js b/sql/k6/script.js
index 3dae1c7..2bd6499 100644
--- a/sql/k6/script.js
+++ b/sql/k6/script.js
@@ -1,9 +1,10 @@
 import http from 'k6/http';
-import {check, sleep} from 'k6';
+import {check} from 'k6';
 import {randomItem, randomString, uuidv4} from 'https://jslib.k6.io/k6-utils/1.4.0/index.js';
 
 export const options = {
     setupTimeout: '3m',
+    rps: 30,
     scenarios: {
         load: {
             executor: 'ramping-arrival-rate',
@@ -63,8 +64,6 @@ export function setup() {
             });
 
             messages.push(message)
-
-            sleep(0.03);
         }
 
         users.push({