test: hard-fail Sepolia stop-loss test when prereqs missing

Revert the t.Skipf change from the previous commit and additionally
convert the existing OWNER_EOA t.Skip to a require.True. Integration
tests that cannot run because env vars or wallet funding are missing
should fail loudly so the gap gets fixed, not be silently skipped.

The testing.Short() guard at the top stays - that is a deliberate
fast-mode opt-out, not a missing-prereq skip.

Author: will-dz

core/taskengine/simulate_uniswap_workflow_test.go

@@ -45,9 +45,7 @@ func TestSimulateTask_StopLossWorkflow_Sepolia(t *testing.T) {
 	}
 
 	ownerAddr, ok := testutil.MustGetTestOwnerAddress()
-	if !ok {
-		t.Skip("OWNER_EOA not set; skipping")
-	}
+	require.True(t, ok, "OWNER_EOA must be set to run this Sepolia integration test")
 	ownerAddress := *ownerAddr
 
 	cfg, err := config.NewConfig(testutil.GetConfigPath(testutil.DefaultConfigPath))
@@ -67,9 +65,8 @@ func TestSimulateTask_StopLossWorkflow_Sepolia(t *testing.T) {
 
 	usdcBalance := fetchERC20Balance(t, client, SEPOLIA_USDC, smartWalletAddr.Hex())
 	t.Logf("USDC balance: %s (6 decimals)", usdcBalance.String())
-	if usdcBalance.Cmp(big.NewInt(2_000_000)) < 0 {
-		t.Skipf("smart wallet %s must hold at least 2 USDC to run this test", smartWalletAddr.Hex())
-	}
+	require.True(t, usdcBalance.Cmp(big.NewInt(2_000_000)) >= 0,
+		"wallet must hold at least 2 USDC; fund %s", smartWalletAddr.Hex())
 
 	db := testutil.TestMustDB()
 	t.Cleanup(func() { storage.Destroy(db.(*storage.BadgerStorage)) })