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

Plugin schemas #2366

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from
Draft

Plugin schemas #2366

wants to merge 10 commits into from

Conversation

greenape
Copy link
Member

Closes

I have:

  • Formatted any Python files with black
  • Brought the branch up to date with master
  • Added any relevant Github labels
  • Added tests for any new additions
  • Added or updated any relevant documentation
  • Added an Architectural Decision Record (ADR), if appropriate
  • Added an MPLv2 License Header if appropriate
  • Updated the Changelog

Description

Adds an entrypoint-based loader for flowmachine schemas (aka, plugins).

@codecov
Copy link

codecov bot commented Apr 20, 2020

Codecov Report

Merging #2366 into master will decrease coverage by 0.00%.
The diff coverage is 96.87%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2366      +/-   ##
==========================================
- Coverage   95.31%   95.30%   -0.01%     
==========================================
  Files         210      216       +6     
  Lines        8618     8652      +34     
  Branches      814      814              
==========================================
+ Hits         8214     8246      +32     
- Misses        287      289       +2     
  Partials      117      117              
Flag Coverage Δ
#autoflow_unit_tests 92.91% <ø> (ø)
#flowapi_unit_tests 82.10% <ø> (ø)
#flowauth_unit_tests 94.74% <ø> (ø)
#flowclient_unit_tests 75.21% <ø> (ø)
#flowetl_unit_tests 100.00% <ø> (ø)
#flowkit_jwt_generator_unit_tests 100.00% <ø> (ø)
#flowmachine_unit_tests 91.18% <92.70%> (+0.06%) ⬆️
#integration_tests 71.51% <96.87%> (+0.10%) ⬆️
Impacted Files Coverage Δ
flowmachine/flowmachine/core/server/server.py 89.36% <ø> (-0.23%) ⬇️
...hine/flowmachine/core/server/query_schemas/util.py 87.50% <87.50%> (ø)
...e/server/query_schemas/joined_spatial_aggregate.py 94.28% <88.88%> (-1.46%) ⬇️
...machine/flowmachine/core/server/action_handlers.py 87.82% <100.00%> (ø)
...wmachine/core/server/query_schemas/displacement.py 100.00% <100.00%> (ø)
...owmachine/core/server/query_schemas/event_count.py 100.00% <100.00%> (ø)
...hine/core/server/query_schemas/flowable_queries.py 100.00% <100.00%> (ø)
...ine/core/server/query_schemas/flowmachine_query.py 100.00% <100.00%> (ø)
...ine/flowmachine/core/server/query_schemas/flows.py 100.00% <100.00%> (ø)
...e/flowmachine/core/server/query_schemas/handset.py 100.00% <100.00%> (ø)
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c0f9a2...3ef8d1b. Read the comment docs.

@codecov
Copy link

codecov bot commented Apr 20, 2020

Codecov Report

Merging #2366 into master will decrease coverage by 0.00%.
The diff coverage is 97.60%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2366      +/-   ##
==========================================
- Coverage   95.46%   95.45%   -0.01%     
==========================================
  Files         235      242       +7     
  Lines        9060     9085      +25     
  Branches      827      827              
==========================================
+ Hits         8649     8672      +23     
- Misses        292      294       +2     
  Partials      119      119              
Flag Coverage Δ
#autoflow_unit_tests 92.91% <ø> (ø)
#flowapi_unit_tests 82.10% <ø> (ø)
#flowauth_unit_tests 94.75% <ø> (ø)
#flowclient_unit_tests 74.05% <ø> (ø)
#flowetl_unit_tests 100.00% <ø> (ø)
#flowkit_jwt_generator_unit_tests 100.00% <ø> (ø)
#flowmachine_unit_tests 91.49% <94.40%> (+0.04%) ⬆️
#integration_tests 72.75% <97.60%> (+0.06%) ⬆️
Impacted Files Coverage Δ
flowmachine/flowmachine/core/server/server.py 89.36% <ø> (-0.23%) ⬇️
...hine/flowmachine/core/server/query_schemas/util.py 87.50% <87.50%> (ø)
...e/server/query_schemas/joined_spatial_aggregate.py 94.28% <91.66%> (-1.46%) ⬇️
...machine/flowmachine/core/server/action_handlers.py 87.82% <100.00%> (ø)
...ery_schemas/active_at_reference_location_counts.py 100.00% <100.00%> (ø)
...erver/query_schemas/consecutive_trips_od_matrix.py 100.00% <100.00%> (ø)
...wmachine/core/server/query_schemas/displacement.py 100.00% <100.00%> (ø)
...owmachine/core/server/query_schemas/event_count.py 100.00% <100.00%> (ø)
...hine/core/server/query_schemas/flowable_queries.py 100.00% <100.00%> (ø)
...ine/core/server/query_schemas/flowmachine_query.py 100.00% <100.00%> (ø)
... and 28 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7b04234...d6096b8. Read the comment docs.

@greenape greenape added enhancement New feature or request FlowMachine Issues related to FlowMachine labels Apr 20, 2020
@cypress
Copy link

cypress bot commented Apr 21, 2020



Test summary

43 0 0 0


Run details

Project FlowAuth
Status Passed
Commit d6096b8
Started Apr 24, 2020 2:10 PM
Ended Apr 24, 2020 2:12 PM
Duration 02:07 💡
OS Linux Debian - 8.11
Browser Electron 80

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request FlowMachine Issues related to FlowMachine
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant