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

Update FlowAuth demo data #1948

Merged
merged 4 commits into from
Feb 13, 2020
Merged

Update FlowAuth demo data #1948

merged 4 commits into from
Feb 13, 2020

Conversation

jc-harrison
Copy link
Member

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

Updates the FlowAuth demo data to include permissions for all query kinds currently exposed, and replace 'cell' and 'site' aggregation units with 'lon-lat' to match the aggregation units exposed through the API.

This includes adding 'handset' permission, which is required for the worked example that would be added in #1938 and would otherwise need to be added manually in FlowAuth by logging in as an admin user.

@jc-harrison jc-harrison added FlowAuth Issues related to FlowAuth worked_examples labels Feb 12, 2020
@cypress
Copy link

cypress bot commented Feb 12, 2020



Test summary

57 0 0 0


Run details

Project FlowAuth
Status Passed
Commit 227be0c
Started Feb 13, 2020 10:25 AM
Ended Feb 13, 2020 10:28 AM
Duration 02:09 💡
OS Linux Debian - 8.11
Browser Electron 78

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

@codecov
Copy link

codecov bot commented Feb 12, 2020

Codecov Report

Merging #1948 into master will increase coverage by 12.1%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1948      +/-   ##
==========================================
+ Coverage   81.86%   93.97%   +12.1%     
==========================================
  Files           8      201     +193     
  Lines         568     8376    +7808     
  Branches       49      799     +750     
==========================================
+ Hits          465     7871    +7406     
- Misses         95      380     +285     
- Partials        8      125     +117
Flag Coverage Δ
#autoflow_unit_tests 91.94% <ø> (?)
#flowapi_unit_tests 100% <ø> (+16.02%) ⬆️
#flowauth_unit_tests 100% <ø> (?)
#flowclient_unit_tests 100% <ø> (+21.21%) ⬆️
#flowetl_unit_tests 100% <ø> (?)
#flowkit_jwt_generator_unit_tests 100% <ø> (?)
#flowmachine_unit_tests 90.72% <ø> (?)
#integration_tests 70.73% <ø> (?)
Impacted Files Coverage Δ
flowapi/flowapi/jwt_auth_callbacks.py 72.22% <0%> (-27.78%) ⬇️
flowclient/flowclient/client.py 67.74% <0%> (-11.05%) ⬇️
flowapi/flowapi/user_model.py 78.82% <0%> (-10.47%) ⬇️
flowapi/flowapi/query_endpoints.py 72.28% <0%> (-7.43%) ⬇️
flowapi/flowapi/flowapi_errors.py 94.87% <0%> (-5.13%) ⬇️
flowapi/flowapi/main.py 96.42% <0%> (-3.58%) ⬇️
...e/server/query_schemas/joined_spatial_aggregate.py 95.74% <0%> (ø)
...achine/core/server/query_schemas/daily_location.py 100% <0%> (ø)
...e/flowmachine/features/raster/raster_statistics.py 87.87% <0%> (ø)
flowmachine/flowmachine/core/custom_query.py 100% <0%> (ø)
... and 191 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 297a5ea...227be0c. Read the comment docs.

@greenape greenape added the ready-to-merge Label indicating a PR is OK to automerge label Feb 13, 2020
@mergify mergify bot merged commit 360ff73 into master Feb 13, 2020
@mergify mergify bot deleted the update-demodata branch February 13, 2020 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FlowAuth Issues related to FlowAuth ready-to-merge Label indicating a PR is OK to automerge worked_examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants