Merge pull request #194 from technote-space/release/v0.6.29

release/v0.6.29

Author: technote-space

__tests__/logger.test.ts

@@ -111,7 +111,9 @@ describe('Logger', () => {
 	describe('getColorString', () => {
 		it('should return color string', () => {
 			expect(logger.getColorString('Hello World!!!', 'blue', 'red', 'bold')).toBe('\x1b[34;41;1mHello World!!!\x1b[0m');
+			expect(logger.getColorString('Hello World!!!')).toBe('\x1b[37;40;0mHello World!!!\x1b[0m');
 			expect(logger.getColorString('Hello World!!!', 'green')).toBe('\x1b[32;40;0mHello World!!!\x1b[0m');
+			expect(logger.c('Hello World!!!')).toBe('\x1b[37;40;0mHello World!!!\x1b[0m');
 			expect(logger.c('Hello World!!!', 'green')).toBe('\x1b[32;40;0mHello World!!!\x1b[0m');
 			expect(logger.c('Hello World!!!', 'yellow', undefined, 'underline')).toBe('\x1b[33;40;4mHello World!!!\x1b[0m');
 		});

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@technote-space/github-action-helper",
-  "version": "0.6.28",
+  "version": "0.6.29",
   "description": "Helper to filter GitHub Action.",
   "author": "Technote <[email protected]> (https://technote.space)",
   "license": "MIT",
@@ -28,11 +28,11 @@
     "sprintf-js": "^1.1.2"
   },
   "devDependencies": {
-    "@technote-space/github-action-test-helper": "^0.0.27",
+    "@technote-space/github-action-test-helper": "^0.0.28",
     "@types/jest": "^24.0.25",
     "@types/node": "^13.1.6",
-    "@typescript-eslint/eslint-plugin": "^2.15.0",
-    "@typescript-eslint/parser": "^2.15.0",
+    "@typescript-eslint/eslint-plugin": "^2.16.0",
+    "@typescript-eslint/parser": "^2.16.0",
     "eslint": "^6.8.0",
     "jest": "^24.9.0",
     "jest-circus": "^24.9.0",

src/logger.ts

@@ -1,11 +1,11 @@
 /* eslint-disable @typescript-eslint/no-explicit-any */
-import { sprintf } from 'sprintf-js';
-import { info, debug, error, warning, startGroup, endGroup } from '@actions/core';
-import { split } from './utils';
+import {sprintf} from 'sprintf-js';
+import {info, debug, error, warning, startGroup, endGroup} from '@actions/core';
+import {split} from './utils';
 
 export type Color = 'black' | 'red' | 'green' | 'yellow' | 'blue' | 'magenta' | 'cyan' | 'white';
 export type Attribute = undefined | 'none' | 'bold' | 'underline' | 'italic';
-const COLOR_MAP     = {
+const COLOR_MAP = {
 	'black': 0,
 	'red': 1,
 	'green': 2,
@@ -156,21 +156,21 @@ export default class Logger {
 
 	/**
 	 * @param {string} string string
-	 * @param {Color} color color
-	 * @param {Color} backColor background color
-	 * @param {Attribute} attribute attribute
+	 * @param {Color|undefined} color color
+	 * @param {Color|undefined} backColor background color
+	 * @param {Attribute|undefined} attribute attribute
 	 * @return {string} color string
 	 */
-	public getColorString = (string: string, color: Color, backColor?: Color, attribute?: Attribute): string => sprintf('\x1b[3%d;4%d;%dm%s\x1b[0m', COLOR_MAP[color], COLOR_MAP[backColor ?? 'black'], ATTRIBUTE_MAP[attribute ?? 'none'], string);
+	public getColorString = (string: string, color?: Color, backColor?: Color, attribute?: Attribute): string => sprintf('\x1b[3%d;4%d;%dm%s\x1b[0m', COLOR_MAP[color ?? 'white'], COLOR_MAP[backColor ?? 'black'], ATTRIBUTE_MAP[attribute ?? 'none'], string);
 
 	/**
 	 * @param {string} string string
-	 * @param {Color} color color
-	 * @param {Color} backColor background color
-	 * @param {Attribute} attribute attribute
+	 * @param {Color|undefined} color color
+	 * @param {Color|undefined} backColor background color
+	 * @param {Attribute|undefined} attribute attribute
 	 * @return {string} color string
 	 */
-	public c = (string: string, color: Color, backColor?: Color, attribute?: Attribute): string => this.getColorString(string, color, backColor, attribute);
+	public c = (string: string, color?: Color, backColor?: Color, attribute?: Attribute): string => this.getColorString(string, color, backColor, attribute);
 
 	/**
 	 * @return {void}