regression: wrong app limits displayed on the UI

ee/packages/license/src/deprecated.ts

@@ -4,7 +4,7 @@ import { getModules } from './modules';
 import { defaultLimits } from './validation/validateDefaultLimits';
 
 export const getLicenseLimit = (license: ILicenseV3 | undefined, kind: LicenseLimitKind) => {
-	const limitList = license?.limits[kind] ?? defaultLimits[kind as keyof typeof defaultLimits];
+	const limitList = license ? license.limits[kind] : defaultLimits[kind as keyof typeof defaultLimits];
 
 	if (!limitList?.length) {
 		return -1;

ee/packages/license/src/getLicenseLimit.spec.ts

@@ -0,0 +1,27 @@
+import { LicenseImp } from '.';
+import { MockedLicenseBuilder, getReadyLicenseManager } from '../__tests__/MockedLicenseBuilder';
+import { getAppsConfig } from './deprecated';
+
+describe('Marketplace Restrictions', () => {
+	it('should respect the default if there is no license applied', async () => {
+		const LicenseManager = new LicenseImp();
+
+		expect(getAppsConfig.call(LicenseManager)).toEqual({
+			maxPrivateApps: 3,
+			maxMarketplaceApps: 5,
+		});
+	});
+
+	it('should return unlimited if there is license but no limits', async () => {
+		const licenseManager = await getReadyLicenseManager();
+
+		const license = await new MockedLicenseBuilder();
+
+		await expect(licenseManager.setLicense(await license.sign())).resolves.toBe(true);
+
+		await expect(getAppsConfig.call(licenseManager)).toEqual({
+			maxPrivateApps: -1,
+			maxMarketplaceApps: -1,
+		});
+	});
+});

ee/packages/license/src/license.spec.ts

@@ -1,3 +1,4 @@
+import { LicenseImp } from '.';
 import { MockedLicenseBuilder, getReadyLicenseManager } from '../__tests__/MockedLicenseBuilder';
 
 it('should not prevent if there is no license', async () => {
@@ -231,3 +232,45 @@ describe('Validate License Limits', () => {
 		});
 	});
 });
+
+describe('License.getInfo', () => {
+	describe('Marketplace Restrictions', () => {
+		it('should respect the default if there is no license applied', async () => {
+			const licenseManager = new LicenseImp();
+
+			expect(
+				(
+					await licenseManager.getInfo({
+						limits: true,
+						currentValues: false,
+						license: false,
+					})
+				).limits,
+			).toMatchObject({
+				privateApps: { max: 3 },
+				marketplaceApps: { max: 5 },
+			});
+		});
+
+		it('should return unlimited if there is license but no limits', async () => {
+			const licenseManager = await getReadyLicenseManager();
+
+			const license = await new MockedLicenseBuilder();
+
+			await expect(licenseManager.setLicense(await license.sign())).resolves.toBe(true);
+
+			await expect(
+				(
+					await licenseManager.getInfo({
+						limits: true,
+						currentValues: false,
+						license: false,
+					})
+				).limits,
+			).toMatchObject({
+				privateApps: { max: -1 },
+				marketplaceApps: { max: -1 },
+			});
+		});
+	});
+});