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.

Sign up for GitHub

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

Jump to bottom

Entities #9

Open
wants to merge 146 commits into
base: master
Choose a base branch
from
Open

Entities #9

wants to merge 146 commits into from

Conversation

Alystrasz
Copy link
Member

@Alystrasz Alystrasz commented Oct 27, 2021 •
edited
Loading

Adding a bunch of entities to the library.

Datastream
  • Entity creation
    • Check phenomenonTime format
    • Check resultTime format
    • Can only be created from Thing.datastreams collection
  • DAO
  • Builder
  • Associated entity collections
  • A datastream may not have a unit of measurement
  • Documentation
Sensor
  • Entity creation
  • DAO
  • Builder
  • Associated entity collection
  • Documentation
ObservedProperty
  • Entity creation
  • DAO
  • Builder
  • Associated entity collection
  • Documentation
Observation
  • Entity creation
    • Check phenomenonTime
    • Check resultTime
    • Check validTime
    • phenomenonTime may be undefined
    • resultTime may be undefined
  • DAO
  • Builder
  • Associated entity collections
  • Documentation
FeatureOfInterest
  • Entity creation
  • DAO
  • Builder
  • Associated entity collection
  • Documentation

@github-actions
Copy link

github-actions bot commented Oct 27, 2021 •
edited
Loading

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
dist/bundle.js 16.84 KB (+14.26% 🔺) 337 ms (+14.26% 🔺) 68 ms (+119.54% 🔺) 404 ms

Alystrasz and others added 24 commits October 28, 2021 00:58
@Alystrasz
Merge branch 'master' into feat/entities
12f913d
@Alystrasz
[fix] DatastreamDao implements BaseDao.buildEntity member
da29860
@Alystrasz
[test] Datastream should throw when created with wrong phenomenon time
fda7e2b
@Alystrasz
[test] Datastream should throw when created with wrong result time
a493975
@Alystrasz
[feat] Datastream throws when created with wrong time range values
18c1998
@Alystrasz
[test] Datastream should throw when created with random string as fir…
a00e24a 
…st phenomenon time range member
@Alystrasz
[test] Datastream should throw when created with random string as sec…
fff8005 
…ond phenomenon time range member
@Alystrasz
[test] Datastream should throw when created with random strings as bo…
292d002 
…th phenomenon time range members
@Alystrasz
[test] Datastream should throw when created with too many phenomenon …
325f8f0 
…time range members
@Alystrasz
[test] adjusting awaited error messages in Datastream build tests
83909ff
@Alystrasz
[feat] Datastream cannot be built with falsy time range values
04d671d
@Alystrasz
[feat] adding ObservedProperty class skeleton
986d400
@Alystrasz
[feat] ObservedProperty builder can create entity instances
7571b11
@Alystrasz
[test] ObservedPropertyBuilder should not build ObservedProperty with…
e6dd021 
…out name
@Alystrasz
[test] ObservedPropertyBuilder should not build ObservedProperty with…
ac143f6 
…out description
@Alystrasz
[test] ObservedPropertyBuilder should not build ObservedProperty with…
c07d060 
…out definition
@Alystrasz
[feat] adding ObservedProperty DAO class
5c11148
@Alystrasz
[feat] datastreams and observed properties DAOs are accessible from s…
5196615 
…ervice
@Alystrasz
[feat] ObservedProperty now have datastreams list attribute
da3cdbf
@Alystrasz
[feat] Thing now have datastreams list attribute
d2c5b92
@Alystrasz
[docs] adding specification links to all entities
70045cb
@Alystrasz
[feat] Datastream have things list attribute
9adcda2
@Alystrasz
[feat] Datastream have observedProperties list attribute
e91dfc2
@Alystrasz
[feat] adding FeatureOfInterest entity skeleton
465e4a6
Alystrasz and others added 30 commits December 17, 2021 22:28
@Alystrasz
[feat] adding documentation to Observation entity
1728a80
@Alystrasz
[feat] adding documentation to ObservedProperty entity
e88444f
@Alystrasz
[feat] adding Sensor entity class skeleton
b9b9a58
@Alystrasz
[refactor] moving Sensor entity to dao/model package
73c740e
@Alystrasz
[feat] Sensor has Datastreams list attribute
6366848
@Alystrasz
[feat] adding SensorDao class skeleton
656bc88
@Alystrasz
[test] SensorDao should return correct path name
8d9f72f
@Alystrasz
[test] SensorDao should return public attributes
35aa976
@Alystrasz
[feat] adding SensorDao.entityPublicAttributes implementation
056b47c
@Alystrasz
[test] SensorDao should build instance from API response
e229e79
@Alystrasz
[feat] adding Sensor missing attributes (encodingType and metadata)
676b7f6
@Alystrasz
[feat] service holds sensor DAO reference
937e77c
@Alystrasz
[feat] Sensor.dao returns an instance of SensorDao
90be2b5
@Alystrasz
[feat] SensorDao can build Sensors through buildEntity method
12f916c
@Alystrasz
[feat] adding SensorBuilder class skeleton
2d9585a
@Alystrasz
[test] SensorBuilder should build an instance with all parameters
5eb9b7b
@Alystrasz
[feat] SensorBuilder can build Sensor instance
dba7dd1
@Alystrasz
[test] SensorBuilder should not build instance without name
3cb6c52
@Alystrasz
[test] SensorBuilder should not build instance without description
e53fa87
@Alystrasz
[test] SensorBuilder should not build instance without encodingType
836cf02
@Alystrasz
[test] SensorBuilder should not build instance without metadata
9f70c30
@Alystrasz
[fix] SensorBuilder throws if a required attribute is not given
0f1712f
@Alystrasz
[feat] Datastream entity has Observations list
2819de3
@Alystrasz
[feat] Datastream entity has Sensors list
8c13367
@Alystrasz
[test] Datastream should be created with no unit of measurement
a5b5cf3
@Alystrasz
[fix] Datastream.unitOfMeasurement can be null
4564728
@Alystrasz
[test] Datastream should not be created with unit.name === null
7e2deb2
@Alystrasz
[test] Datastream should not be created with unit.definition === null
d8b7eea
@Alystrasz
[test] Datastream should not be created with unit.symbol === null
5fc584b
@Alystrasz
[fix] unitOfMeasurement must have all of its properties either define…
fd273cf 
…d or null
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Alystrasz