FlexMeasures / flexmeasures / 24635809326
83%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 163
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 79.405% (+0.02%) from 79.387%
24635809326

push

github

web-flow 
Feat/sensor data source filtering (#2083)

* Initial plan

* data_sources: add account_id field to DataSource model

- Add account_id column (FK to account.id, nullable) to DataSource
- Add account relationship (backref 'data_sources' on Account)
- Update __table_args__ UniqueConstraint to include account_id
- Set self.account from user.account in __init__ when user is provided

* migration: add account_id to data_source table (9877450113f6)

- Add account_id column (nullable, FK to account.id)
- Data migration: populate account_id from user's account where user_id IS NOT NULL
- Update UniqueConstraint to include account_id column
- down_revision: 8b62f8129f34

* migration: replace N+1 loop with single UPDATE...FROM for account_id backfill

* tests/sensor_data: verify account_id is set on data source when posting sensor data

Context:
- DataSource.__init__ now sets self.account = user.account when a user is provided
- The POST sensor data endpoint calls get_or_create_source(current_user)
- Need to verify account_id is populated correctly on the resulting data source

Change:
- Added test_post_sensor_data_sets_account_id_on_data_source
- Posts sensor data as test_supplier_user_4@seita.nl
- Asserts data_source.account_id == user.account_id

* migration: use correlated subquery for account_id data migration; add changelog entry

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/7969297e-88c2-4ace-99c9-ec74109f03c8

* data_sources: fix account_id assignment to avoid SAWarning and handle unflushed users

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>
Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/7969297e-88c2-4ace-99c9-ec74109f03c8

* agents/architecture: add migration checklist and DataSource account_id invariants

Context:
- PR #2058 added account_id FK to DataSource table
- No migration checklist existed for reviewing/writing ... (continued)

12056 of 15183 relevant lines covered (79.4%)

0.79 hits per line

Coverage Regressions

Lines Coverage ∆ File
33
88.67
 0.0% flexmeasures/api/v3_0/sensors.py
31
80.19
 0.0% flexmeasures/data/models/time_series.py
26
61.11
 2.29% flexmeasures/data/queries/utils.py
16
89.57
 0.33% flexmeasures/api/common/schemas/sensor_data.py
Jobs
ID Job ID Ran Files Coverage
1 24635809326.1 163
79.4
 GitHub Action Run
Source Files on build 24635809326