Bug 1020682 - Integration test for nullable=true
Summary: Integration test for nullable=true
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss BRMS Platform 6
Classification: Retired
Component: OptaPlanner
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ER6
: 6.0.0
Assignee: Geoffrey De Smet
QA Contact: Radovan Synek
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-18 07:38 UTC by Geoffrey De Smet
Modified: 2014-08-06 20:19 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-06 20:19:12 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Geoffrey De Smet 2013-10-18 07:38:43 UTC
For a strategic customer it's absolutely critical that @PlanningVariable(nullable=true) works in OptaPlanner 6, even with multiple variables. This hasn't been tested in an integration test or an example yet. We need to do that.

Comment 2 Geoffrey De Smet 2013-10-18 09:42:07 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.

Comment 3 Geoffrey De Smet 2013-10-24 11:39:18 UTC
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.

Comment 4 Geoffrey De Smet 2013-11-26 15:29:57 UTC
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).

Comment 5 Radovan Synek 2014-01-16 09:34:02 UTC
Verified with BRMS-6.0.0.ER7

Comment 6 Radovan Synek 2014-03-18 16:30:30 UTC
qe_test_coverage: covered by community unit tests


Note You need to log in before you can comment on or make changes to this bug.