npm - @itwin/reports-config-widget-react - Versions diffs - 0.5.0 → 0.7.0 - Mend

@itwin/reports-config-widget-react 0.5.0 → 0.7.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (386) hide show

package/coverage/lcov-report/src/widget/components/Reports.tsx.html CHANGED Viewed

@@ -44,9 +44,9 @@
             <div class='fl pad1y space-right2'>
-                <span class="strong">89.28% </span>
+                <span class="strong">90.38% </span>
                 <span class="quiet">Lines</span>
-                <span class='fraction'>50/56</span>
+                <span class='fraction'>47/52</span>
             </div>
@@ -232,61 +232,7 @@
 <a name='L167'></a><a href='#L167'>167</a>
 <a name='L168'></a><a href='#L168'>168</a>
 <a name='L169'></a><a href='#L169'>169</a>
-<a name='L170'></a><a href='#L170'>170</a>
-<a name='L171'></a><a href='#L171'>171</a>
-<a name='L172'></a><a href='#L172'>172</a>
-<a name='L173'></a><a href='#L173'>173</a>
-<a name='L174'></a><a href='#L174'>174</a>
-<a name='L175'></a><a href='#L175'>175</a>
-<a name='L176'></a><a href='#L176'>176</a>
-<a name='L177'></a><a href='#L177'>177</a>
-<a name='L178'></a><a href='#L178'>178</a>
-<a name='L179'></a><a href='#L179'>179</a>
-<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>
-<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>
-<a name='L211'></a><a href='#L211'>211</a>
-<a name='L212'></a><a href='#L212'>212</a>
-<a name='L213'></a><a href='#L213'>213</a>
-<a name='L214'></a><a href='#L214'>214</a>
-<a name='L215'></a><a href='#L215'>215</a>
-<a name='L216'></a><a href='#L216'>216</a>
-<a name='L217'></a><a href='#L217'>217</a>
-<a name='L218'></a><a href='#L218'>218</a>
-<a name='L219'></a><a href='#L219'>219</a>
-<a name='L220'></a><a href='#L220'>220</a>
-<a name='L221'></a><a href='#L221'>221</a>
-<a name='L222'></a><a href='#L222'>222</a>
-<a name='L223'></a><a href='#L223'>223</a>
-<a name='L224'></a><a href='#L224'>224</a></td><td class="line-coverage quiet"><span class="cline-any cline-neutral">&nbsp;</span>
+<a name='L170'></a><a href='#L170'>170</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>
@@ -295,10 +241,6 @@
 <span class="cline-any cline-yes">7x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">7x</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-yes">7x</span>
 <span class="cline-any cline-yes">7x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
@@ -338,53 +280,34 @@
 <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-yes">7x</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-yes">7x</span>
 <span class="cline-any cline-yes">75x</span>
 <span class="cline-any cline-yes">75x</span>
 <span class="cline-any cline-yes">75x</span>
-<span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">75x</span>
 <span class="cline-any cline-yes">75x</span>
 <span class="cline-any cline-yes">75x</span>
 <span class="cline-any cline-yes">75x</span>
 <span class="cline-any cline-yes">75x</span>
-<span class="cline-any cline-yes">8x</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-yes">75x</span>
 <span class="cline-any cline-yes">8x</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-yes">75x</span>
 <span class="cline-any cline-yes">1x</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-yes">75x</span>
-<span class="cline-any cline-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">50x</span>
-<span class="cline-any cline-yes">122x</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-yes">196x</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>
@@ -392,10 +315,6 @@
 <span class="cline-any cline-yes">4x</span>
 <span class="cline-any cline-neutral">&nbsp;</span>
 <span class="cline-any cline-yes">4x</span>
-<span class="cline-any cline-yes">4x</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-yes">75x</span>
@@ -453,29 +372,7 @@
 <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-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-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-neutral">&nbsp;</span>
-<span class="cline-any cline-yes">144x</span>
+<span class="cline-any cline-yes">216x</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>
@@ -486,9 +383,7 @@
 <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-yes">26x</span>
-<span class="cline-any cline-neutral">&nbsp;</span>
+<span class="cline-any cline-yes">28x</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>
@@ -506,9 +401,6 @@
 <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></td><td class="text"><pre class="prettyprint lang-js">/*---------------------------------------------------------------------------------------------
  * Copyright (c) Bentley Systems, Incorporated. All rights reserved.
  * See LICENSE.md in the project root for license terms and full copyright notice.
@@ -517,14 +409,10 @@ import { SvgAdd, SvgPlay, SvgRefresh } from "@itwin/itwinui-icons-react";
 import { Button, IconButton } from "@itwin/itwinui-react";
 import React, { useCallback, useEffect, useMemo, useState } from "react";
 import type { CreateTypeFromInterface } from "./utils";
-import {
-  EmptyMessage,
-  handleError,
-  LoadingOverlay,
-} from "./utils";
+import { EmptyMessage, handleError, LoadingOverlay } from "./utils";
 import "./Reports.scss";
 import DeleteModal from "./DeleteModal";
-import type { Report, ReportsClient } from "@itwin/insights-client";
+import type { IReportsClient, Report } from "@itwin/insights-client";
 import { ReportHorizontalTile } from "./ReportHorizontalTile";
 import { SearchBar } from "./SearchBar";
 import { useReportsConfigApi } from "../context/ReportsConfigApiContext";
@@ -539,8 +427,8 @@ const fetchReports = async (
   setReports: (reports: Report[]) =&gt; void,
   iTwinId: string | undefined,
   setIsLoading: (isLoading: boolean) =&gt; void,
-  reportsClient: ReportsClient,
-  getAccessToken: () =&gt; Promise&lt;AccessToken&gt;
+  reportsClient: IReportsClient,
+  getAccessToken: () =&gt; Promise&lt;AccessToken&gt;,
 ) =&gt; {
   try {
     <span class="missing-if-branch" title="if path not taken" >I</span>if (!iTwinId) <span class="cstat-no" title="statement not covered" >return;</span>
@@ -555,70 +443,47 @@ const fetchReports = async (
   }
 };
 &nbsp;
+/**
+ * Props for {@link Reports} component.
+ * @public
+ */
 export interface ReportsProps {
   onClickAddReport?: () =&gt; void;
   onClickReportTitle?: (report: Report) =&gt; void;
   onClickReportModify?: (report: Report) =&gt; void;
 }
 &nbsp;
-export const Reports = ({
-  onClickAddReport,
-  onClickReportModify,
-  onClickReportTitle,
-}: ReportsProps) =&gt; {
+/**
+ * Component to manage and list reports for an iTwin.
+ * @public
+ */
+export const Reports = ({ onClickAddReport, onClickReportModify, onClickReportTitle }: ReportsProps) =&gt; {
   const { iTwinId, getAccessToken, reportsClient } = useReportsConfigApi();
   const { bulkExtractor } = useBulkExtractor();
-  const [showDeleteModal, setShowDeleteModal] = useState&lt;Report | undefined&gt;(
-    undefined
-  );
+  const [showDeleteModal, setShowDeleteModal] = useState&lt;Report | undefined&gt;(undefined);
   const [selectedReportIds, setSelectedReportIds] = useState&lt;string[]&gt;([]);
   const [isLoading, setIsLoading] = useState&lt;boolean&gt;(true);
   const [searchValue, setSearchValue] = useState&lt;string&gt;("");
   const [reports, setReports] = useState&lt;Report[]&gt;([]);
-  const jobStartEvent = useMemo(
-    () =&gt; new BeEvent&lt;(reportId: string) =&gt; void&gt;(),
-    []
-  );
+  const jobStartEvent = useMemo(() =&gt; new BeEvent&lt;(reportId: string) =&gt; void&gt;(), []);
 &nbsp;
   useEffect(() =&gt; {
-    void fetchReports(
-      setReports,
-      iTwinId,
-      setIsLoading,
-      reportsClient,
-      getAccessToken
-    );
+    void fetchReports(setReports, iTwinId, setIsLoading, reportsClient, getAccessToken);
   }, [getAccessToken, iTwinId, reportsClient, setIsLoading]);
 &nbsp;
   const refresh = useCallback(async () =&gt; {
-    await fetchReports(
-      setReports,
-      iTwinId,
-      setIsLoading,
-      reportsClient,
-      getAccessToken
-    );
+    await fetchReports(setReports, iTwinId, setIsLoading, reportsClient, getAccessToken);
   }, [getAccessToken, iTwinId, reportsClient]);
 &nbsp;
   const filteredReports = useMemo(
-    () =&gt;
-      reports.filter((x) =&gt;
-        [x.displayName, x.description]
-          .join(" ")
-          .toLowerCase()
-          .includes(searchValue.toLowerCase())
-      ),
-    [reports, searchValue]
+    () =&gt; reports.filter((x) =&gt; [x.displayName, x.description].join(" ").toLowerCase().includes(searchValue.toLowerCase())),
+    [reports, searchValue],
   );
 &nbsp;
   const onSelectionChange = (reportId: string, control: boolean) =&gt; {
     if (!control) setSelectedReportIds([]);
 &nbsp;
-    setSelectedReportIds((sr) =&gt;
-      sr.some(<span class="fstat-no" title="function not covered" >(r</span>) =&gt; <span class="cstat-no" title="statement not covered" >reportId === r)</span>
-        ? <span class="branch-0 cbranch-no" title="branch not covered" >sr.filter(<span class="fstat-no" title="function not covered" >(r</span>) =&gt; <span class="cstat-no" title="statement not covered" >reportId !== r)</span></span>
-        : [...sr, reportId]
-    );
+    setSelectedReportIds((sr) =&gt; (sr.some(<span class="fstat-no" title="function not covered" >(r</span>) =&gt; <span class="cstat-no" title="statement not covered" >reportId === r)</span> ? <span class="branch-0 cbranch-no" title="branch not covered" >sr.filter(<span class="fstat-no" title="function not covered" >(r</span>) =&gt; <span class="cstat-no" title="statement not covered" >reportId !== r)</span> </span>: [...sr, reportId]));
   };
 &nbsp;
   const updateDatasets = useCallback(<span class="fstat-no" title="function not covered" >async </span>() =&gt; {
@@ -636,46 +501,28 @@ export const Reports = ({
         &lt;div className="rcw-toolbar"&gt;
           &lt;div className="rcw-button-spacing"&gt;
             {onClickAddReport &amp;&amp; (
-              &lt;Button
-                startIcon={&lt;SvgAdd /&gt;}
-                onClick={onClickAddReport}
-                styleType="high-visibility"
-                title="New Report"
-              &gt;
-                {ReportsConfigWidget.localization.getLocalizedString(
-                  "ReportsConfigWidget:New"
-                )}
+              &lt;Button startIcon={&lt;SvgAdd /&gt;} onClick={onClickAddReport} styleType="high-visibility" title="New Report"&gt;
+                {ReportsConfigWidget.localization.getLocalizedString("ReportsConfigWidget:New")}
               &lt;/Button&gt;
             )}
             &lt;IconButton
-              title={ReportsConfigWidget.localization.getLocalizedString(
-                "ReportsConfigWidget:UpdateDatasets"
-              )}
+              title={ReportsConfigWidget.localization.getLocalizedString("ReportsConfigWidget:UpdateDatasets")}
               onClick={updateDatasets}
               disabled={selectedReportIds.length === 0}
             &gt;
               &lt;SvgPlay /&gt;
             &lt;/IconButton&gt;
           &lt;/div&gt;
-          &lt;div
-            className="rcw-search-bar-container"
-            data-testid="rcw-search-bar"
-          &gt;
+          &lt;div className="rcw-search-bar-container" data-testid="rcw-search-bar"&gt;
             &lt;IconButton
-              title={ReportsConfigWidget.localization.getLocalizedString(
-                "ReportsConfigWidget:Refresh"
-              )}
+              title={ReportsConfigWidget.localization.getLocalizedString("ReportsConfigWidget:Refresh")}
               onClick={refresh}
               disabled={isLoading}
               styleType="borderless"
             &gt;
               &lt;SvgRefresh /&gt;
             &lt;/IconButton&gt;
-            &lt;SearchBar
-              searchValue={searchValue}
-              setSearchValue={setSearchValue}
-              disabled={isLoading}
-            /&gt;
+            &lt;SearchBar searchValue={searchValue} setSearchValue={setSearchValue} disabled={isLoading} /&gt;
           &lt;/div&gt;
         &lt;/div&gt;
         {isLoading ? (
@@ -683,14 +530,10 @@ export const Reports = ({
         ) : reports.length === 0 &amp;&amp; onClickAddReport ? (
           &lt;EmptyMessage&gt;
             &lt;&gt;
-              {ReportsConfigWidget.localization.getLocalizedString(
-                "ReportsConfigWidget:NoReports"
-              )}
+              {ReportsConfigWidget.localization.getLocalizedString("ReportsConfigWidget:NoReports")}
               &lt;div&gt;
                 &lt;Button onClick={onClickAddReport} styleType="cta"&gt;
-                  {ReportsConfigWidget.localization.getLocalizedString(
-                    "ReportsConfigWidget:CreateOneReportCTA"
-                  )}
+                  {ReportsConfigWidget.localization.getLocalizedString("ReportsConfigWidget:CreateOneReportCTA")}
                 &lt;/Button&gt;
               &lt;/div&gt;
             &lt;/&gt;
@@ -709,9 +552,7 @@ export const Reports = ({
                 }}
                 onClickModify={onClickReportModify}
                 onSelectionChange={onSelectionChange}
-                selected={selectedReportIds.some(
-                  (reportId) =&gt; report.id === reportId
-                )}
+                selected={selectedReportIds.some((reportId) =&gt; report.id === reportId)}
               /&gt;
             ))}
           &lt;/div&gt;
@@ -722,10 +563,7 @@ export const Reports = ({
         onClose={() =&gt; setShowDeleteModal(undefined)}
         onDelete={async () =&gt; {
           const accessToken = await getAccessToken();
-          await reportsClient.deleteReport(
-            accessToken,
-            showDeleteModal?.id ?? <span class="branch-1 cbranch-no" title="branch not covered" >""</span>
-          );
+          await reportsClient.deleteReport(accessToken, showDeleteModal?.id ?? <span class="branch-1 cbranch-no" title="branch not covered" >"")</span>;
         }}
         refresh={refresh}
       /&gt;
@@ -739,7 +577,7 @@ export const Reports = ({
             <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 2024-01-29T21:35:13.448Z
+                at 2024-06-11T15:49:29.425Z
             </div>
         <script src="../../../prettify.js"></script>
         <script>