Merge pull request #40197 from shobhitchittora/terminal-max-font-size-test

Adding test for integrated terminal font-size overflow

src/vs/workbench/parts/terminal/test/electron-browser/terminalConfigHelper.test.ts

@@ -104,6 +104,21 @@ suite('Workbench - TerminalConfigHelper', () => {
 		configHelper.panelContainer = fixture;
 		assert.equal(configHelper.getFont().fontSize, 6, 'The minimum terminal font size should be used when terminal.integrated.fontSize less than it');
 
+		configurationService = new MockConfigurationService({
+			editor: {
+				fontFamily: 'foo'
+			},
+			terminal: {
+				integrated: {
+					fontFamily: 0,
+					fontSize: 1500
+				}
+			}
+		});
+		configHelper = new TerminalConfigHelper(configurationService, null, null, null);
+		configHelper.panelContainer = fixture;
+		assert.equal(configHelper.getFont().fontSize, 25, 'The maximum terminal font size should be used when terminal.integrated.fontSize more than it');
+
 		configurationService = new MockConfigurationService({
 			editor: {
 				fontFamily: 'foo',