Test that notes can be moved within a lane

Author: bebraw

immutable_app/app/reducers/lanes.js

@@ -63,13 +63,12 @@ export default function lanes(state = initialState, action) {
  const sourceId = action.sourceId;
  const targetId = action.targetId;
 
- const lanes = state;
- const sourceLane = lanes.filter((lane) => {
+ const sourceLane = state.find((lane) => {
  return lane.notes.indexOf(sourceId) >= 0;
- })[0];
- const targetLane = lanes.filter((lane) => {
+ });
+ const targetLane = state.find((lane) => {
  return lane.notes.indexOf(targetId) >= 0;
- })[0];
+ });
  const sourceNoteIndex = sourceLane.notes.indexOf(sourceId);
  const targetNoteIndex = targetLane.notes.indexOf(targetId);
 

immutable_app/tests/lane_reducer_test.js

@@ -141,7 +141,37 @@ describe('LaneReducer', () => {
  });
 
  it('should allow moving notes within a lane', () => {
+ const lane = {
+ id: 'foobar',
+ name: 'demo lane',
+ notes: []
+ };
+ const sourceNoteId = '123456';
+ const targetNoteId = '654321';
+
+ let lanes = reducer(undefined, {
+ type: types.CREATE_LANE,
+ lane: lane
+ });
+ lanes = reducer(lanes, {
+ type: types.ATTACH_TO_LANE,
+ laneId: lane.id,
+ noteId: sourceNoteId
+ });
+ lanes = reducer(lanes, {
+ type: types.ATTACH_TO_LANE,
+ laneId: lane.id,
+ noteId: targetNoteId
+ });
+ lanes = reducer(lanes, {
+ type: types.MOVE,
+ sourceId: sourceNoteId,
+ targetId: targetNoteId,
+ });
 
+ assert.equal(lanes.get(0).notes.length, 2);
+ assert.equal(lanes.get(0).notes[0], targetNoteId);
+ assert.equal(lanes.get(0).notes[1], sourceNoteId);
  });
 
  it('should allow moving notes from a lane to lane', () => {