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

Features/tests #52

Merged
merged 80 commits into from
Feb 2, 2019
Merged

Features/tests #52

merged 80 commits into from
Feb 2, 2019

Conversation

fwitte
Copy link
Member

@fwitte fwitte commented Jan 27, 2019 •
edited
Loading

Introducing test-files for higher coverage, testing:

  • exceptions and errors
  • some fluid property calculations
  • component equations
  • benchmark case

Coverage Status

@fwitte fwitte mentioned this pull request Jan 27, 2019
Closed
16 tasks
@fwitte
fixed doctest
5a9ffed
@fwitte
added tests folder
1221361
@fwitte
fixed FutureWarning from pandas
2653fd0
@fwitte
typo fix
71b29ff
@fwitte
first version of fluid_property_test
9f144ef
@fwitte
adjusted Errors
d0f25a8
@fwitte
first try for error testing
48172ee
@fwitte
code simplification
c5f2be2
@fwitte
adjusted Error Type
556b564
@fwitte
added connection error tests
444e1e7
@fwitte
adjusted error messages for connections
d961322
@fwitte
adjusted fluid specification handling in connections
a9b068b
@fwitte
added tests for bus and ref errors
a7324f0
@fwitte
typo fix in set_attr
07f9efa
@fwitte
fiddling with tests
a89e291
@fwitte
adjustments to tests
721e11c
@fwitte
typo fix
aec3c9d
@fwitte
fixed tests
e575446
@fwitte
typo fix
c7cf242
@fwitte
fixed subsystem dr_eva_natural
23f9a95
@fwitte
added stoichiometric combustion chamber example
5f34f81
@fwitte
removed yes/no question from helpers
67847f4
@fwitte
added error tests for network set_attr, get_attr and instanciation
aed4a31
@fwitte
error message adjustments
6c6e129
@fwitte
adjusted error messages
bf90afb
@fwitte
removed logging message
7fd2b21
@fwitte
Copy link
Member Author

fwitte commented Jan 31, 2019

I will add two benchmark cases by tomorrow.

@fwitte fwitte merged commit 9d2b00e into dev Feb 2, 2019
@fwitte fwitte deleted the features/tests branch February 2, 2019 11:06
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.

None yet
1 participant
@fwitte