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

Create examples-inspired integTests #100

Open
dk1844 opened this issue Jul 21, 2021 · 0 comments · Fixed by #103
Open

Create examples-inspired integTests #100

dk1844 opened this issue Jul 21, 2021 · 0 comments · Fixed by #103
Assignees
@dk1844

Comments

@dk1844
Copy link
Collaborator

dk1844 commented Jul 21, 2021 •
edited
Loading

Currently, the code in examples submodule already serves as integration test (sort of), but pursuing other features (separate module/runnable aspirations/... see #99).

But it would be nice to have actual integration tests instead of logically residing in atum (core module) under src/test.

Optional:

  • the tests can be distinguished by suffix (-IntegrationSuite perhaps) and we can even a named profile to run them
@dk1844 dk1844 self-assigned this Jul 27, 2021
dk1844 added a commit that referenced this issue Jul 27, 2021
@dk1844
#100 SampleMeasurements1IntegrationSuite
1e449ca 
build-all.sh header fix
dk1844 added a commit that referenced this issue Jul 27, 2021
@dk1844
#100 SampleMeasurementsIntegrationSuite (sample1+sample2(<=3)) with c…
57f0401 
…hecking enhanced
dk1844 added a commit that referenced this issue Jul 28, 2021
@dk1844
#100 ControlMeasure integTest not needed, sufficiently covered by Con…
9f8e998 
…trolMeasureBuilderTest (+adjusted the to check the usage of `cmb.withSoftware()`)
dk1844 added a commit that referenced this issue Jul 28, 2021
@dk1844
#100 ControlMeasure integTest not needed, sufficiently covered by Con…
9fa17a9 
…trolMeasureBuilderTest (+adjusted the to check the usage of `cmb.withSoftware()`)
@dk1844 dk1844 linked a pull request Jul 30, 2021 that will close this issue
Examples-inspired integtests #103
Merged
dk1844 added a commit that referenced this issue Aug 5, 2021
@dk1844
#100 post-merge build fix
20c8832
dk1844 added a commit that referenced this issue Aug 6, 2021
@dk1844
#100 check for numeric controlType vs numeric datatype moved "higher"
dd3a53e
dk1844 added a commit that referenced this issue Aug 11, 2021
@dk1844
Examples-inspired integtests (#103)
e7b7d9a 
* #100 SampleMeasurementsIntegrationSuite (sample1+sample2(<=3)) with checking enhanced

* #100 ControlMeasure integTest not needed, sufficiently covered by ControlMeasureBuilderTest (+adjusted the to check the usage of `cmb.withSoftware()`)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dk1844 dk1844

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Examples-inspired integtests AbsaOSS/atum
1 participant
@dk1844