Better maxId calculation.

This commit is contained in:
Roger Braun 2016-11-13 22:09:27 +01:00
parent 5e348dc149
commit 5888697c0d
2 changed files with 31 additions and 5 deletions

View file

@ -1,4 +1,4 @@
import { reduce, map, slice, last, intersectionBy, sortBy, unionBy, toInteger, groupBy, differenceBy, each, find } from 'lodash' import { reduce, map, slice, last, intersectionBy, sortBy, unionBy, toInteger, groupBy, differenceBy, each, find, flatten, maxBy } from 'lodash'
import moment from 'moment' import moment from 'moment'
import apiService from '../services/api/api.service.js' import apiService from '../services/api/api.service.js'
import parse from '../services/status_parser/status_parser.js' import parse from '../services/status_parser/status_parser.js'
@ -41,7 +41,7 @@ const statusType = (status) => {
return !status.is_post_verb && status.uri.match(/fave/) ? 'fave' : 'status' return !status.is_post_verb && status.uri.match(/fave/) ? 'fave' : 'status'
} }
const addStatusesToTimeline = (addedStatuses, showImmediately, { statuses, visibleStatuses, newStatusCount, faves, loading }) => { const addStatusesToTimeline = (addedStatuses, showImmediately, { statuses, visibleStatuses, newStatusCount, faves, loading, maxId }) => {
const statusesAndFaves = groupBy(addedStatuses, statusType) const statusesAndFaves = groupBy(addedStatuses, statusType)
const addedFaves = statusesAndFaves['fave'] || [] const addedFaves = statusesAndFaves['fave'] || []
const unseenFaves = differenceBy(addedFaves, faves, 'id') const unseenFaves = differenceBy(addedFaves, faves, 'id')
@ -92,9 +92,9 @@ const addStatusesToTimeline = (addedStatuses, showImmediately, { statuses, visib
statuses: newStatuses, statuses: newStatuses,
visibleStatuses: newVisibleStatuses, visibleStatuses: newVisibleStatuses,
newStatusCount: newNewStatusCount, newStatusCount: newNewStatusCount,
maxId: newStatuses[0].id,
minVisibleId: (last(newVisibleStatuses) || { id: undefined }).id, minVisibleId: (last(newVisibleStatuses) || { id: undefined }).id,
faves: unionBy(faves, addedFaves, 'id'), faves: unionBy(faves, addedFaves, 'id'),
maxId,
loading loading
} }
} }
@ -109,8 +109,18 @@ const updateTimestampsInStatuses = (statuses) => {
}) })
} }
export const findMaxId = (...args) => {
return (maxBy(flatten(args), 'id') || {}).id
}
export const mutations = { export const mutations = {
addNewStatuses (state, { statuses, showImmediately = false, timeline }) { addNewStatuses (state, { statuses, showImmediately = false, timeline }) {
const timelineObject = state.timelines[timeline]
// Set new maxId
const maxId = findMaxId(statuses, timelineObject.statuses)
timelineObject.maxId = maxId
state.timelines[timeline] = addStatusesToTimeline(statuses, showImmediately, state.timelines[timeline]) state.timelines[timeline] = addStatusesToTimeline(statuses, showImmediately, state.timelines[timeline])
state.allStatuses = unionBy(state.timelines[timeline].statuses, state.allStatuses, 'id') state.allStatuses = unionBy(state.timelines[timeline].statuses, state.allStatuses, 'id')

View file

@ -1,5 +1,5 @@
import { cloneDeep } from 'lodash' import { cloneDeep } from 'lodash'
import { defaultState, mutations } from '../../../../src/modules/statuses.js' import { defaultState, mutations, findMaxId } from '../../../../src/modules/statuses.js'
const makeMockStatus = ({id, text}) => { const makeMockStatus = ({id, text}) => {
return { return {
@ -11,6 +11,21 @@ const makeMockStatus = ({id, text}) => {
} }
} }
describe('findMaxId', () => {
it('returns the largest id in any of the given arrays', () => {
const statusesOne = [{ id: 100 }, { id: 2 }]
const statusesTwo = [{ id: 3 }]
const maxId = findMaxId(statusesOne, statusesTwo)
expect(maxId).to.eq(100)
})
it('returns undefined for empty arrays', () => {
const maxId = findMaxId([], [])
expect(maxId).to.eq(undefined)
})
})
describe('The Statuses module', () => { describe('The Statuses module', () => {
it('adds the status to allStatuses and to the given timeline', () => { it('adds the status to allStatuses and to the given timeline', () => {
const state = cloneDeep(defaultState) const state = cloneDeep(defaultState)
@ -96,7 +111,7 @@ describe('The Statuses module', () => {
const favorite = { const favorite = {
id: 2, id: 2,
is_post_verb: false, is_post_verb: false,
in_reply_to_status_id: 1, // The API uses strings here... in_reply_to_status_id: '1', // The API uses strings here...
uri: 'tag:shitposter.club,2016-08-21:fave:3895:note:773501:2016-08-21T16:52:15+00:00', uri: 'tag:shitposter.club,2016-08-21:fave:3895:note:773501:2016-08-21T16:52:15+00:00',
text: 'a favorited something by b' text: 'a favorited something by b'
} }
@ -106,6 +121,7 @@ describe('The Statuses module', () => {
expect(state.timelines.public.visibleStatuses.length).to.eql(1) expect(state.timelines.public.visibleStatuses.length).to.eql(1)
expect(state.timelines.public.visibleStatuses[0].fave_num).to.eql(1) expect(state.timelines.public.visibleStatuses[0].fave_num).to.eql(1)
expect(state.timelines.public.maxId).to.eq(favorite.id)
// Adding again shouldn't change anything // Adding again shouldn't change anything
mutations.addNewStatuses(state, { statuses: [favorite], showImmediately: true, timeline: 'public' }) mutations.addNewStatuses(state, { statuses: [favorite], showImmediately: true, timeline: 'public' })