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

Setup of end to end tests and coverage #320

Merged
merged 10 commits into from
Oct 19, 2023
Merged

Setup of end to end tests and coverage #320

merged 10 commits into from
Oct 19, 2023

Conversation

zonia3000
Copy link
Collaborator

@zonia3000 zonia3000 commented Oct 19, 2023

Checklist before merging

  • I added an appropriate entry to CHANGELOG.md

Closes #8

@zonia3000 zonia3000 force-pushed the playwright branch 10 times, most recently from c63d4b1 to c5c474f Compare October 19, 2023 10:19
@github-actions
Copy link

github-actions bot commented Oct 19, 2023

Coverage report
 ---------------------------------------|---------|----------|---------|---------|-------------------
File                                   | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
---------------------------------------|---------|----------|---------|---------|-------------------
All files                              |   22.52 |    12.62 |   31.48 |   25.57 |                   
 src                                   |       0 |        0 |       0 |       0 |                   
  hooks.server.js                      |       0 |        0 |       0 |       0 | 4-40              
 src/lib/common                        |   37.37 |       30 |    42.1 |   36.47 |                   
  component_utilities.js               |   41.26 |    40.42 |      40 |   38.46 | ...1,62-67,82-132 
  errors.js                            |       0 |        0 |       0 |       0 | 9-92              
  jschema_validation.js                |   84.61 |       50 |     100 |     100 | 26-31             
 src/lib/components/common             |   47.39 |    34.09 |      30 |   46.15 |                   
  ConfirmActionButton.svelte           |   97.22 |    33.33 |     100 |     100 | 5-48,50-52        
  Modal.svelte                         |   79.48 |    86.66 |     100 |   78.37 | 27,35-43,60-71,84 
  StandardDismissableAlert.svelte      |   19.27 |    19.04 |      30 |   20.54 | ...27,133-140,158 
  StandardErrorAlert.svelte            |       0 |        0 |     100 |       0 | 4-27              
 src/lib/components/common/filterable  |       0 |        0 |     100 |       0 |                   
  Th.svelte                            |       0 |        0 |     100 |       0 | 2-22              
 src/lib/components/common/jschema     |    33.4 |    25.91 |   71.42 |   36.57 |                   
  ArrayProperty.svelte                 |       0 |        0 |       0 |       0 | 6-86              
  BooleanProperty.svelte               |       0 |        0 |     100 |       0 | 5-26              
  JSchema.svelte                       |       0 |        0 |       0 |       0 | 34-173            
  NumberProperty.svelte                |       0 |        0 |     100 |       0 | 5-26              
  ObjectProperty.svelte                |       0 |        0 |     100 |       0 | 6-119             
  PropertiesBlock.svelte               |       0 |        0 |       0 |       0 | 6-28              
  PropertyDescription.svelte           |       0 |        0 |       0 |       0 | 4-17              
  PropertyDiscriminator.svelte         |       0 |        0 |     100 |       0 | 8-40              
  SchemaInput.svelte                   |       0 |        0 |     100 |       0 | 4-58              
  StringProperty.svelte                |       0 |        0 |     100 |       0 | 5-23              
  schema_management.js                 |   83.85 |    69.64 |   81.08 |   83.69 | ...94-304,361-368 
 src/lib/components/jobs               |       0 |        0 |       0 |       0 |                   
  JobInfoModal.svelte                  |       0 |        0 |       0 |       0 | 7-92              
  JobLogsModal.svelte                  |       0 |        0 |     100 |       0 | 6-53              
  StatusBadge.svelte                   |       0 |        0 |     100 |       0 | 2-23              
  TimestampBadge.svelte                |       0 |        0 |     100 |       0 | 2-6               
 src/lib/components/projects           |   31.01 |    15.93 |   28.57 |   34.28 |                   
  CreateUpdateDatasetModal.svelte      |   27.99 |    14.94 |   25.14 |   31.98 | ...1989,2053-2057 
  ProjectDatasetsList.svelte           |    27.5 |     5.26 |      25 |   29.68 | ...54,469-498,514 
  ProjectInfoModal.svelte              |   40.32 |     6.06 |     100 |      50 | 3,39-55           
  ProjectsList.svelte                  |      84 |    86.36 |   66.66 |    87.5 | 14-16,39,61-63    
  WorkflowImport.svelte                |   32.14 |    19.23 |   33.33 |   36.32 | ...97,409-415,439 
  WorkflowsList.svelte                 |   32.89 |    13.88 |    41.5 |    35.8 | ...56-757,763-764 
  controller.js                        |       0 |        0 |       0 |       0 | 6-38              
 src/lib/components/tasks              |       0 |        0 |       0 |       0 |                   
  TaskCollection.svelte                |       0 |        0 |       0 |       0 | 11-337            
  TaskCollectionLogsModal.svelte       |       0 |        0 |     100 |       0 | 6-39              
  TaskEditModal.svelte                 |       0 |        0 |     100 |       0 | 7-236             
  TaskInfoModal.svelte                 |       0 |        0 |     100 |       0 | 17-84             
 src/lib/components/workflow           |       0 |        0 |       0 |       0 |                   
  ArgumentForm.svelte                  |       0 |        0 |     100 |       0 | 15-45             
  ArgumentsSchema.svelte               |       0 |        0 |       0 |       0 | 8-80              
  MetaPropertiesForm.svelte            |       0 |        0 |     100 |       0 | 16-55             
  WorkflowTaskSelection.svelte         |       0 |        0 |       0 |       0 | 10-251            
  task_form_utils.js                   |       0 |        0 |       0 |       0 | 19-122            
 src/lib/components/workflow/common    |       0 |        0 |       0 |       0 |                   
  FormBaseEntry.svelte                 |       0 |        0 |       0 |       0 | 7-104             
  FormBuilder.svelte                   |       0 |        0 |     100 |       0 | 5-30              
  FormStructure.svelte                 |       0 |        0 |     100 |       0 | 9-186             
  NewEntryProperty.svelte              |       0 |        0 |       0 |       0 | 3-135             
 src/lib/server/api/v1                 |       0 |        0 |       0 |       0 |                   
  auth_api.js                          |       0 |        0 |       0 |       0 | 11-62             
  monitoring_api.js                    |       0 |        0 |       0 |       0 | 12-24             
  project_api.js                       |       0 |        0 |       0 |       0 | 12-115            
  task_api.js                          |       0 |        0 |       0 |       0 | 10-22             
  workflow_api.js                      |       0 |        0 |       0 |       0 | 12-24             
 src/lib/stores                        |      25 |      100 |     100 |      25 |                   
  authStores.js                        |       0 |      100 |     100 |       0 | 3                 
  errorStores.js                       |       0 |      100 |     100 |       0 | 3                 
  projectStores.js                     |     100 |      100 |     100 |     100 |                   
  serverStores.js                      |       0 |      100 |     100 |       0 | 3                 
  taskStores.js                        |       0 |      100 |     100 |       0 | 4-7               
 src/routes                            |   43.26 |    52.38 |       0 |   46.47 |                   
  +layout.server.js                    |       0 |        0 |       0 |       0 | 6-38              
  +layout.svelte                       |      90 |    73.33 |     100 |   97.05 | 48                
  +page.svelte                         |       0 |        0 |     100 |       0 | 5-41              
 src/routes/api/[...path]              |       0 |        0 |       0 |       0 |                   
  +server.js                           |       0 |        0 |       0 |       0 | 7-82              
 src/routes/auth/login                 |       0 |        0 |       0 |       0 |                   
  +page.server.js                      |       0 |      100 |       0 |       0 | 13-45             
  +page.svelte                         |       0 |        0 |     100 |       0 | 2-36              
 src/routes/auth/logout                |       0 |      100 |       0 |       0 |                   
  +page.svelte                         |       0 |      100 |       0 |       0 | 6-15              
 src/routes/auth/token/logout          |       0 |      100 |       0 |       0 |                   
  +server.js                           |       0 |      100 |       0 |       0 | 13-26             
 src/routes/projects                   |   53.84 |      100 |       0 |      50 |                   
  +page.server.js                      |       0 |      100 |       0 |       0 | 4-17              
  +page.svelte                         |     100 |      100 |     100 |     100 |                   
 src/routes/projects/[projectId]       |   45.43 |       25 |   59.37 |   50.24 |                   
  +page.server.js                      |       0 |      100 |       0 |       0 | 4-11              
  +page.svelte                         |   45.82 |       25 |   60.31 |   50.73 | ...56-688,694-695 
 ...s/[projectId]/datasets/[datasetId] |       0 |        0 |       0 |       0 |                   
  +page.server.js                      |       0 |      100 |       0 |       0 | 7-14              
  +page.svelte                         |       0 |        0 |     100 |       0 | 6-185             
 src/routes/projects/[projectId]/jobs  |       0 |        0 |       0 |       0 |                   
  +page.server.js                      |       0 |        0 |       0 |       0 | 4-12              
  +page.svelte                         |       0 |        0 |       0 |       0 | 18-289            
 ...[projectId]/workflows/[workflowId] |       0 |        0 |       0 |       0 |                   
  +page.server.js                      |       0 |        0 |       0 |       0 | 5-18              
  +page.svelte                         |       0 |        0 |       0 |       0 | 17-939            
 src/routes/sandbox                    |       0 |        0 |       0 |       0 |                   
  +layout.js                           |       0 |        0 |       0 |       0 | 4-5               
 src/routes/sandbox/jsonschema         |       0 |      100 |     100 |       0 |                   
  +page.svelte                         |       0 |      100 |     100 |       0 | 5-15              
 src/routes/tasks                      |       0 |        0 |       0 |       0 |                   
  +page.server.js                      |       0 |      100 |       0 |       0 | 4-5               
  +page.svelte                         |       0 |        0 |       0 |       0 | 16-283            
---------------------------------------|---------|----------|---------|---------|-------------------
 

@zonia3000 zonia3000 marked this pull request as ready for review October 19, 2023 10:53
@tcompa tcompa mentioned this pull request Oct 19, 2023
@tcompa
Copy link
Collaborator

tcompa commented Oct 19, 2023

This is great, thanks @zonia3000!

I'm merging now, and let's use #321 to keep track of next steps.

@tcompa tcompa merged commit 9ffa473 into main Oct 19, 2023
10 checks passed
@tcompa tcompa deleted the playwright branch October 19, 2023 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set up end-to-end testing
2 participants