Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

EVEREST-107 update api-tests for multinamespace #431

Merged
merged 16 commits into from
Feb 9, 2024
Merged
16 changes: 0 additions & 16 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -366,22 +366,6 @@ jobs:
# API_TOKEN is used later by "make test"
echo "API_TOKEN=$(./bin/everest token reset --json | jq .token -r)" >> $GITHUB_ENV

- name: Provision monitoring
shell: bash
continue-on-error: true
run: |
cd percona-everest-cli

while true; do kubectl port-forward -n everest-system deployment/percona-everest 8080:8080; done &

sleep 2

./bin/everest monitoring enable \
--everest-url http://127.0.0.1:8080 \
--everest-token $API_TOKEN \
--instance-name pmm-local \
--skip-wizard


- name: Patch Everest Deployment to use the PR image
run: |
Expand Down
6 changes: 4 additions & 2 deletions api-tests/tests/auth.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.
import { expect, test } from '@fixtures'
import {checkError} from "@tests/tests/helpers";

test('auth header fails with invalid token', async ({ request }) => {
const version = await request.get('/v1/version', {
Expand All @@ -30,7 +31,8 @@ test('auth header is preferred over cookie', async ({ browser }) => {
const request = ctx.request

const version = await request.get('/v1/version')
expect(version.ok()).toBeTruthy()
await checkError(version)

})

test.describe('no authorization header', () => {
Expand Down Expand Up @@ -67,6 +69,6 @@ test.describe('no authorization header', () => {
const request = ctx.request

const version = await request.get('/v1/version')
expect(version.ok()).toBeTruthy()
await checkError(version)
})
})
Loading
Loading