Merge remote-tracking branch 'upstream/master'

This commit is contained in:
Clare Liguori
2023-01-12 11:02:16 -08:00
@@ -28,7 +28,6 @@ import {
TestApiProvider,
} from '@backstage/test-utils';
import { setupServer } from 'msw/node';
import { DateTime } from 'luxon';
import { siteMock } from '../../mocks/mocks';
import { AzureSitesOverviewTable } from './AzureSitesOverviewTable';
import { azureSiteApiRef } from '../../api';
@@ -72,15 +71,19 @@ describe('AzureSitesOverviewWidget', () => {
</TestApiProvider>,
);
expect(await rendered.findByText(siteMock.name)).toBeInTheDocument();
expect(await rendered.findByText(siteMock.location)).toBeInTheDocument();
expect(await rendered.findByText(siteMock.state)).toBeInTheDocument();
expect(
await rendered.findByText(
DateTime.fromISO(siteMock.lastModifiedDate).toLocaleString(
DateTime.DATETIME_MED,
),
),
).toBeInTheDocument();
await expect(
rendered.findByText(siteMock.name),
).resolves.toBeInTheDocument();
expect(rendered.getByText(siteMock.location)).toBeInTheDocument();
expect(rendered.getByText(siteMock.state)).toBeInTheDocument();
// TODO(Rugvip): This check is disabled, because in Node.js 18.13 an unexpected
// invisible whitespace character is present in the formatted time.
// expect(
// rendered.getByText(
// DateTime.fromISO(siteMock.lastModifiedDate).toLocaleString(
// DateTime.DATETIME_MED,
// ),
// ),
// ).toBeInTheDocument();
});
});