Correct bottleneck vault params. Remove retries in tests

.github/workflows/ci-dev-ops-local.yml

@@ -1,4 +1,4 @@
-name: ci-dev-ops
+name: ci-dev-ops-local
 
 defaults:
   run:

packages/ops/playwright.config.ts

@@ -10,7 +10,7 @@ export default defineConfig({
   /* Fail the build on CI if you accidentally left test.only in the source code. */
   forbidOnly: !!process.env.CI,
   /* Retry on CI only */
-  retries: process.env.CI ? 2 : 2,
+  retries: process.env.CI ? 1 : 1,
   /* Opt out of parallel tests on CI. */
   workers: process.env.CI ? 1 : 1,
   /* Reporter to use. See https://playwright.dev/docs/test-reporters */

packages/ops/test/create-vault.spec.ts

@@ -24,7 +24,19 @@ test.beforeAll(async () => {
   config = loadConfiguration();
 });
 
-const createVaultWithPause = limiter.wrap(createVault);
+const createVaultWithPause = limiter.wrap(
+  async (
+    config: any,
+    isMatured: boolean,
+    isUpside: boolean,
+    isTenant: boolean,
+    upsideVault?: string,
+    tenantEmail?: string,
+    override?: { treasuryAddress: string; activityRewardAddress: string; collateralPercentage: number },
+  ): Promise<any> => {
+    return createVault(config, isMatured, isUpside, isTenant, upsideVault, tenantEmail, override);
+  },
+);
 
 /*
   TODO - move param validations and tests to separate classes to simplfy createVault testing.
@@ -131,6 +143,7 @@ test.describe('Create Vault', async () => {
 
     test('a non-matured, ready, Fixed Yield vault, closed for deposits/redemption, Maturity Check OFF', async () => {
       const created = await createVaultWithPause(config, true, false, false);
+
       expect(created).toMatchObject({ type: 'fixed_yield', status: 'ready' });
       expect(isPast(created.deposits_opened_at)).toBe(true);
       expect(isPast(created.deposits_closed_at)).toBe(true);