Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add docker stack for e2e tests, remove cypress #524

Merged
merged 26 commits into from
Nov 18, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
6c4e2ba
Add e2e testing to npm scripts and CI
MattyMay Sep 30, 2024
a882a18
Update docs, scripts, and add two project settings tests
MattyMay Oct 11, 2024
22acdd1
Add hard deadline test
MattyMay Oct 1, 2024
d2146f4
add project access test
MattyMay Oct 1, 2024
d77b7c5
Add group settings test
MattyMay Oct 11, 2024
48f14a9
Add grading policy test and invalid group size test
MattyMay Oct 11, 2024
8499a59
Add error handling tests for group settings and deadlines
MattyMay Oct 4, 2024
0a9f186
Add submission limits test and hidden content test
MattyMay Oct 4, 2024
377ee4f
Refactor API cypress commands to use ag-typescript-client
MattyMay Oct 7, 2024
98d3d3f
Add test for trying to delete a project when there is a test suite
MattyMay Oct 7, 2024
1df87a2
Fix eslint errors and format new files
MattyMay Oct 11, 2024
05bd0ec
Fix broken test, tweak static_analysis to skip test_stack
MattyMay Oct 11, 2024
e33dec4
Move test_stack to project root
MattyMay Oct 11, 2024
21af0d2
Fix static_analysis with more specific patterns, delete .eslintignore
MattyMay Oct 11, 2024
720404b
WIP
MattyMay Oct 14, 2024
e66c95a
Experimenting with e2e test data setup
james-perretta Oct 14, 2024
df2fd00
WIP: Refactoring project_settings e2e tests to be fully typed.
james-perretta Oct 15, 2024
6c16940
Remove old test_stack directory.
Oct 15, 2024
cc49c9a
Refactored project_settings e2e tests to use typed setup data.
Oct 15, 2024
bd94d15
WIP: Adding form input test helpers.
Oct 15, 2024
d4a32b9
Linter fixes
james-perretta Oct 16, 2024
efb319b
Skip a not-implemented test.
james-perretta Oct 16, 2024
65876a4
Fixed proxy settings for e2e stack. Removed nginx proxy in favor of v…
james-perretta Oct 16, 2024
c9c61e3
Remove cypress and revert cypress-specific changes.
james-perretta Nov 18, 2024
84218b0
Update package-lock.json and CI
james-perretta Nov 18, 2024
01bb6bb
Merge pull request #527 from eecs-autograder/remove_cypress
james-perretta Nov 18, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion .eslintrc.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,11 @@
"extends": [
"eslint:recommended",
"plugin:vue/essential",
"plugin:@typescript-eslint/recommended"
"plugin:@typescript-eslint/recommended",
"plugin:@typescript-eslint/recommended-type-checked"
],
"parserOptions": {
"project": true,
"ecmaVersion": 12,
"parser": "@typescript-eslint/parser",
"sourceType": "module"
Expand Down
34 changes: 27 additions & 7 deletions .github/workflows/node.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,9 @@ on:
push:
branches: [ develop ]
pull_request:
branches: [ develop ]

jobs:
build:

lint:
runs-on: ubuntu-latest

strategy:
Expand All @@ -21,14 +19,36 @@ jobs:

steps:
- uses: actions/checkout@v2

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'
- run: npm install

- run: npm ci
- run: npm run lint
- run: npm run build
- run: npm test
- run: npx cypress run --component

component-tests:
# Lint is fast. If we wait for it we guarantee a cache hit on dependencies.
needs: lint

runs-on: ubuntu-latest

strategy:
matrix:
node-version: [16.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/

steps:
- uses: actions/checkout@v2

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'

- run: npm ci
- run: npm run build
- run: npm run test
3 changes: 3 additions & 0 deletions .gitmodules
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
[submodule "e2e_test_stack/autograder-server"]
path = e2e_test_stack/autograder-server
url = [email protected]:eecs-autograder/autograder-server.git
16 changes: 0 additions & 16 deletions cypress.config.ts

This file was deleted.

5 changes: 0 additions & 5 deletions cypress/fixtures/example.json

This file was deleted.

37 changes: 0 additions & 37 deletions cypress/support/commands.ts

This file was deleted.

12 changes: 0 additions & 12 deletions cypress/support/component-index.html

This file was deleted.

40 changes: 0 additions & 40 deletions cypress/support/component.ts

This file was deleted.

20 changes: 0 additions & 20 deletions cypress/support/e2e.ts

This file was deleted.

149 changes: 0 additions & 149 deletions docker-compose.yml

This file was deleted.

1 change: 1 addition & 0 deletions e2e_test_stack/autograder-server
Submodule autograder-server added at 3a9045
Loading
Loading