Merge branch 'sabrams/fix_composer_installation_code' into 'master'

Fix Composer package installation code block

See merge request gitlab-org/gitlab!39400

app/assets/javascripts/packages/details/store/getters.js

@@ -110,6 +110,6 @@ export const composerRegistryInclude = ({ composerPath }) => {
   return JSON.stringify(base);
 };
 export const composerPackageInclude = ({ packageEntity }) => {
-  const base = { package_name: packageEntity.name };
+  const base = { [packageEntity.name]: packageEntity.version };
   return JSON.stringify(base);
 };

changelogs/unreleased/sabrams-fix_composer_installation_code.yml

+---
+title: Fix Composer installation code snippet to include package name and version
+merge_request: 39400
+author:
+type: fixed

spec/frontend/packages/details/store/getters_spec.js

@@ -13,6 +13,8 @@ import {
   nugetSetupCommand,
   pypiPipCommand,
   pypiSetupCommand,
+  composerRegistryInclude,
+  composerPackageInclude,
 } from '~/packages/details/store/getters';
 import {
   conanPackage,
@@ -68,6 +70,10 @@ describe('Getters PackageDetails Store', () => {
   const nugetSetupCommandStr = `nuget source Add -Name "GitLab" -Source "${registryUrl}" -UserName <your_username> -Password <your_token>`;
 
   const pypiPipCommandStr = `pip install ${pypiPackage.name} --index-url ${registryUrl}`;
+  const composerRegistryIncludeStr = '{"type":"composer","url":"foo"}';
+  const composerPackageIncludeStr = JSON.stringify({
+    [packageWithoutBuildInfo.name]: packageWithoutBuildInfo.version,
+  });
 
   describe('packagePipeline', () => {
     it('should return the pipeline info when pipeline exists', () => {
@@ -214,4 +220,18 @@ describe('Getters PackageDetails Store', () => {
       expect(pypiSetupCommand(state)).toBe(pypiSetupCommandStr);
     });
   });
+
+  describe('composer string getters', () => {
+    it('gets the correct composerRegistryInclude command', () => {
+      setupState({ composerPath: 'foo' });
+
+      expect(composerRegistryInclude(state)).toBe(composerRegistryIncludeStr);
+    });
+
+    it('gets the correct composerPackageInclude command', () => {
+      setupState();
+
+      expect(composerPackageInclude(state)).toBe(composerPackageIncludeStr);
+    });
+  });
 });