-
Notifications
You must be signed in to change notification settings - Fork 4
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
Stub out API test cases related to Reports #95
Milestone
Comments
We should suggest that time series reports get punted to the future, because this does not make sense without scheduling |
Sample "density report" as I imagine it:
Obviously the 27.5 cores is silly |
|
Also add test stub (functionality not implemented yet) for redacting fields of reports at some "export" stage |
kdelee
added a commit
that referenced
this issue
Jan 25, 2018
Outline test cases for reports. Reports are still very much in flux and we need some more details about what facts will be present, with what priority they are chosen between sources when different information is merged, and what queries to the reports endpoint will be acceptable. That being said, this is a good start and gets the test cases in a format so we can view them from our read the docs website. Closes #95 Closes #94 Closes #98
kdelee
added a commit
that referenced
this issue
Jan 25, 2018
Outline test cases for reports. Reports are still very much in flux and we need some more details about what facts will be present, with what priority they are chosen between sources when different information is merged, and what queries to the reports endpoint will be acceptable. That being said, this is a good start and gets the test cases in a format so we can view them from our read the docs website. Closes #95 Closes #94 Closes #98
kdelee
added a commit
that referenced
this issue
Jan 26, 2018
Outline test cases for reports. Reports are still very much in flux and we need some more details about what facts will be present, with what priority they are chosen between sources when different information is merged, and what queries to the reports endpoint will be acceptable. That being said, this is a good start and gets the test cases in a format so we can view them from our read the docs website. Closes #95 Closes #94 Closes #98
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This is another area where we need to stub out tests for functionality that does not exist much yet, so they need to be relatively high level.
Tests for all reports:
Test by Report type:
/etc/redhat-release
which is a string, but a boolean or ternary fact that says YES/NO/MAYBE this is RHEL 4)One outstanding question regarding Density and Raw reports if they will also aggregate data from multiple scans and do any deduplication. Time series by definition must take in multiple fact collections, but these would obstesibly be from different points in time. It is unclear if they must be able to merge fact collections from the same "time step"
The text was updated successfully, but these errors were encountered: