Skip to content

Commit

Permalink
fix: deal with undefined variables passed into query string and updat…
Browse files Browse the repository at this point in the history
…e addattachmentonissue to take mime and filename
  • Loading branch information
Aeolun committed Jul 10, 2024
1 parent 9256b93 commit 97fc03b
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
5 changes: 3 additions & 2 deletions .github/workflows/test-and-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,14 +8,15 @@ jobs:
node-version: ['18', '20', '22', 'lts/*', 'latest']
steps:
- uses: actions/checkout@v4
- uses: pnpm/action-setup@v2
- uses: pnpm/action-setup@v4
with:
version: latest
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
cache: 'pnpm'
- run: pnpm install
- run: pnpm run build
- run: pnpm run test
- run: pnpm run lint
release:
Expand All @@ -36,7 +37,7 @@ jobs:
uses: actions/checkout@v4
with:
fetch-depth: 0
- uses: pnpm/action-setup@v2
- uses: pnpm/action-setup@v4
with:
version: latest
- name: Setup Node.js
Expand Down
6 changes: 3 additions & 3 deletions src/jira.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -75,21 +75,21 @@ describe("Jira API Tests", () => {
strictSSL: false,
});

expect(jira.httpsAgent.options.rejectUnauthorized).toBe(false);
expect(jira.httpsAgent?.options.rejectUnauthorized).toBe(false);
});

it("Constructor with ssl default empty", () => {
const { jira } = createTestInstance({});

expect(jira.httpsAgent.options.rejectUnauthorized).toBe(true);
expect(jira.httpsAgent?.options.rejectUnauthorized).toBe(true);
});

it("Constructor with with ssl checking enabled", () => {
const { jira } = createTestInstance({
strictSSL: true,
});

expect(jira.httpsAgent.options.rejectUnauthorized).toBe(true);
expect(jira.httpsAgent?.options.rejectUnauthorized).toBe(true);
});

it("should allow the user to pass in a certificate authority", () => {
Expand Down

0 comments on commit 97fc03b

Please sign in to comment.