test(assets): describe the pinned native addresses and derived-address cases plainly

The comment above the pinned native SAC addresses referred to an internal
task-plan step and to hardcoding that no longer exists at this commit. Say
why the values are pinned instead. Rename the derived-address cases to state
what they check rather than what preceded them.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Author: CassioMG

@shared/helpers/__tests__/assetIdentity.test.ts

@@ -21,9 +21,9 @@ import {
 const XLM_CODED_ISSUER =
   "GA5ZSEJYB37JRC5AVCIA5MOP4RHTM335X2KGX3IHOJAPP5RE34K4KZVN";
 
-// The published native SAC addresses. Pinned rather than re-derived, so these
-// tests also assert that deriving from the passphrase reproduces the values
-// getNativeContractDetails hardcodes before Task 8 removes them.
+// The published native SAC addresses, pinned rather than re-derived, so these
+// tests assert that deriving from the passphrase reproduces the known-good
+// values rather than just agreeing with themselves.
 const NATIVE_SAC_PUBLIC =
   "CAS3J7GYLGXMF6TDJBBYYSE3HQ6BBSMLNUQ34T6TZMYMW2EVH34XOWMA";
 const NATIVE_SAC_TESTNET =
@@ -130,7 +130,7 @@ describe("getNativeContractId", () => {
     expect(getNativeContractId(Networks.TESTNET)).toBe(NATIVE_SAC_TESTNET);
   });
 
-  it("returns a real address on a network the old lookup table omitted", () => {
+  it("derives a distinct address on FUTURENET", () => {
     const futurenet = getNativeContractId(Networks.FUTURENET);
     expect(futurenet).toMatch(/^C[A-Z2-7]{55}$/);
     expect(futurenet).not.toBe(NATIVE_SAC_PUBLIC);

extension/src/popup/helpers/__tests__/searchAsset.test.js

@@ -81,10 +81,9 @@ describe("searchAsset", () => {
       org: "",
     });
   });
-  it("should derive getNativeContractDetails for a network the table doesn't cover", () => {
-    // The contract address is now derived from the passphrase rather than
-    // read from a table, so a network outside PUBLIC/TESTNET gets a real
-    // address instead of the empty string the table used to fall back to.
+  it("derives the native contract address on FUTURENET", () => {
+    // The contract address is derived from the network passphrase, so every
+    // network gets a real address.
     expect(
       SearchAsset.getNativeContractDetails({
         network: "FUTURENET",

extension/src/popup/helpers/__tests__/searchAsset.test.ts

@@ -57,7 +57,7 @@ describe("getNativeContractDetails", () => {
     ).toBe("CAS3J7GYLGXMF6TDJBBYYSE3HQ6BBSMLNUQ34T6TZMYMW2EVH34XOWMA");
   });
 
-  it("returns a contract address on a network the table omitted", () => {
+  it("returns a derived contract address on FUTURENET", () => {
     expect(
       getNativeContractDetails({
         network: "FUTURENET",

extension/src/popup/helpers/__tests__/soroban.test.ts

@@ -9,7 +9,7 @@ const futurenetDetails = {
 } as NetworkDetails;
 
 describe("isAssetSac", () => {
-  it("recognises the native contract on a network the old table omitted", () => {
+  it("recognises the native contract on FUTURENET", () => {
     expect(
       isAssetSac({
         asset: {