[Gardening] Refactor parse xml

__tests__/parser.test.ts

@@ -1,7 +1,29 @@
+import path from 'path'
 import {Annotation} from '../src/annotation'
-import {parseXml} from '../src/parser'
+import {parseXmls, parseXml} from '../src/parser'
 
-test('test parse', () => {
+test('test parseXmls', () => {
+  const file = path.join(__dirname, 'resource', 'sample-lint-results.xml')
+
+  const annotation1 = new Annotation(
+    'Warning',
+    'Useless parent layout: This `RelativeLayout` layout or its `FrameLayout` parent is useless; transfer the `background` attribute to the other view',
+    'layout.xml',
+    11,
+    22
+  )
+  const annotation2 = new Annotation(
+    'Error',
+    'Ignoring results: The result of `subscribe` is not used',
+    'Foo.kt',
+    33,
+    44
+  )
+
+  expect(parseXmls([file])).resolves.toEqual([annotation1, annotation2])
+})
+
+test('test parseXml', () => {
   const xml = `<?xml version="1.0" encoding="UTF-8"?>
   <issues format="5" by="lint 3.6.1">
       <issue

src/main.ts

@@ -1,8 +1,7 @@
-import fs from 'fs'
 import * as core from '@actions/core'
 import * as glob from '@actions/glob'
 import {echoMessages} from './command'
-import {parseXml} from './parser'
+import {parseXmls} from './parser'
 
 async function run(): Promise<void> {
   try {
@@ -14,14 +13,9 @@ async function run(): Promise<void> {
     const globber = await glob.create(xmlPath, globOptions)
     const files = await globber.glob()
 
-    const annotationsList = await Promise.all(
-      files.map(async file => {
-        const xml = fs.readFileSync(file, 'utf-8')
-        return await parseXml(xml)
-      })
-    )
+    const annotations = await parseXmls(files)
 
-    echoMessages(annotationsList.flat())
+    echoMessages(annotations)
   } catch (error) {
     core.setFailed(error.message)
   }

src/parser.ts

@@ -1,10 +1,21 @@
+import fs from 'fs'
 import * as core from '@actions/core'
 import * as xml2js from 'xml2js'
 import {Annotation} from './annotation'
 
-export const parseXml = async (reportXml: string): Promise<Annotation[]> => {
+export const parseXmls = async (files: string[]): Promise<Annotation[]> => {
+  const list = await Promise.all(
+    files.map(async file => {
+      const xml = fs.readFileSync(file, 'utf-8')
+      return await parseXml(xml)
+    })
+  )
+  return list.flat()
+}
+
+export const parseXml = async (text: string): Promise<Annotation[]> => {
   const parser = new xml2js.Parser()
-  const xml = await parser.parseStringPromise(reportXml)
+  const xml = await parser.parseStringPromise(text)
   return new Promise(resolve => {
     try {
       const annotations: Annotation[] = []