resolve some todos

src/createXcmTypes/SystemToPara.spec.ts

@@ -287,7 +287,6 @@ describe('SystemToPara XcmVersioned Generation', () => {
 
 	describe('createSystemToParaMultiAssets', () => {
 		it('Should correctly create system multi assets for SystemToPara xcm direction', async () => {
-			// TODO: This is unsorted, and once the sorting is fixed this test should be switch to ordered assets
 			const expected: FungibleStrMultiAsset[] = [
 				{
 					fun: {

src/createXcmTypes/util/foreignAssetMultiLocationIsInCacheOrRegistry.ts

@@ -35,10 +35,7 @@ const checkForeignAssetExists = (
 		});
 
 		for (const asset of foreignAssets) {
-			// TODO: How to ensure 2 foreign assets are exactly the same. Best
-			// to sanitize both then compare. Also what if some value is of a `parents: number` and another
-			// is of `parents: string`...
-
+			// We sanitize `assets` here since multiLocation is already sanitized.
 			if (JSON.stringify(sanitizeKeys(JSON.parse(asset))) === JSON.stringify(multiLocation)) {
 				return true;
 			}

src/createXcmTypes/util/sortMultiAssetsAscending.spec.ts

@@ -278,7 +278,6 @@ describe('sortMultiAssetsAscending', () => {
 	});
 
 	it('Should correctly sort an unsorted multiasset array with in ascending order when parents values are different', () => {
-		// TODO: verify if the refactor of this test is correct.
 		const multiAssets: FungibleStrMultiAsset[] = [
 			{
 				fun: {