reuse duplicated code

This commit is contained in:
Devin Howard 2018-03-20 20:13:34 -07:00
parent 2a2e13de54
commit b89fdf3b0a

View file

@ -8,19 +8,6 @@ import * as EmojiMart from 'emoji-mart'
const sandbox = sinon.sandbox.create() const sandbox = sinon.sandbox.create()
function mockMGraph({ x, y, sx, sy, px, py, width, height, ox, oy }) {
return {
canvas: {
getSize: () => ({ width, height }),
getPos: () => ({ x: px, y: py }),
translateOffsetX: ox,
translateOffsetY: oy,
scaleOffsetX: sx,
scaleOffsetY: sy
}
}
}
describe('Metamaps.Util.js', function() { describe('Metamaps.Util.js', function() {
describe('splitLine', function() { describe('splitLine', function() {
it('splits on words', function() { it('splits on words', function() {
@ -89,80 +76,81 @@ describe('Metamaps.Util.js', function() {
.to.equal('8.6023') .to.equal('8.6023')
}) })
}) })
describe('coordsToPixels', function() { describe('coords/pixels conversions', function() {
function assertCoordsToPixels(expectedX, expectedY, function mockMGraph({ x, y, sx, sy, px, py, width, height, ox, oy }) {
return {
canvas: {
getSize: () => ({ width, height }),
getPos: () => ({ x: px, y: py }),
translateOffsetX: ox,
translateOffsetY: oy,
scaleOffsetX: sx,
scaleOffsetY: sy
}
}
}
function assertConversion(testFunction, expectedX, expectedY,
{ x, y, sx, sy, px, py, width, height, ox, oy }) { { x, y, sx, sy, px, py, width, height, ox, oy }) {
const mGraph = mockMGraph({ const mGraph = mockMGraph({
x, y, sx, sy, px, py, width, height, ox, oy x, y, sx, sy, px, py, width, height, ox, oy
}) })
const coords = { x, y } const coords = { x, y }
const actual = Util.coordsToPixels(mGraph, coords) const actual = testFunction(mGraph, coords)
expect(actual.x).to.equal(expectedX) expect(actual.x).to.equal(expectedX)
expect(actual.y).to.equal(expectedY) expect(actual.y).to.equal(expectedY)
} }
it('returns 0,0 for null canvas', function() { it('coordsToPixels returns 0,0 for null canvas', function() {
expect(Util.coordsToPixels(null, {}).x).to.equal(0) expect(Util.coordsToPixels(null, {}).x).to.equal(0)
expect(Util.coordsToPixels(null, {}).y).to.equal(0) expect(Util.coordsToPixels(null, {}).y).to.equal(0)
}) })
it('does the correct calculation', function() { it('coordsToPixels', function() {
assertCoordsToPixels(0, 0, assertConversion(Util.coordsToPixels, 0, 0,
{ x: 0, y: 0, sx: 1, sy: 1, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 }) { x: 0, y: 0, sx: 1, sy: 1, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(1, 1, assertConversion(Util.coordsToPixels, 1, 1,
{ x: 1, y: 1, sx: 1, sy: 1, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 1, sy: 1, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(2, 1, assertConversion(Util.coordsToPixels, 2, 1,
{ x: 1, y: 1, sx: 2, sy: 1, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 1, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(2, 2, assertConversion(Util.coordsToPixels, 2, 2,
{ x: 1, y: 1, sx: 2, sy: 2, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 2, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(3, 2, assertConversion(Util.coordsToPixels, 3, 2,
{ x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 0, width: 0, height: 0, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 0, width: 0, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(3, 3, assertConversion(Util.coordsToPixels, 3, 3,
{ x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 0, height: 0, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 0, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(4, 3, assertConversion(Util.coordsToPixels, 4, 3,
{ x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 0, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 0, ox: 0, oy: 0 })
assertCoordsToPixels(4, 4, assertConversion(Util.coordsToPixels, 4, 4,
{ x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 2, ox: 0, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 2, ox: 0, oy: 0 })
assertCoordsToPixels(9, 4, assertConversion(Util.coordsToPixels, 9, 4,
{ x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 2, ox: 5, oy: 0 }) { x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 2, ox: 5, oy: 0 })
assertCoordsToPixels(9, 9, assertConversion(Util.coordsToPixels, 9, 9,
{ x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 2, ox: 5, oy: 5 }) { x: 1, y: 1, sx: 2, sy: 2, px: 1, py: 1, width: 2, height: 2, ox: 5, oy: 5 })
}) })
}) it('pixelsToCoords returns 0,0 for null canvas', function() {
describe('pixelsToCoords', function() {
function assertPixelsToCoords(expectedX, expectedY,
{ x, y, px, py, width, height, ox, oy, sx, sy }) {
const mGraph = mockMGraph({
x, y, sx, sy, px, py, width, height, ox, oy
})
const coords = { x, y }
const actual = Util.pixelsToCoords(mGraph, coords)
expect(actual.x).to.equal(expectedX)
expect(actual.y).to.equal(expectedY)
}
it('returns 0,0 for null canvas', function() {
expect(Util.pixelsToCoords(null, {}).x).to.equal(0) expect(Util.pixelsToCoords(null, {}).x).to.equal(0)
expect(Util.pixelsToCoords(null, {}).y).to.equal(0) expect(Util.pixelsToCoords(null, {}).y).to.equal(0)
}) })
it('does the correct calculation', function() { it('pixelsToCoords', function() {
assertPixelsToCoords(0, 0, assertConversion(Util.pixelsToCoords, 0, 0,
{ x: 0, y: 0, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 }) { x: 0, y: 0, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(5, 5, assertConversion(Util.pixelsToCoords, 5, 5,
{ x: 5, y: 5, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 }) { x: 5, y: 5, px: 0, py: 0, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(4, 5, assertConversion(Util.pixelsToCoords, 4, 5,
{ x: 5, y: 5, px: 1, py: 0, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 }) { x: 5, y: 5, px: 1, py: 0, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(4, 4, assertConversion(Util.pixelsToCoords, 4, 4,
{ x: 5, y: 5, px: 1, py: 1, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 }) { x: 5, y: 5, px: 1, py: 1, width: 0, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(3, 4, assertConversion(Util.pixelsToCoords, 3, 4,
{ x: 5, y: 5, px: 1, py: 1, width: 2, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 }) { x: 5, y: 5, px: 1, py: 1, width: 2, height: 0, ox: 0, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(3, 3, assertConversion(Util.pixelsToCoords, 3, 3,
{ x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 0, oy: 0, sx: 1, sy: 1 }) { x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 0, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(2, 3, assertConversion(Util.pixelsToCoords, 2, 3,
{ x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 0, sx: 1, sy: 1 }) { x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 0, sx: 1, sy: 1 })
assertPixelsToCoords(2, 2, assertConversion(Util.pixelsToCoords, 2, 2,
{ x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 1, sx: 1, sy: 1 }) { x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 1, sx: 1, sy: 1 })
assertPixelsToCoords(4, 2, assertConversion(Util.pixelsToCoords, 4, 2,
{ x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 1, sx: 0.5, sy: 1 }) { x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 1, sx: 0.5, sy: 1 })
assertPixelsToCoords(4, 4, assertConversion(Util.pixelsToCoords, 4, 4,
{ x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 1, sx: 0.5, sy: 0.5 }) { x: 5, y: 5, px: 1, py: 1, width: 2, height: 2, ox: 1, oy: 1, sx: 0.5, sy: 0.5 })
}) })
}) })
@ -299,7 +287,6 @@ describe('Metamaps.Util.js', function() {
Util.resizeCanvas(canvas, jQueryStub) Util.resizeCanvas(canvas, jQueryStub)
debugger
expect(canvas.resize.calledWith(7, 8), 'resize not called') expect(canvas.resize.calledWith(7, 8), 'resize not called')
.to.equal(true) .to.equal(true)
expect(canvas.scale.calledWith(1, 2), 'scale not called') expect(canvas.scale.calledWith(1, 2), 'scale not called')
@ -308,7 +295,7 @@ describe('Metamaps.Util.js', function() {
'translate not called') 'translate not called')
.to.equal(true) .to.equal(true)
}) })
}), })
describe('emoji', function() { describe('emoji', function() {
EmojiMart.emojiIndex = { EmojiMart.emojiIndex = {
emojis: { emojis: {
@ -320,8 +307,8 @@ describe('Metamaps.Util.js', function() {
':(': 'emoji2' ':(': 'emoji2'
} }
} }
const withEmojiText = 'test __EMOJI1__ test __EMOJI2__ test'; const withEmojiText = 'test __EMOJI1__ test __EMOJI2__ test'
const noEmojiText = 'test :emoji1: test :emoji2: test'; const noEmojiText = 'test :emoji1: test :emoji2: test'
const emoticonsText = 'test :) test :( test' const emoticonsText = 'test :) test :( test'
const emoticonsReplacedText = 'test __EMOJI1__ test __EMOJI2__ test' const emoticonsReplacedText = 'test __EMOJI1__ test __EMOJI2__ test'
it('removeEmoji replaces emoji with text', function() { it('removeEmoji replaces emoji with text', function() {