Bug 1020682
| Summary: | Integration test for nullable=true | ||
|---|---|---|---|
| Product: | [Retired] JBoss BRMS Platform 6 | Reporter: | Geoffrey De Smet <gdesmet> |
| Component: | OptaPlanner | Assignee: | Geoffrey De Smet <gdesmet> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | Radovan Synek <rsynek> |
| Severity: | high | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 6.0.0 | CC: | mwinkler, rrajasek |
| Target Milestone: | ER6 | ||
| Target Release: | 6.0.0 | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2014-08-06 20:19:12 UTC | Type: | Bug |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
| Embargoed: | |||
|
Description
Geoffrey De Smet
2013-10-18 07:38:43 UTC
I believe this bug will break CH's for that strategic customer: https://issues.jboss.org/browse/PLANNER-197 Note: they aren't using CH at the moment (so it's not a regression), but they are expecting to be able to use it in BRMS 6.0. Integration test for single nullable variable added. https://github.com/droolsjbpm/optaplanner/commit/9a2584d9b0871b05885900d52e81cc290fb5d232 CH problem reproduced on that test and fixed with this commit: https://github.com/droolsjbpm/optaplanner/commit/3a27feb384aed7d546167c1bd6313a87aae5b94b The isMoveDoable() method still has the wrong name, so most of PLANNER-197 is still open. The isMoveDoable() naming will be handled for 6.1. It does not prevent nullable=true from working correctly. So given this issue is fixed (see comment above that links the fix and the integration test to prove it works). Verified with BRMS-6.0.0.ER7 qe_test_coverage: covered by community unit tests |