Feat(medusa): Integration testing of cli (#1689)
* add initial testing files * prepare for github * remove empty files * remove ids * set user and password for postgres service * remove node setup * remove unused flag from yarn install * correct medusajs package * arguments for create-medusa project * make create command into one line * working directory update * update workflow * add build * run scripts from medusa-cli * move scripts to medusa-cli * add install * get server output * rename build to install * update login script to get feedback * add arguments to login script * add argument for directory to get-product * pass cli-test to get-products script * add wait command * fix waiting * update wait-for-server * echo status * update start testing * refactor code into test-action * add list config for testing * include shell * update get-products script * test to see if action fails when given wrong input * use custom action for testing devleopment server and extend waiting for server to spin up * update actions * remove cache version * yarn and build in cli-test directory * update core * update working directories * test working dir after cd up * update * fix directories * update directories * use setup server action * invoke correct action * test * update cli action * remove action * test with medusa-dev * remove medusa dev for now * test * include medusa-dev * remove invalid medusa new command * try moving project one dir up * add silent to curl output from live server wait * update seed data directory * test * remove sub workflow * add matrix and postgres override for medusa-config * fix matrix syntax * add database url to job * update yaml * cli should fail on logging in * create setup server action * update test-server * update scripts * run scripts directory instead of cli scripts * make directories current * remove tests from cli * run only on pr * update name * remove batch job model (for testing purposes) * remove ls from test server action * get content from develop * undo changes to generate reference * remove comments from scripts * test failing command * redo migrate command failing * move scripts into interation tests * Apply suggestions from code review Co-authored-by: Oliver Windall Juhl <59018053+olivermrbl@users.noreply.github.com> Co-authored-by: Oliver Windall Juhl <59018053+olivermrbl@users.noreply.github.com>
This commit is contained in:
1 parent
93aaaa71bd
commit
aca90a47ac
6 files changed
+243
No files matched your search
@@ -0,0 +1,42 @@
|
|||||||
|
name: "Setup test env"
|
||||||
|
description: "Setup test environment for actions"
|
||||||
|
|
||||||
|
inputs:
|
||||||
|
node-version:
|
||||||
|
description: "Node version"
|
||||||
|
required: false
|
||||||
|
default: "14"
|
||||||
|
cache-extension:
|
||||||
|
description: "Extension for fetching cached dependencies"
|
||||||
|
required: true
|
||||||
|
|
||||||
|
runs:
|
||||||
|
using: "composite"
|
||||||
|
steps:
|
||||||
|
- name: Cancel Previous Runs
|
||||||
|
uses: styfle/cancel-workflow-action@0.9.1
|
||||||
|
with:
|
||||||
|
access_token: ${{ github.token }}
|
||||||
|
|
||||||
|
- name: Setup Node.js environment
|
||||||
|
uses: actions/setup-node@v2.4.1
|
||||||
|
with:
|
||||||
|
node-version: ${{ inputs.node-version }}
|
||||||
|
cache: "yarn"
|
||||||
|
|
||||||
|
- name: Bootstrap packages
|
||||||
|
uses: ./.github/actions/cache-bootstrap
|
||||||
|
with:
|
||||||
|
extension: ${{ inputs.cache-extension }}
|
||||||
|
|
||||||
|
- name: Build Packages
|
||||||
|
shell: "bash"
|
||||||
|
run: yarn build
|
||||||
|
|
||||||
|
- name: Install dev cli
|
||||||
|
shell: "bash"
|
||||||
|
run: sudo npm i -g medusa-dev-cli
|
||||||
|
|
||||||
|
- name: Set path to medusa repo
|
||||||
|
shell: "bash"
|
||||||
|
run: medusa-dev --set-path-to-repo $(pwd)
|
||||||
@@ -0,0 +1,33 @@
|
|||||||
|
name: "Test server"
|
||||||
|
description: "Test the currently running medusa server to see if a user has been created and that the server is seeded"
|
||||||
|
|
||||||
|
inputs:
|
||||||
|
email:
|
||||||
|
description: "email of user to log in"
|
||||||
|
required: false
|
||||||
|
default: "test@test.com"
|
||||||
|
password:
|
||||||
|
description: "password of user to log in"
|
||||||
|
required: false
|
||||||
|
default: "password"
|
||||||
|
pathToSeedData:
|
||||||
|
description: "path to seed data"
|
||||||
|
required: false
|
||||||
|
default: "../cli-test/data/seed.json"
|
||||||
|
|
||||||
|
runs:
|
||||||
|
using: "composite"
|
||||||
|
steps:
|
||||||
|
- name: Wait for live server response
|
||||||
|
shell: "bash"
|
||||||
|
run: ./integration-tests/scripts/cli/wait-for-server-live.sh
|
||||||
|
- name: Log in with user
|
||||||
|
shell: "bash"
|
||||||
|
run: ./integration-tests/scripts/cli/login.sh ${{ inputs.email }} ${{ inputs.password }}
|
||||||
|
- name: GetProducts
|
||||||
|
shell: "bash"
|
||||||
|
run: ./integration-tests/scripts/cli/get-products.sh ${{ inputs.pathToSeedData }}
|
||||||
|
|
||||||
|
- name: Kill server
|
||||||
|
shell: "bash"
|
||||||
|
run: kill -9 $(lsof -t -i :9000)
|
||||||
@@ -0,0 +1,100 @@
|
|||||||
|
name: CLI Pipeline
|
||||||
|
on: [pull_request]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
test-cli-with-database:
|
||||||
|
env:
|
||||||
|
REDIS_URL: redis://localhost:6379
|
||||||
|
DATABASE_URL: "postgres://postgres:postgres@localhost/cli-test"
|
||||||
|
|
||||||
|
strategy:
|
||||||
|
matrix:
|
||||||
|
db: [sqlite, postgres]
|
||||||
|
|
||||||
|
services:
|
||||||
|
redis:
|
||||||
|
image: redis
|
||||||
|
# Set health checks to wait until redis has started
|
||||||
|
options: >-
|
||||||
|
--health-cmd "redis-cli ping"
|
||||||
|
--health-interval 10s
|
||||||
|
--health-timeout 5s
|
||||||
|
--health-retries 5
|
||||||
|
ports:
|
||||||
|
- 6379:6379
|
||||||
|
|
||||||
|
postgres:
|
||||||
|
image: postgres
|
||||||
|
env:
|
||||||
|
POSTGRES_PASSWORD: postgres
|
||||||
|
POSTGRES_USER: postgres
|
||||||
|
POSTGRES_DB: cli-test
|
||||||
|
options: >-
|
||||||
|
--health-cmd pg_isready
|
||||||
|
--health-interval 10s
|
||||||
|
--health-timeout 5s
|
||||||
|
--health-retries 5
|
||||||
|
ports:
|
||||||
|
- 5432:5432
|
||||||
|
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v2.3.5
|
||||||
|
with:
|
||||||
|
fetch-depth: 0
|
||||||
|
|
||||||
|
- name: Setup development server
|
||||||
|
uses: ./.github/actions/setup-server
|
||||||
|
with:
|
||||||
|
cache-extension: "cli-test"
|
||||||
|
|
||||||
|
- name: Install Medusa cli
|
||||||
|
run: yarn global add @medusajs/medusa-cli
|
||||||
|
|
||||||
|
- name: Create Medusa project
|
||||||
|
run: |
|
||||||
|
medusa new cli-test
|
||||||
|
working-directory: ..
|
||||||
|
|
||||||
|
- name: Install postgres config
|
||||||
|
if: matrix.db == 'postgres'
|
||||||
|
run: |
|
||||||
|
curl \
|
||||||
|
https://raw.githubusercontent.com/medusajs/medusa-starter-default/feat/postgres-config/medusa-config.js \
|
||||||
|
--output medusa-config.js
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
- name: run medusa dev
|
||||||
|
run: medusa-dev --force-install
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
- name: Run migrations
|
||||||
|
run: medusa migrations run
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
- name: Seed db
|
||||||
|
run: yarn seed
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
- name: Create admin user
|
||||||
|
run: medusa user -e test@test.com -p password -i admin_123
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
########################## Test medusa develop ###############################
|
||||||
|
|
||||||
|
- name: Run development server
|
||||||
|
run: medusa develop > /dev/null 2>&1 &
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
- name: Testing development server
|
||||||
|
uses: ./.github/actions/test-server
|
||||||
|
|
||||||
|
########################### Test medusa start ################################
|
||||||
|
|
||||||
|
- name: Starting medusa
|
||||||
|
run: medusa start > /dev/null 2>&1 &
|
||||||
|
working-directory: ../cli-test
|
||||||
|
|
||||||
|
- name: Testing server
|
||||||
|
uses: ./.github/actions/test-server
|
||||||
+24
@@ -0,0 +1,24 @@
|
|||||||
|
#!/bin/bash
|
||||||
|
|
||||||
|
seedJson=$(pwd)/$1
|
||||||
|
|
||||||
|
res=$(curl -s 'http://localhost:9000/store/products' | \
|
||||||
|
python3 -c "
|
||||||
|
|
||||||
|
import sys, json;
|
||||||
|
|
||||||
|
loadedProducts = json.load(sys.stdin)['products'];
|
||||||
|
|
||||||
|
seededProducts = json.load(open(\""$seedJson"\"))['products'];
|
||||||
|
|
||||||
|
result = set([product['title'] for product in loadedProducts]) == set([product['title'] for product in seededProducts])
|
||||||
|
|
||||||
|
print(result)
|
||||||
|
")
|
||||||
|
|
||||||
|
if [[ "$res" != "True" ]] ; then
|
||||||
|
echo "Seed failed, products are not equal"
|
||||||
|
exit 1
|
||||||
|
else
|
||||||
|
exit 0
|
||||||
|
fi
|
||||||
Executable
+15
@@ -0,0 +1,15 @@
|
|||||||
|
#!/bin/bash
|
||||||
|
|
||||||
|
status_code=$(curl \
|
||||||
|
-X POST\
|
||||||
|
-H "Content-Type: application/json"\
|
||||||
|
-d '{"email":"'$1'", "password":"'$2'"}'\
|
||||||
|
--write-out %{http_code}\
|
||||||
|
http://localhost:9000/admin/auth)
|
||||||
|
|
||||||
|
if [[ "$status_code" -ne 200 ]] ; then
|
||||||
|
echo "Site status changed to $status_code"
|
||||||
|
exit 1
|
||||||
|
else
|
||||||
|
exit 0
|
||||||
|
fi
|
||||||
+29
@@ -0,0 +1,29 @@
|
|||||||
|
#!/bin/bash
|
||||||
|
|
||||||
|
for i in {1..6}
|
||||||
|
do
|
||||||
|
echo $i
|
||||||
|
status_code=$(curl \
|
||||||
|
-X GET \
|
||||||
|
--write-out %{http_code} \
|
||||||
|
--silent\
|
||||||
|
--output /dev/null\
|
||||||
|
http://localhost:9000/store/products)
|
||||||
|
|
||||||
|
echo $status_code
|
||||||
|
if [[ "$status_code" -ne 000 ]] ; then
|
||||||
|
echo "exiting"
|
||||||
|
exit 0
|
||||||
|
else
|
||||||
|
sleep 5
|
||||||
|
fi
|
||||||
|
done
|
||||||
|
|
||||||
|
echo $status_code
|
||||||
|
|
||||||
|
if [[ "$status_code" = 000 ]] ; then
|
||||||
|
echo "Site status changed to $status_code"
|
||||||
|
exit 1
|
||||||
|
else
|
||||||
|
exit 0
|
||||||
|
fi
|
||||||
Reference in new issue
Block a user