[RFE] Ironic functional tests and hacking checks
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ironic |
Fix Released
|
Wishlist
|
jiang wei |
Bug Description
For now, there is nothing in ironic/
The functional tests could come from these perspectives below:
1. API tests: Also known as API sample tests. A test case in this part is
going to send JSON based API requests to Ironic server, and gets a
response and check if the response is correct as we expects.
2. DB tests: Tests for DB migrations, models and so on.
3. Driver tests: Driver providers could add functional tests for their
specific driver in some specific scenarios.
4. Regression tests: Tests for some fixed bugs.
And also hacking checks:
1. Hacking checks: These could be some additional checks for pep8 tests.
For example, if we want to assert if a value is None, we should use
assertIsNone
things like this, and we can add hacking checks to enhance it.
Please also take a look at this REF below:
https:/
Now, Ironic does its unit tests more like functional, which is totally
unnecessary. I think if we want to do functional tests, we have to split
the functional-like part of code from the unit tests first.
And also refer to ironic client functional tests:
https:/
Changed in ironic: | |
status: | New → Triaged |
importance: | Undecided → Wishlist |
Changed in ironic: | |
assignee: | nobody → John L. Villalovos (happycamp) |
status: | Triaged → In Progress |
description: | updated |
description: | updated |
summary: |
- ironic does not have functional testing + [RFE] ironic does not have functional testing |
tags: | added: rfe |
tags: |
added: rfe-approved removed: rfe |
Changed in ironic: | |
assignee: | John L. Villalovos (happycamp) → Tang Chen (tangchen) |
description: | updated |
description: | updated |
Changed in ironic: | |
assignee: | Tang Chen (tangchen) → jiang wei (timjiang) |
Changed in ironic: | |
status: | In Progress → Fix Released |
Reviewed: https:/ /review. openstack. org/228612 /git.openstack. org/cgit/ openstack/ ironic/ commit/ ?id=f350d87af82 ba8e33c19fb03ee b67625027838db
Committed: https:/
Submitter: Jenkins
Branch: master
commit f350d87af82ba8e 33c19fb03eeb676 25027838db
Author: John L. Villalovos <email address hidden>
Date: Mon Sep 28 11:36:37 2015 -0700
Prepare for functional testing
Prepare for functional testing by creating two new directories: tests/functiona l
* ironic/tests/unit
* ironic/
Move all files currently in ironic/tests/* to ironic/tests/unit/
Update imports from 'ironic.tests.YYY' to 'ironic. tests.unit. YYY'
Fix any PEP8 issues due to longer lines
Partial-Bug: #1491670 b2042b62e348926 502a4851e08
Change-Id: I1166f20adf8c84