fix(tests): expect proxy config keys as injected in connector-migration-test

config/test.json configures http_proxy, https_proxy and no_proxy, so
injectProxyConfiguration always adds HTTP_PROXY, HTTPS_PROXY, NO_PROXY, and
HTTPS_PROXY_REJECT_UNAUTHORIZED (always injected regardless of config) to
manager_contract_configuration in this test environment. The previous fix
incorrectly assumed these were only injected when configured and therefore
absent, causing the assertion on HTTP_PROXY to fail.

Move these keys from EXCLUDED_RUNTIME_KEYS to INJECTED_KEYS, matching actual
runtime behavior confirmed by running the integration test.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: fellowseb

opencti-platform/opencti-graphql/tests/03-integration/02-resolvers/connector-migration-test.ts

@@ -197,17 +197,19 @@ describe('Check connector migration', () => {
         const rawConfig = managedConnector.manager_contract_configuration;
 
         // ManagedConnector.manager_contract_configuration injects these keys dynamically at read time
-        // (see computeManagerConnectorConfiguration), they are not part of the persisted/schema-driven config.
-        const INJECTED_KEYS = ['CONNECTOR_ID', 'CONNECTOR_NAME', 'CONNECTOR_TYPE', 'OPENCTI_TOKEN'];
+        // (see computeManagerConnectorConfiguration / injectProxyConfiguration), they are not part of the
+        // persisted/schema-driven config. Proxy vars are injected because config/test.json configures
+        // http_proxy, https_proxy and no_proxy, and https_proxy_reject_unauthorized is always injected.
+        const INJECTED_KEYS = ['CONNECTOR_ID', 'CONNECTOR_NAME', 'CONNECTOR_TYPE', 'OPENCTI_TOKEN', 'HTTP_PROXY', 'HTTPS_PROXY', 'NO_PROXY', 'HTTPS_PROXY_REJECT_UNAUTHORIZED'];
         INJECTED_KEYS.forEach((injectedKey) => {
           const found = rawConfig.find((c: { key: string }) => c.key === injectedKey);
           expect(found).toBeDefined();
         });
 
         // These runtime keys are never persisted/returned in manager_contract_configuration:
         // OPENCTI_URL and CONNECTOR_RUN_AND_TERMINATE are excluded from the contract's config
-        // (same exclusion as catalog-domain), and proxy vars are only injected when configured.
-        const EXCLUDED_RUNTIME_KEYS = ['OPENCTI_URL', 'CONNECTOR_RUN_AND_TERMINATE', 'HTTP_PROXY', 'HTTPS_PROXY', 'NO_PROXY', 'HTTPS_PROXY_REJECT_UNAUTHORIZED'];
+        // (same exclusion as catalog-domain) and never re-injected by this resolver.
+        const EXCLUDED_RUNTIME_KEYS = ['OPENCTI_URL', 'CONNECTOR_RUN_AND_TERMINATE'];
         EXCLUDED_RUNTIME_KEYS.forEach((runtimeKey) => {
           const found = rawConfig.find((c: { key: string }) => c.key === runtimeKey);
           expect(found).toBeUndefined();