Skip to content

Commit

Permalink
fix: use proper wording for repo checkout (#496)
Browse files Browse the repository at this point in the history
Fixes #449

Signed-off-by: Jeff MAURY <[email protected]>
  • Loading branch information
jeffmaury authored Mar 11, 2024
1 parent bb33f23 commit c44228c
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions packages/backend/src/managers/applicationManager.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -432,7 +432,7 @@ describe('doCheckout', () => {
expect(cloneRepositoryMock).toBeCalledWith(gitCloneOptions);
expect(mocks.updateTaskMock).toHaveBeenLastCalledWith({
id: expect.any(String),
name: 'Checkout repository',
name: 'Checking out repository',
state: 'success',
labels: {
git: 'checkout',
Expand Down Expand Up @@ -472,7 +472,7 @@ describe('doCheckout', () => {
expect(cloneRepositoryMock).toBeCalledWith(gitCloneOptions);
expect(mocks.updateTaskMock).toHaveBeenLastCalledWith({
id: expect.any(String),
name: 'Checkout repository',
name: 'Checking out repository',
state: 'error',
labels: {
git: 'checkout',
Expand Down Expand Up @@ -512,7 +512,7 @@ describe('doCheckout', () => {
expect(cloneRepositoryMock).not.toHaveBeenCalled();
expect(mocks.updateTaskMock).toHaveBeenLastCalledWith({
id: expect.any(String),
name: 'Checkout repository (cached).',
name: 'Checking out repository (cached).',
state: 'success',
labels: {
git: 'checkout',
Expand Down
4 changes: 2 additions & 2 deletions packages/backend/src/managers/applicationManager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -553,7 +553,7 @@ export class ApplicationManager extends Publisher<ApplicationState[]> {

async doCheckout(gitCloneInfo: GitCloneInfo, labels?: { [id: string]: string }) {
// Creating checkout task
const checkoutTask: Task = this.taskRegistry.createTask('Checkout repository', 'loading', {
const checkoutTask: Task = this.taskRegistry.createTask('Checking out repository', 'loading', {
...labels,
git: 'checkout',
});
Expand All @@ -562,7 +562,7 @@ export class ApplicationManager extends Publisher<ApplicationState[]> {
// We might already have the repository cloned
if (fs.existsSync(gitCloneInfo.targetDirectory) && fs.statSync(gitCloneInfo.targetDirectory).isDirectory()) {
// Update checkout state
checkoutTask.name = 'Checkout repository (cached).';
checkoutTask.name = 'Checking out repository (cached).';
checkoutTask.state = 'success';
} else {
// Create folder
Expand Down

0 comments on commit c44228c

Please sign in to comment.