test: retain service on next schema if url if not in input (#6697)

Author: jdolle

integration-tests/tests/cli/__snapshots__/schema.spec.ts.snap

@@ -84,7 +84,13 @@ stdout--------------------------------------------:
 ℹ Available at http://__URL__
 `;
 
-exports[`FEDERATION > can update a service without providing a url if previously published > schema publish same url 1`] = ``;
+exports[`FEDERATION > can update a service without providing a url if previously published > schema publish same url 1`] = `
+:::::::::::::::: CLI SUCCESS OUTPUT :::::::::::::::::
+
+stdout--------------------------------------------:
+✔ Schema published
+ℹ Available at http://__URL__
+`;
 
 exports[`FEDERATION > can update a service without providing a url if previously published > schemaPublish (initial) 1`] = `
 :::::::::::::::: CLI SUCCESS OUTPUT :::::::::::::::::

integration-tests/tests/cli/schema.spec.ts

@@ -2,6 +2,7 @@
 import { createHash } from 'node:crypto';
 import { ProjectType } from 'testkit/gql/graphql';
 import * as GraphQLSchema from 'testkit/gql/graphql';
+import type { CompositeSchema } from '@hive/api/__generated__/types';
 import { createCLI, schemaCheck, schemaPublish } from '../../testkit/cli';
 import { cliOutputSnapshotSerializer } from '../../testkit/cli-snapshot-serializer';
 import { initSeed } from '../../testkit/seed';
@@ -423,25 +424,45 @@ describe.each([ProjectType.Stitching, ProjectType.Federation, ProjectType.Single
             }),
           ).resolves.toMatchSnapshot('schema publish initial');
 
+          const sdl2 = /* GraphQL */ `
+            type Query {
+              users: [User!]
+            }
+
+            type User {
+              id: ID!
+              name: String!
+              email: String!
+              phone: String
+            }
+          `;
+
           await expect(
             cli.publish({
-              sdl,
+              sdl: sdl2,
               commit: 'push2',
               serviceName,
-              expect: 'ignored',
+              serviceUrl: undefined,
+              expect: 'latest-composable',
             }),
           ).resolves.toMatchSnapshot('schema publish same url');
 
           const versions = await fetchVersions(3);
-          expect(versions).toHaveLength(1);
+          expect(versions).toHaveLength(2);
 
           const versionWithNewServiceUrl = versions[0];
 
+          const schema = versionWithNewServiceUrl.schemas.nodes?.[0];
+          expect(schema.__typename).toBe('CompositeSchema');
+          expect((schema as CompositeSchema).url).toBe('http://localhost:4000');
+
           expect(await compareToPreviousVersion(versionWithNewServiceUrl.id)).toEqual(
             expect.objectContaining({
               target: expect.objectContaining({
                 schemaVersion: expect.objectContaining({
-                  safeSchemaChanges: null,
+                  safeSchemaChanges: {
+                    nodes: expect.anything(),
+                  },
                   schemaCompositionErrors: null,
                 }),
               }),