Skip to content

Commit

Permalink
Move function
Browse files Browse the repository at this point in the history
  • Loading branch information
ehmicky committed Dec 19, 2021
1 parent dbeb321 commit 505b58e
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 11 deletions.
9 changes: 0 additions & 9 deletions src/combination/ids/get.js
Original file line number Diff line number Diff line change
@@ -1,14 +1,5 @@
import { getCombsDimensions, getCombDimensions } from '../dimensions.js'

// Check if two combinations have same identifiers for all dimensions
export const hasSameCombinationIds = function (combinationA, combinationB) {
const combinationIdsA = getCombinationIds(combinationA)
const combinationIdsB = getCombinationIds(combinationB)
return combinationIdsA.every((combinationId) =>
combinationIdsB.includes(combinationId),
)
}

// Retrieve each dimension's id of a given combination
// Follows `DIMENSIONS` array order.
export const getCombinationIds = function (combination) {
Expand Down
11 changes: 10 additions & 1 deletion src/combination/result.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { hasSameCombinationIds } from './ids/get.js'
import { getCombinationIds } from './ids/get.js'

// Filter out the `resultA.combinations` that are not in `resultB`
export const pickResultCombinations = function (resultA, resultB) {
Expand Down Expand Up @@ -38,3 +38,12 @@ export const getMatchingCombination = function (combinations, combination) {
hasSameCombinationIds(combinationA, combination),
)
}

// Check if two combinations have same identifiers for all dimensions
export const hasSameCombinationIds = function (combinationA, combinationB) {
const combinationIdsA = getCombinationIds(combinationA)
const combinationIdsB = getCombinationIds(combinationB)
return combinationIdsA.every((combinationId) =>
combinationIdsB.includes(combinationId),
)
}
2 changes: 1 addition & 1 deletion src/report/reporters/history.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { hasSameCombinationIds } from '../../combination/ids/get.js'
import { hasSameCombinationIds } from '../../combination/result.js'
import { fieldColor } from '../utils/colors.js'
import { getCombinationTitleColor } from '../utils/combination_title.js'
import { STATS_SEPARATOR_COLORED } from '../utils/separator.js'
Expand Down

0 comments on commit 505b58e

Please sign in to comment.