Merge pull request #17579 from adamdmharvey/adamdmharvey/minor-typos
chore: Fix various code and comment typos
This commit is contained in:
@@ -39,7 +39,7 @@ describe('MockRootLoggerService', () => {
|
||||
expect(console.debug).not.toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('shuld be able to set none level', () => {
|
||||
it('should be able to set none level', () => {
|
||||
jest.spyOn(console, 'error').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'warn').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'info').mockImplementation(() => {});
|
||||
@@ -57,7 +57,7 @@ describe('MockRootLoggerService', () => {
|
||||
expect(console.debug).not.toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('shuld be able to set error level', () => {
|
||||
it('should be able to set error level', () => {
|
||||
jest.spyOn(console, 'error').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'warn').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'info').mockImplementation(() => {});
|
||||
@@ -75,7 +75,7 @@ describe('MockRootLoggerService', () => {
|
||||
expect(console.debug).not.toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('shuld be able to set warn level', () => {
|
||||
it('should be able to set warn level', () => {
|
||||
jest.spyOn(console, 'error').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'warn').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'info').mockImplementation(() => {});
|
||||
@@ -93,7 +93,7 @@ describe('MockRootLoggerService', () => {
|
||||
expect(console.debug).not.toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('shuld be able to set info level', () => {
|
||||
it('should be able to set info level', () => {
|
||||
jest.spyOn(console, 'error').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'warn').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'info').mockImplementation(() => {});
|
||||
@@ -111,7 +111,7 @@ describe('MockRootLoggerService', () => {
|
||||
expect(console.debug).not.toHaveBeenCalled();
|
||||
});
|
||||
|
||||
it('shuld be able to set debug level', () => {
|
||||
it('should be able to set debug level', () => {
|
||||
jest.spyOn(console, 'error').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'warn').mockImplementation(() => {});
|
||||
jest.spyOn(console, 'info').mockImplementation(() => {});
|
||||
|
||||
@@ -34,7 +34,7 @@ describe('logCollector', () => {
|
||||
expect(logs.error).toEqual(['c', '3']);
|
||||
});
|
||||
|
||||
it('should collect some logs asynchrnously', async () => {
|
||||
it('should collect some logs asynchronously', async () => {
|
||||
const logs = await withLogCollector(async () => {
|
||||
console.log('a');
|
||||
console.warn('b');
|
||||
@@ -70,7 +70,7 @@ describe('logCollector', () => {
|
||||
expect(missedLogs.error).toEqual(['c', '3']);
|
||||
});
|
||||
|
||||
it('should collect specific logs asynchrnously', async () => {
|
||||
it('should collect specific logs asynchronously', async () => {
|
||||
const missedLogs = await withLogCollector(async () => {
|
||||
const logs = await withLogCollector(['error'], async () => {
|
||||
console.log('a');
|
||||
|
||||
+1
-1
@@ -209,7 +209,7 @@ describe('confluence:transform:markdown', () => {
|
||||
expect(writeFile).toHaveBeenCalledTimes(1);
|
||||
});
|
||||
|
||||
it('shoud fail on the first fetch call with response.ok set to false', async () => {
|
||||
it('should fail on the first fetch call with response.ok set to false', async () => {
|
||||
const options = {
|
||||
reader,
|
||||
integrations,
|
||||
|
||||
+1
-1
@@ -107,7 +107,7 @@ export const createConfluenceToMarkdownAction = (options: {
|
||||
`Could not find document ${url}. Please check your input.`,
|
||||
);
|
||||
}
|
||||
// This gets attachements for the confluence page if they exist
|
||||
// This gets attachments for the confluence page if they exist
|
||||
const getDocAttachments = await fetchConfluence(
|
||||
`/rest/api/content/${getConfluenceDoc.results[0].id}/child/attachment`,
|
||||
config,
|
||||
|
||||
@@ -324,7 +324,7 @@ function containsSkippedContent(localOutputPath: string): boolean {
|
||||
// if the path is empty means that there is a file skipped in the root
|
||||
// if the path starts with a separator it means that the root directory has been skipped
|
||||
// if the path includes // means that there is a subdirectory skipped
|
||||
// All paths returned are considered with / seperator because of globby returning the linux seperator for all os'.
|
||||
// All paths returned are considered with / separator because of globby returning the linux separator for all os'.
|
||||
return (
|
||||
localOutputPath === '' ||
|
||||
localOutputPath.startsWith('/') ||
|
||||
|
||||
@@ -48,7 +48,7 @@ export interface FactRetrieverEngine {
|
||||
* Schedules fact retriever run cycles based on configuration provided in the registration.
|
||||
*
|
||||
* Default implementation uses backend-tasks to handle scheduling. This function can be called multiple
|
||||
* times, where initial calls schedule the tasks and subsequents invocations update the schedules.
|
||||
* times, where initial calls schedule the tasks and subsequent invocations update the schedules.
|
||||
*/
|
||||
schedule(): Promise<void>;
|
||||
|
||||
|
||||
@@ -506,7 +506,7 @@ describe('AzureBlobStoragePublish', () => {
|
||||
name: 'path',
|
||||
};
|
||||
|
||||
const techDocsMetadaFilePath = path.posix.join(
|
||||
const techDocsMetadataFilePath = path.posix.join(
|
||||
...Object.values(invalidEntityName),
|
||||
'techdocs_metadata.json',
|
||||
);
|
||||
@@ -514,7 +514,7 @@ describe('AzureBlobStoragePublish', () => {
|
||||
const fails = publisher.fetchTechDocsMetadata(invalidEntityName);
|
||||
|
||||
await expect(fails).rejects.toMatchObject({
|
||||
message: `TechDocs metadata fetch failed; caused by Error: The file ${techDocsMetadaFilePath} does not exist!`,
|
||||
message: `TechDocs metadata fetch failed; caused by Error: The file ${techDocsMetadataFilePath} does not exist!`,
|
||||
});
|
||||
});
|
||||
});
|
||||
|
||||
Reference in New Issue
Block a user