Skip to content
This repository was archived by the owner on Jul 14, 2026. It is now read-only.

make .octomind folder the default for push, pull, delete and edit.#317

Merged
fabianboth merged 5 commits into
edit-test-casefrom
require-octomind-folder
Jan 12, 2026
Merged

make .octomind folder the default for push, pull, delete and edit.#317
fabianboth merged 5 commits into
edit-test-casefrom
require-octomind-folder

Conversation

@fabianboth

Copy link
Copy Markdown
Contributor

This makes the .octomind folder the required folder for pull, push, delete and edit commands. Much like with .git, we can find the .octomind folder for the current working directory (traversing upwards).

Comment thread src/tools/test-cases.ts
Comment thread src/tools/test-cases.ts
Comment thread src/tools/test-cases.ts Outdated
Comment thread src/helpers.ts Outdated
Comment thread src/helpers.ts
Comment thread src/tools/test-cases.ts Outdated
@fabianboth
fabianboth merged commit 34694b4 into edit-test-case Jan 12, 2026
data: { success: true },
error: undefined,
afterEach(() => {
console.log = originalConsoleLog;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that this reset is necessary, since jest isolated the module registry. (I can't find the documentation where I read this before, but I would just try and leave it out and see if a console.log in another test still works ;))

testCaseId: "test-case-id",
});
expect(handleError).toHaveBeenCalledWith(undefined);
expect(console.log).toHaveBeenCalledWith("Test Case deleted successfully");

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess really we should assert that client.DELETE has been called.

In general I think console.log assertions are quite brittle, but i guess in this case they are part of the interface?

Comment thread tests/helpers.spec.ts
@Germandrummer92
Germandrummer92 deleted the require-octomind-folder branch January 16, 2026 14:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants