@rnx-kit/cli 0.12.3 → 0.12.6

package/coverage/lcov-report/src/typescript/project-cache.ts.html

@@ -23,16 +23,16 @@
         <div class='clearfix'>
             
             <div class='fl pad1y space-right2'>
-                <span class="strong">8.69% </span>
+                <span class="strong">7.84% </span>
                 <span class="quiet">Statements</span>
-                <span class='fraction'>4/46</span>
+                <span class='fraction'>4/51</span>
             </div>
         
             
             <div class='fl pad1y space-right2'>
                 <span class="strong">0% </span>
                 <span class="quiet">Branches</span>
-                <span class='fraction'>0/14</span>
+                <span class='fraction'>0/18</span>
             </div>
         
             
@@ -44,9 +44,9 @@
         
             
             <div class='fl pad1y space-right2'>
-                <span class="strong">9.09% </span>
+                <span class="strong">8.16% </span>
                 <span class="quiet">Lines</span>
-                <span class='fraction'>4/44</span>
+                <span class='fraction'>4/49</span>
             </div>
         
             
@@ -245,7 +245,38 @@
 <a name='L180'></a><a href='#L180'>180</a>
 <a name='L181'></a><a href='#L181'>181</a>
 <a name='L182'></a><a href='#L182'>182</a>
-<a name='L183'></a><a href='#L183'>183</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
+<a name='L183'></a><a href='#L183'>183</a>
+<a name='L184'></a><a href='#L184'>184</a>
+<a name='L185'></a><a href='#L185'>185</a>
+<a name='L186'></a><a href='#L186'>186</a>
+<a name='L187'></a><a href='#L187'>187</a>
+<a name='L188'></a><a href='#L188'>188</a>
+<a name='L189'></a><a href='#L189'>189</a>
+<a name='L190'></a><a href='#L190'>190</a>
+<a name='L191'></a><a href='#L191'>191</a>
+<a name='L192'></a><a href='#L192'>192</a>
+<a name='L193'></a><a href='#L193'>193</a>
+<a name='L194'></a><a href='#L194'>194</a>
+<a name='L195'></a><a href='#L195'>195</a>
+<a name='L196'></a><a href='#L196'>196</a>
+<a name='L197'></a><a href='#L197'>197</a>
+<a name='L198'></a><a href='#L198'>198</a>
+<a name='L199'></a><a href='#L199'>199</a>
+<a name='L200'></a><a href='#L200'>200</a>
+<a name='L201'></a><a href='#L201'>201</a>
+<a name='L202'></a><a href='#L202'>202</a>
+<a name='L203'></a><a href='#L203'>203</a>
+<a name='L204'></a><a href='#L204'>204</a>
+<a name='L205'></a><a href='#L205'>205</a>
+<a name='L206'></a><a href='#L206'>206</a>
+<a name='L207'></a><a href='#L207'>207</a>
+<a name='L208'></a><a href='#L208'>208</a>
+<a name='L209'></a><a href='#L209'>209</a>
+<a name='L210'></a><a href='#L210'>210</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -333,6 +364,17 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
@@ -352,6 +394,10 @@
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -403,11 +449,19 @@
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
-<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
+<span class="cline-any cline-no">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -438,6 +492,7 @@ import {
   Project,
   readConfigFile,
 } from "@rnx-kit/typescript-service";
+import fs from "fs";
 import path from "path";
 import ts from "typescript";
 &nbsp;
@@ -473,7 +528,10 @@ export interface ProjectCache {
    * @param sourceFile Source file
    * @returns Project targeting the given platform and containing the given source file
    */
-  getProjectInfo(sourceFile: string, platform: AllPlatforms): ProjectInfo;
+  getProjectInfo(
+    sourceFile: string,
+    platform: AllPlatforms
+  ): ProjectInfo | undefined;
 }
 &nbsp;
 /**
@@ -513,8 +571,19 @@ export function createProjectCache(
 <span class="cstat-no" title="statement not covered" >    return root;</span>
   }
 &nbsp;
-  function <span class="fstat-no" title="function not covered" >readTSConfig(</span>root: string): ts.ParsedCommandLine {
+  function <span class="fstat-no" title="function not covered" >readTSConfig(</span>root: string): ts.ParsedCommandLine | undefined {
     const configFileName = <span class="cstat-no" title="statement not covered" >path.join(root, "tsconfig.json");</span>
+<span class="cstat-no" title="statement not covered" >    if (!fs.existsSync(configFileName)) {</span>
+      // Allow for packages that aren't TypeScript.
+      //
+      // Example: Users who enable bundling with all the config defaults will
+      // have type validation enabled automatically. They may not actually be
+      // using TypeScript.
+      //
+      // We shouldn't break them. We should use TS validation only for TS packages.
+      //
+<span class="cstat-no" title="statement not covered" >      return undefined;</span>
+    }
 &nbsp;
     const cmdLine = <span class="cstat-no" title="statement not covered" >readConfigFile(configFileName);</span>
 <span class="cstat-no" title="statement not covered" >    if (!cmdLine) {</span>
@@ -531,9 +600,13 @@ export function createProjectCache(
   function <span class="fstat-no" title="function not covered" >createProjectInfo(</span>
     root: string,
     platform: AllPlatforms
-  ): ProjectInfo {
+  ): ProjectInfo | undefined {
     // Load the TypeScript configuration file for this project.
     const cmdLine = <span class="cstat-no" title="statement not covered" >readTSConfig(root);</span>
+<span class="cstat-no" title="statement not covered" >    if (!cmdLine) {</span>
+      // Not a TypeScript project
+<span class="cstat-no" title="statement not covered" >      return undefined;</span>
+    }
 &nbsp;
     //  Trim down the list of source files found by TypeScript. This ensures
     //  that only explicitly added files are loaded and parsed by TypeScript.
@@ -580,15 +653,23 @@ export function createProjectCache(
   function <span class="fstat-no" title="function not covered" >getProjectInfo(</span>
     sourceFile: string,
     platform: AllPlatforms
-  ): ProjectInfo {
+  ): ProjectInfo | undefined {
     const root = <span class="cstat-no" title="statement not covered" >findProjectRoot(sourceFile);</span>
 <span class="cstat-no" title="statement not covered" >    projects[root] ||= {};</span>
 &nbsp;
-    let info = <span class="cstat-no" title="statement not covered" >projects[root][platform];</span>
-<span class="cstat-no" title="statement not covered" >    if (!info) {</span>
-<span class="cstat-no" title="statement not covered" >      info = createProjectInfo(root, platform);</span>
-<span class="cstat-no" title="statement not covered" >      projects[root][platform] = info;</span>
+    const platforms = <span class="cstat-no" title="statement not covered" >projects[root];</span>
+&nbsp;
+    // Have we seen the project/platform for this source file before,
+    // even if what we saw is 'undefined' (e.g. not a TS project)?
+<span class="cstat-no" title="statement not covered" >    if (Object.prototype.hasOwnProperty.call(platforms, platform)) {</span>
+<span class="cstat-no" title="statement not covered" >      return platforms[platform];</span>
     }
+&nbsp;
+    // We haven't seen this project/platform before. Try to load it,
+    // even if it isn't a TS project. Cache the result so we don't
+    // do this again.
+    const info = <span class="cstat-no" title="statement not covered" >createProjectInfo(root, platform);</span>
+<span class="cstat-no" title="statement not covered" >    platforms[platform] = info;</span>
 <span class="cstat-no" title="statement not covered" >    return info;</span>
   }
 &nbsp;
@@ -616,7 +697,7 @@ export function createProjectCache(
             <div class='footer quiet pad2 space-top1 center small'>
                 Code coverage generated by
                 <a href="https://istanbul.js.org/" target="_blank" rel="noopener noreferrer">istanbul</a>
-                at Tue May 10 2022 07:45:57 GMT+0000 (Coordinated Universal Time)
+                at Fri May 13 2022 10:52:50 GMT+0000 (Coordinated Universal Time)
             </div>
         <script src="../../prettify.js"></script>
         <script>