Remove asSet as it appears to no longer be working as expected. asLis…

…t can be passed to a set constructor instead.
ipavlic · Jul 5, 2024 · 0e584de · 0e584de
1 parent 2624285
commit 0e584de
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/force-app/main/default/classes/collection/SObjectCollection.cls b/force-app/main/default/classes/collection/SObjectCollection.cls
@@ -29,16 +29,6 @@ public with sharing class SObjectCollection {
 		return typedList;
 	}
 
-	public Set<SObject> asSet() {
-		return new Set<SObject>(this.records);
-	}
-
-	public Set<SObject> asSet(Type setType) {
-		Set<SObject> typedSet = (Set<SObject>) setType.newInstance();
-		typedSet.addAll(this.records);
-		return typedSet;
-	}
-
 	public Map<Id, SObject> asMap() {
 		return new Map<Id, SObject>(this.records);
 	}

diff --git a/force-app/main/default/classes/test/classes/collection/SObjectCollectionTest.cls b/force-app/main/default/classes/test/classes/collection/SObjectCollectionTest.cls
@@ -16,6 +16,7 @@ private class SObjectCollectionTest {
 	@IsTest
 	private static void collectionCanBeInstantiatedFromAList() {
 		SObjectCollection c = SObjectCollection.of(new List<Account>{new Account(Name = 'Foo'), new Account(Name = 'Bar')});
+		System.Assert.isNotNull(c);
 	}
 
 	@IsTest