From 2ac0ee8785c4983472660b905705f2cd51ec3581 Mon Sep 17 00:00:00 2001
From: Jake <jake.kennedy@guardian.co.uk>
Date: Wed, 4 Sep 2024 10:04:19 +0100
Subject: [PATCH] Remove `optimiseSpacefinderInline` test and set changes live
 (#1572)

---
 .changeset/dry-pumpkins-brake.md              |  5 ++++
 src/experiments/ab-tests.ts                   |  2 --
 .../tests/optimise-spacefinder-inline.ts      | 28 -------------------
 src/insert/spacefinder/rules.ts               | 15 +++-------
 4 files changed, 9 insertions(+), 41 deletions(-)
 create mode 100644 .changeset/dry-pumpkins-brake.md
 delete mode 100644 src/experiments/tests/optimise-spacefinder-inline.ts

diff --git a/.changeset/dry-pumpkins-brake.md b/.changeset/dry-pumpkins-brake.md
new file mode 100644
index 000000000..127967536
--- /dev/null
+++ b/.changeset/dry-pumpkins-brake.md
@@ -0,0 +1,5 @@
+---
+'@guardian/commercial': minor
+---
+
+Remove `optimiseSpacefinderInline` test and set changes live
diff --git a/src/experiments/ab-tests.ts b/src/experiments/ab-tests.ts
index 1d7f81244..6ea72ede6 100644
--- a/src/experiments/ab-tests.ts
+++ b/src/experiments/ab-tests.ts
@@ -1,7 +1,6 @@
 import type { ABTest } from '@guardian/ab-core';
 import { mpuWhenNoEpic } from './tests/mpu-when-no-epic';
 import { optOutFrequencyCap } from './tests/opt-out-frequency-cap';
-import { optimiseSpacefinderInline } from './tests/optimise-spacefinder-inline';
 
 /**
  * You only need to add tests to this file if the code you are testing is here in
@@ -12,5 +11,4 @@ export const concurrentTests: ABTest[] = [
 	// one test per line
 	mpuWhenNoEpic,
 	optOutFrequencyCap,
-	optimiseSpacefinderInline,
 ];
diff --git a/src/experiments/tests/optimise-spacefinder-inline.ts b/src/experiments/tests/optimise-spacefinder-inline.ts
deleted file mode 100644
index 7d463cbb5..000000000
--- a/src/experiments/tests/optimise-spacefinder-inline.ts
+++ /dev/null
@@ -1,28 +0,0 @@
-import type { ABTest } from '@guardian/ab-core';
-
-export const optimiseSpacefinderInline: ABTest = {
-	id: 'OptimiseSpacefinderInline',
-	author: '@commercial-dev',
-	start: '2024-08-08',
-	expiry: '2024-09-13',
-	audience: 5 / 100,
-	audienceOffset: 0 / 100,
-	audienceCriteria: '',
-	successMeasure: '',
-	description: 'Test new spacefinder rules for inline1 ads on desktop.',
-	variants: [
-		{
-			id: 'control',
-			test: (): void => {
-				/* no-op */
-			},
-		},
-		{
-			id: 'variant',
-			test: (): void => {
-				/* no-op */
-			},
-		},
-	],
-	canRun: () => true,
-};
diff --git a/src/insert/spacefinder/rules.ts b/src/insert/spacefinder/rules.ts
index f49160a64..da544bd68 100644
--- a/src/insert/spacefinder/rules.ts
+++ b/src/insert/spacefinder/rules.ts
@@ -1,7 +1,5 @@
 import { adSizes } from 'core';
 import { adSlotContainerClass } from 'core/create-ad-slot';
-import { isUserInVariant } from 'experiments/ab';
-import { optimiseSpacefinderInline } from 'experiments/tests/optimise-spacefinder-inline';
 import type { OpponentSelectorRules, SpacefinderRules } from './spacefinder';
 import { isInHighValueSection } from './utils';
 
@@ -43,11 +41,6 @@ const inlineOpponentSelector = ['inline', 'supporting', 'showcase']
 
 const headingSelector = `:scope > h2, [data-spacefinder-role="nested"] > h2, :scope > h3, [data-spacefinder-role="nested"] > h3`;
 
-const isInInlineSpacefinderOptimisationTest = isUserInVariant(
-	optimiseSpacefinderInline,
-	'variant',
-);
-
 const desktopInline1: SpacefinderRules = {
 	bodySelector,
 	candidateSelector,
@@ -65,15 +58,15 @@ const desktopInline1: SpacefinderRules = {
 		},
 		[inlineOpponentSelector]: {
 			marginBottom: 35,
-			marginTop: isInInlineSpacefinderOptimisationTest ? 200 : 400,
+			marginTop: 200,
 		},
 		[leftColumnOpponentSelector]: {
-			marginBottom: isInInlineSpacefinderOptimisationTest ? 50 : 35,
-			marginTop: isInInlineSpacefinderOptimisationTest ? 100 : 400,
+			marginBottom: 50,
+			marginTop: 100,
 		},
 		[rightColumnOpponentSelector]: {
 			marginBottom: 0,
-			marginTop: isInInlineSpacefinderOptimisationTest ? 150 : 600,
+			marginTop: 150,
 		},
 		['[data-spacefinder-role="supporting"]']: {
 			marginBottom: 0,