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

VER-3: repo update. #9

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

VER-3: repo update. #9

wants to merge 5 commits into from

Conversation

n3op2
Copy link

@n3op2 n3op2 commented Jun 8, 2023

What

  • upgraded dependencies
  • left mock-jwks due to the need of ESM modules, please find error below
  sub-wallet methods
    unauthenticated
      ✔ should fail with 401 Unauthorized
    listing all dids
      ✔ should list 1 did
    listing all dids without a wallet
      ✔ should error 401
    listing dids with matching method
      ✔ should list 1 did
    listing dids with non-matching method
      ✔ should list 0 dids
    listing dids with invalid query
      ✔ should error 422
    creating did
      ✔ should create a did
    creating did with invalid body
      ✔ should create a did

  wallet creation
    unauthenticated
      ✔ should fail with 401 Unauthorized
    as a new user
      ✔ should create a new wallet
    as a new user with invalid body
      ✔ should pass through error
    as an existing user
      ✔ should not create a new wallet

  api-docs
    ✔ should return 200

  health
    ✔ health check

  server status
    unauthenticated
      ✔ should fail with 401
    server status without a wallet
      ✔ should succeed and return server status

  transformApiDoc
    ✔ should transform input -> expectation (case 1)
    ✔ should transform input -> expectation (case 2)


  18 passing (2s)
Error [ERR_REQUIRE_ESM]: require() of ES Module /Users/PMichelevicius/workspace/veritable-acapy-proxy/node_modules/mock-jwks/build/index.js from /Users/PMichelevicius/workspace/veritable-acapy-proxy/test/helper/auth0.js not supported.
Instead change the require of index.js in /Users/PMichelevicius/workspace/veritable-acapy-proxy/test/helper/auth0.js to a dynamic import() which is available in all CommonJS modules.
    at Object.<anonymous> (/Users/PMichelevicius/workspace/veritable-acapy-proxy/test/helper/auth0.js:2:24)
    at Object.<anonymous> (/Users/PMichelevicius/workspace/veritable-acapy-proxy/test/integration/acapyCall.test.js:5:27)
$ 

Copy link
Contributor

@dblane-digicatapult dblane-digicatapult left a comment

Choose a reason for hiding this comment

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

Remove all the helm stuff.

You have missed some node-version changes in the workflow.

Add a new docker-build workflow to the test.yaml

- uses: actions/checkout@master
- uses: actions/setup-node@master
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 16.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Node version

- uses: actions/checkout@master
- uses: actions/setup-node@master
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 16.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Node version

- uses: actions/checkout@master
- uses: actions/setup-node@master
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 16.x
Copy link
Contributor

Choose a reason for hiding this comment

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

node version

- uses: actions/checkout@master
- uses: actions/setup-node@master
- uses: actions/checkout@v3
- uses: actions/setup-node@v3
with:
node-version: 16.x
Copy link
Contributor

Choose a reason for hiding this comment

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

node version

Comment on lines 10 to 20
local HELM_CHART_VERSION=$(yq eval -o y '.version' ./helm/veritable-acapy-proxy/Chart.yaml)
local HELM_CHART_APP_VERSION=$(yq eval -o y '.appVersion' ./helm/veritable-acapy-proxy/Chart.yaml)

if [ "$PACKAGE_VERSION" != "$PACKAGE_LOCK_VERSION" ] ||
[ "v$PACKAGE_VERSION" != "$HELM_VALUES_TAG_VERSION" ] ||
[ "$PACKAGE_VERSION" != "$HELM_CHART_VERSION" ] ||
[ "$PACKAGE_VERSION" != "$HELM_CHART_APP_VERSION" ]; then
echo "Inconsistent versions detected"
echo "PACKAGE_VERSION: $PACKAGE_VERSION"
echo "PACKAGE_LOCK_VERSION: $PACKAGE_LOCK_VERSION"
echo "HELM_VALUES_TAG_VERSION: $HELM_VALUES_TAG_VERSION"
echo "HELM_CHART_VERSION: $HELM_CHART_VERSION"
echo "HELM_CHART_APP_VERSION: $HELM_CHART_APP_VERSION"
exit 1
[ "v$PACKAGE_VERSION" != "$HELM_VALUES_TAG_VERSION" ] ||
[ "$PACKAGE_VERSION" != "$HELM_CHART_VERSION" ] ||
[ "$PACKAGE_VERSION" != "$HELM_CHART_APP_VERSION" ]; then
echo "Inconsistent versions detected"
echo "PACKAGE_VERSION: $PACKAGE_VERSION"
echo "PACKAGE_LOCK_VERSION: $PACKAGE_LOCK_VERSION"
exit 1
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove all the helm references

Comment on lines 1 to 8
apiVersion: v2
name: veritable-acapy-proxy
appVersion: '1.1.0'
appVersion: '1.1.1'
description: A Helm chart for veritable-acapy-proxy
version: '1.1.0'
version: '1.1.1'
type: application
maintainers:
- name: digicatapult
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete the helm folder

Comment on lines 70 to 78
- name: Set up chart-testing
uses: helm/[email protected]
- name: Setup Minikube
uses: manusa/actions-setup-minikube@v2.3.0
uses: manusa/actions-setup-minikube@v2.7.2
with:
minikube version: 'v1.17.1'
kubernetes version: 'v1.19.2'
minikube version: 'v1.28.0'
kubernetes version: 'v1.25.4'
- name: Login to minikube docker registry
run: eval $(minikube -p minikube docker-env)
run: eval $(minikube -p minikube docker-env --unset)
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete all the helm testing

Comment on lines 76 to 87
- name: Set up chart-testing
uses: helm/[email protected]
- name: Setup Minikube
uses: manusa/actions-setup-minikube@v2.3.0
uses: manusa/actions-setup-minikube@v2.7.2
with:
minikube version: 'v1.17.1'
kubernetes version: 'v1.19.2'
minikube version: 'v1.28.0'
kubernetes version: 'v1.25.4'
- name: Login to minikube docker registry
run: eval $(minikube -p minikube docker-env)
run: eval $(minikube -p minikube docker-env --unset)
- name: Build and Publish image
uses: docker/build-push-action@v2
with:
Copy link
Contributor

Choose a reason for hiding this comment

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

Delete all the helm testing

Copy link
Contributor

@dblane-digicatapult dblane-digicatapult left a comment

Choose a reason for hiding this comment

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

in release.yaml line 127 needs altering
needs: [preconditions, helm-lint, helm-test, lint, dependency-check, tests, check-version]

Comment on lines 29 to 30
echo "::set-output name=REPO_NAME::$REPO_NAME"
echo "::set-output name=ORG_NAME::$ORG_NAME"
Copy link
Contributor

Choose a reason for hiding this comment

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

Update to new output setting notation

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