Skip to content

Commit c27a792

Browse files
Merge pull request #15 from dremio/quickstart
Fixing issues on quickstart and adding tests
2 parents 80e1fe8 + 6502e2b commit c27a792

12 files changed

Lines changed: 754 additions & 50 deletions

File tree

.github/workflows/pytest.yml

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
name: Run Pytest with uv
2+
3+
on:
4+
push:
5+
branches: [ main ]
6+
pull_request:
7+
branches: [ main ]
8+
9+
jobs:
10+
test:
11+
runs-on: ubuntu-latest
12+
13+
steps:
14+
- name: Checkout code
15+
uses: actions/checkout@v3
16+
17+
- name: Install uv
18+
run: |
19+
curl -LsSf https://astral.sh/uv/install.sh | sh
20+
echo "$HOME/.cargo/bin" >> $GITHUB_PATH
21+
22+
- name: Set up Python
23+
uses: actions/setup-python@v4
24+
with:
25+
python-version: '3.13'
26+
27+
- name: Run tests
28+
run: |
29+
uv run pytest tests

.vscode/settings.json

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"python.testing.pytestArgs": [
3+
"tests"
4+
],
5+
"python.testing.unittestEnabled": false,
6+
"python.testing.pytestEnabled": true
7+
}

README.md

Lines changed: 18 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ architecture-beta
4949

5050
The MCP server runs locally on the machine that runs the LLM frontend (eg Claude). The installation steps are simple
5151

52-
1. Clone or download this repository.
52+
1. Clone or download this repository.
5353
2. Install the [uv](https://docs.astral.sh/uv/guides/install-python/) package manager. The MCP server requires python (3.11 or later)
5454
3. Do a sanity check by doing
5555

@@ -83,9 +83,9 @@ There are two configurations necessary before the MCP server can be invoked.
8383

8484
## Quick start
8585

86-
The quickest way to do this setup is -
86+
The quickest way to do this setup is -
8787

88-
1. Create the dremio config file using
88+
1. Create the dremio config file using
8989

9090
```shell
9191
$ uv run dremio-mcp-server config create dremio \
@@ -95,13 +95,13 @@ $ uv run dremio-mcp-server config create dremio \
9595
# --project-id <dremio project id>
9696
```
9797

98-
2. Download and install Claude desktop. And then create the Claude config file using
98+
2. Download and install Claude desktop. And then create the Claude config file using
9999

100100
```shell
101101
$ uv run dremio-mcp-server config create claude
102102
```
103103

104-
3. Validate the config files using
104+
3. Validate the config files using
105105

106106
```shell
107107
$ uv run dremio-mcp-server config list --type claude`
@@ -229,6 +229,18 @@ This will pickup the default location of MCP server config file. It can also be
229229

230230
This repository is intended to be open source software that encourages contributions of any kind, like adding features, reporting issues and contributing fixes. This is not a part of Dremio product support.
231231

232+
## Testing
233+
234+
The project uses pytest for testing. To run the tests:
235+
236+
```shell
237+
# Run all tests
238+
$ uv run pytest tests
239+
240+
```
241+
242+
GitHub Actions automatically runs tests on pull requests and pushes to the main branch.
243+
232244
## Contributing
233245

234246
Please see our [Contributing Guide](CONTRIBUTING.md) for details on:
@@ -237,3 +249,4 @@ Please see our [Contributing Guide](CONTRIBUTING.md) for details on:
237249
- Making contributions
238250
- Code style guidelines
239251
- Documentation requirements
252+
- Running tests

0 commit comments

Comments
 (0)