NAH.SH tests: use new validators

test/nahsh.js

@@ -1,57 +1,42 @@
 'use strict'
 
-// todo
-// const getStations = require('db-stations').full
 const tapePromise = require('tape-promise').default
 const tape = require('tape')
 const isRoughlyEqual = require('is-roughly-equal')
-const validateFptf = require('validate-fptf')
-
-const validateLineWithoutMode = require('./lib/validate-line-without-mode')
 
+const {createWhen} = require('./lib/util')
 const co = require('./lib/co')
 const createClient = require('..')
 const nahshProfile = require('../p/nahsh')
-const allProducts = require('../p/nahsh/products')
+const products = require('../p/nahsh/products')
 const {
-	assertValidStation: _assertValidStation,
+	line: createValidateLine,
-	assertValidPoi,
+	station: createValidateStation
-	assertValidAddress,
+} = require('./lib/validators')
-	assertValidLocation,
+const createValidate = require('./lib/validate-fptf-with')
-	assertValidStopover,
-	hour, createWhen, assertValidWhen
-} = require('./lib/util.js')
 
 const when = createWhen('Europe/Berlin', 'de-DE')
 
-// todo: DRY with other tests, move into lib
+const cfg = {
-const assertValidStation = (t, s, coordsOptional = false) => {
+	when,
-	_assertValidStation(t, s, coordsOptional)
+	stationCoordsOptional: false,
-	t.ok(s.products)
+	products
-	for (let product of allProducts) {
+}
-		product = product.id
+
-		const msg = `station.products[${product}] must be a boolean`
+const _validateLine = createValidateLine(cfg)
-		t.equal(typeof s.products[product], 'boolean', msg)
+const validateLine = (validate, l, name) => {
+	if (l && l.product === 'onCall') {
+		// skip line validation
+		// https://github.com/derhuerst/hafas-client/issues/8#issuecomment-355839965
+		l = Object.assign({}, l)
+		l.mode = 'taxi'
 	}
+	_validateLine(validate, l, name)
 }
 
-const isKielHbf = (s) => {
+const validate = createValidate(cfg, {
-	return s.type === 'station' &&
+	line: validateLine
-	(s.id === '8000199') &&
+})
-	s.name === 'Kiel Hbf' &&
-	s.location &&
-	isRoughlyEqual(s.location.latitude, 54.314982, .0005) &&
-	isRoughlyEqual(s.location.longitude, 10.131976, .0005)
-}
-
-const assertIsKielHbf = (t, s) => {
-	t.equal(s.type, 'station')
-	t.ok(s.id === '8000199', 'id should be 8000199')
-	t.equal(s.name, 'Kiel Hbf')
-	t.ok(s.location)
-	t.ok(isRoughlyEqual(s.location.latitude, 54.314982, .0005))
-	t.ok(isRoughlyEqual(s.location.longitude, 10.131976, .0005))
-}
 
 const assertValidPrice = (t, p) => {
 	t.ok(p)
@@ -65,152 +50,102 @@ const assertValidPrice = (t, p) => {
 	}
 }
 
-const assertValidLine = (t, l) => { // with optional mode
-	const validators = Object.assign({}, validateFptf.defaultValidators, {
-		line: validateLineWithoutMode
-	})
-	const recurse = validateFptf.createRecurse(validators)
-	try {
-		recurse(['line'], l, 'line')
-	} catch (err) {
-		t.ifError(err)
-	}
-}
-
 const test = tapePromise(tape)
 const client = createClient(nahshProfile)
 
 const kielHbf = '8000199'
 const flensburg = '8000103'
+const holstentor = '970003547'
 const luebeckHbf = '8000237'
 const husum = '8000181'
 const schleswig = '8005362'
 
 test('Kiel Hbf to Flensburg', co(function* (t) {
 	const journeys = yield client.journeys(kielHbf, flensburg, {
-		when, passedStations: true
+		when, passedStations: true, results: 3
 	})
 
-	t.ok(Array.isArray(journeys))
+	validate(t, journeys, 'journeys', 'journeys')
-	t.ok(journeys.length > 0, 'no journeys')
+	t.strictEqual(journeys.length, 3)
-	for (let journey of journeys) {
+	for (let i = 0; i < journeys.length; i++) {
-		t.equal(journey.type, 'journey')
+		const j = journeys[i]
 
-		t.ok(Array.isArray(journey.legs))
+		const firstLeg = j.legs[0]
-		t.ok(journey.legs.length > 0, 'no legs')
+		const lastLeg = j.legs[j.legs.length - 1]
-		const leg = journey.legs[0] // todo: all legs
+		t.strictEqual(firstLeg.origin.id, kielHbf)
+		t.strictEqual(lastLeg.destination.id, flensburg)
 
-		assertValidStation(t, leg.origin)
+		// todo: find a journey where there pricing info is always available
-		// todo
+		if (j.price) assertValidPrice(t, j.price)
-		// if (!(yield findStation(leg.origin.id))) {
-		// 	console.error('unknown station', leg.origin.id, leg.origin.name)
-		// }
-		assertValidWhen(t, leg.departure, when)
-		t.equal(typeof leg.departurePlatform, 'string')
-
-		assertValidStation(t, leg.destination)
-		// todo
-		// if (!(yield findStation(leg.destination.id))) {
-		// 	console.error('unknown station', leg.destination.id, leg.destination.name)
-		// }
-		assertValidWhen(t, leg.arrival, when)
-		t.equal(typeof leg.arrivalPlatform, 'string')
-
-		assertValidLine(t, leg.line)
-
-		t.ok(Array.isArray(leg.passed))
-		for (let stopover of leg.passed) assertValidStopover(t, stopover)
-
-		if (journey.price) assertValidPrice(t, journey.price)
 	}
-
 	t.end()
 }))
 
+// todo: journeys, only one product
+// todo: journeys, fails with no product
+
 test('Kiel Hbf to Husum, Zingel 10', co(function* (t) {
+	const latitude = 54.475359
+	const longitude = 9.050798
 	const zingel = {
 		type: 'location',
-		latitude: 54.475359,
+		address: 'Husum, Zingel 10',
-		longitude: 9.050798,
+		latitude, longitude
-		address: 'Husum, Zingel 10'
 	}
-
 	const journeys = yield client.journeys(kielHbf, zingel, {when})
 
-	t.ok(Array.isArray(journeys))
+	validate(t, journeys, 'journeys', 'journeys')
-	t.ok(journeys.length >= 1, 'no journeys')
-	const journey = journeys[0]
-	const firstLeg = journey.legs[0]
-	const lastLeg = journey.legs[journey.legs.length - 1]
 
-	assertValidStation(t, firstLeg.origin)
+	const i = journeys[0].legs.length - 1
-	// todo
+	const d = journeys[0].legs[i].destination
-	// if (!(yield findStation(leg.origin.id))) {
+	const name = `journeys[0].legs[${i}].destination`
-	// 	console.error('unknown station', leg.origin.id, leg.origin.name)
-	// }
-	assertValidWhen(t, firstLeg.departure, when)
-	assertValidWhen(t, firstLeg.arrival, when)
-	assertValidWhen(t, lastLeg.departure, when)
-	assertValidWhen(t, lastLeg.arrival, when)
 
-	const d = lastLeg.destination
+	t.strictEqual(d.address, 'Husum, Zingel 10', name + '.address is invalid')
-	assertValidAddress(t, d)
+	t.ok(isRoughlyEqual(.0001, d.latitude, latitude), name + '.latitude is invalid')
-	t.equal(d.address, 'Husum, Zingel 10')
+	t.ok(isRoughlyEqual(.0001, d.longitude, longitude), name + '.longitude is invalid')
-	t.ok(isRoughlyEqual(.0001, d.latitude, 54.475359))
-	t.ok(isRoughlyEqual(.0001, d.longitude, 9.050798))
 
 	t.end()
 }))
 
-test('Holstentor to Kiel Hbf', co(function* (t) {
+test('Kiel Hbf to Holstentor', co(function* (t) {
-	const holstentor = {
+	const latitude = 53.866321
+	const longitude = 10.679976
+	const name = 'Hansestadt Lübeck, Holstentor (Denkmal)'
+	const journeys = yield client.journeys(kielHbf, {
 		type: 'location',
-		latitude: 53.866321,
+		id: holstentor,
-		longitude: 10.679976,
+		name,
-		name: 'Hansestadt Lübeck, Holstentor (Denkmal)',
+		latitude, longitude
-		id: '970003547'
+	}, {when})
-	}
-	const journeys = yield client.journeys(holstentor, kielHbf, {when})
 
-	t.ok(Array.isArray(journeys))
+	validate(t, journeys, 'journeys', 'journeys')
-	t.ok(journeys.length >= 1, 'no journeys')
-	const journey = journeys[0]
-	const firstLeg = journey.legs[0]
-	const lastLeg = journey.legs[journey.legs.length - 1]
 
-	const o = firstLeg.origin
+	const i = journeys[0].legs.length - 1
-	assertValidPoi(t, o)
+	const d = journeys[0].legs[i].destination
-	t.equal(o.name, 'Hansestadt Lübeck, Holstentor (Denkmal)')
+	const k = `journeys[0].legs[${i}].destination`
-	t.ok(isRoughlyEqual(.0001, o.latitude, 53.866321))
-	t.ok(isRoughlyEqual(.0001, o.longitude, 10.679976))
 
-	assertValidWhen(t, firstLeg.departure, when)
+	t.strictEqual(d.id, holstentor, k + '.id is invalid')
-	assertValidWhen(t, firstLeg.arrival, when)
+	t.strictEqual(d.name, name, k + '.name is invalid')
-	assertValidWhen(t, lastLeg.departure, when)
+	t.ok(isRoughlyEqual(.0001, d.latitude, latitude), k + '.latitude is invalid')
-	assertValidWhen(t, lastLeg.arrival, when)
+	t.ok(isRoughlyEqual(.0001, d.longitude, longitude), k + '.longitude is invalid')
-
-	assertValidStation(t, lastLeg.destination)
-	// todo
-	// if (!(yield findStation(leg.destination.id))) {
-	// 	console.error('unknown station', leg.destination.id, leg.destination.name)
-	// }
 
 	t.end()
 }))
 
-test('Husum to Lübeck Hbf with stopover at Husum', co(function* (t) {
+test('Husum to Lübeck Hbf with stopover at Kiel Hbf', co(function* (t) {
-	const [journey] = yield client.journeys(husum, luebeckHbf, {
+	const journeys = yield client.journeys(husum, luebeckHbf, {
 		via: kielHbf,
-		results: 1,
+		results: 1, when, passedStations: true
-		when
 	})
 
-	const i1 = journey.legs.findIndex(leg => leg.destination.id === kielHbf)
+	validate(t, journeys, 'journeys', 'journeys')
-	t.ok(i1 >= 0, 'no leg with Kiel Hbf as destination')
 
-	const i2 = journey.legs.findIndex(leg => leg.origin.id === kielHbf)
+	const leg = journeys[0].legs.some((leg) => {
-	t.ok(i2 >= 0, 'no leg with Kiel Hbf as origin')
+		return leg.passed && leg.passed.some((passed) => {
-	t.ok(i2 > i1, 'leg with Kiel Hbf as origin must be after leg to it')
+			return passed.station.id === kielHbf
+		})
+	})
+	t.ok(leg, 'Kiel Hbf is not being passed')
 
 	t.end()
 }))
@@ -220,6 +155,7 @@ test('earlier/later journeys, Kiel Hbf -> Flensburg', co(function* (t) {
 		results: 3, when
 	})
 
+	// todo: move to journeys validator?
 	t.equal(typeof model.earlierRef, 'string')
 	t.ok(model.earlierRef)
 	t.equal(typeof model.laterRef, 'string')
@@ -230,11 +166,15 @@ test('earlier/later journeys, Kiel Hbf -> Flensburg', co(function* (t) {
 		client.journeys(kielHbf, flensburg, {
 			when, earlierThan: model.earlierRef
 		})
+		// silence rejections, we're only interested in exceptions
+		.catch(() => {})
 	})
 	t.throws(() => {
 		client.journeys(kielHbf, flensburg, {
 			when, laterThan: model.laterRef
 		})
+		// silence rejections, we're only interested in exceptions
+		.catch(() => {})
 	})
 
 	let earliestDep = Infinity, latestDep = -Infinity
@@ -265,7 +205,7 @@ test('earlier/later journeys, Kiel Hbf -> Flensburg', co(function* (t) {
 	t.end()
 }))
 
-test('leg details for Flensburg to Husum', co(function* (t) {
+test('journey leg details for Flensburg to Husum', co(function* (t) {
 	const journeys = yield client.journeys(flensburg, husum, {
 		results: 1, when
 	})
@@ -275,17 +215,7 @@ test('leg details for Flensburg to Husum', co(function* (t) {
 	t.ok(p.line.name, 'precondition failed')
 	const leg = yield client.journeyLeg(p.id, p.line.name, {when})
 
-	t.equal(typeof leg.id, 'string')
+	validate(t, leg, 'journeyLeg', 'leg')
-	t.ok(leg.id)
-
-	assertValidLine(t, leg.line)
-
-	t.equal(typeof leg.direction, 'string')
-	t.ok(leg.direction)
-
-	t.ok(Array.isArray(leg.passed))
-	for (let passed of leg.passed) assertValidStopover(t, passed)
-
 	t.end()
 }))
 
@@ -294,16 +224,34 @@ test('departures at Kiel Hbf', co(function* (t) {
 		duration: 30, when
 	})
 
-	t.ok(Array.isArray(deps))
+	validate(t, deps, 'departures', 'departures')
-	for (let dep of deps) {
+	for (let i = 0; i < deps.length; i++) {
-		assertValidStation(t, dep.station)
+		const dep = deps[i]
-		// todo
+		const name = `deps[${i}]`
-		// if (!(yield findStation(dep.station.id))) {
-		// 	console.error('unknown station', dep.station.id, dep.station.name)
-		// }
-		assertValidWhen(t, dep.when, when)
-	}
 
+		// todo: fix this
+		// t.equal(dep.station.name, 'Kiel Hauptbahnhof', name + '.station.name is invalid')
+		// t.equal(dep.station.id, kielHbf, name + '.station.id is invalid')
+	}
+	// todo: move into deps validator
+	t.deepEqual(deps, deps.sort((a, b) => t.when > b.when))
+
+	t.end()
+}))
+
+test('departures with station object', co(function* (t) {
+	const deps = yield client.departures({
+		type: 'station',
+		id: kielHbf,
+		name: 'Kiel Hbf',
+		location: {
+			type: 'location',
+			latitude: 1.23,
+			longitude: 2.34
+		}
+	}, {when})
+
+	validate(t, deps, 'departures', 'departures')
 	t.end()
 }))
 
@@ -317,92 +265,65 @@ test('nearby Kiel Hbf', co(function* (t) {
 		results: 2, distance: 400
 	})
 
+	validate(t, nearby, 'locations', 'nearby')
+
 	t.ok(Array.isArray(nearby))
 	t.equal(nearby.length, 2)
 
-	assertIsKielHbf(t, nearby[0])
+	t.equal(nearby[0].id, kielHbf)
+	t.equal(nearby[0].name, 'Kiel Hbf')
 	t.ok(nearby[0].distance >= 0)
 	t.ok(nearby[0].distance <= 100)
 
-	for (let n of nearby) {
-		if (n.type === 'station') assertValidStation(t, n)
-		else assertValidLocation(t, n)
-	}
-
 	t.end()
 }))
 
 test('locations named Kiel', co(function* (t) {
 	const locations = yield client.locations('Kiel', {
-		results: 10
+		results: 20
 	})
 
-	t.ok(Array.isArray(locations))
+	validate(t, locations, 'locations', 'locations')
-	t.ok(locations.length > 0)
+	t.ok(locations.length <= 20)
-	t.ok(locations.length <= 10)
 
-	for (let l of locations) {
+	t.ok(locations.find(s => s.type === 'station'))
-		if (l.type === 'station') assertValidStation(t, l)
+	t.ok(locations.find(s => s.id && s.name)) // POIs
-		else assertValidLocation(t, l)
+	t.ok(locations.some(l => l.id === kielHbf))
-	}
-	t.ok(locations.some(isKielHbf))
 
 	t.end()
 }))
 
 test('location', co(function* (t) {
-	const loc = yield client.location(schleswig)
+	const s = yield client.location(kielHbf)
 
-	assertValidStation(t, loc)
+	validate(t, s, 'station', 'station')
-	t.equal(loc.id, schleswig)
+	t.equal(s.id, kielHbf)
 
 	t.end()
 }))
 
-// todo: see #34
+test('radar', co(function* (t) {
-test.skip('radar Kiel', co(function* (t) {
+	const vehicles = yield client.radar({
-	const vehicles = yield client.radar(54.4, 10.0, 54.2, 10.2, {
+		north: 54.4,
+		west: 10.0,
+		south: 54.2,
+		east: 10.2
+	}, {
 		duration: 5 * 60, when
 	})
 
-	t.ok(Array.isArray(vehicles))
+	const allProducts = products.reduce((acc, p) => (acc[p.id] = true, acc), {})
-	t.ok(vehicles.length > 0)
+	const validateStation = createValidateStation(cfg)
-	for (let v of vehicles) {
+	const validate = createValidate(cfg, {
-
+		station: (validate, s, name) => {
-		// todo
+			s = Object.assign({
-		// t.ok(findStation(v.direction))
+				products: allProducts // todo: fix station.products
-		assertValidLine(t, v.line)
+			}, s)
-
+			if (!s.name) s.name = 'foo' // todo, see #34
-		t.equal(typeof v.location.latitude, 'number')
+			validateStation(validate, s, name)
-		t.ok(v.location.latitude <= 57, 'vehicle is too far away')
-		t.ok(v.location.latitude >= 51, 'vehicle is too far away')
-		t.equal(typeof v.location.longitude, 'number')
-		t.ok(v.location.longitude >= 7, 'vehicle is too far away')
-		t.ok(v.location.longitude <= 13, 'vehicle is too far away')
-
-		t.ok(Array.isArray(v.nextStops))
-		for (let st of v.nextStops) {
-			assertValidStopover(t, st, true)
-
-			if (st.arrival) {
-				t.equal(typeof st.arrival, 'string')
-				const arr = +new Date(st.arrival)
-				// note that this can be an ICE train
-				t.ok(isRoughlyEqual(14 * hour, +when, arr))
-			}
-			if (st.departure) {
-				t.equal(typeof st.departure, 'string')
-				const dep = +new Date(st.departure)
-				t.ok(isRoughlyEqual(14 * hour, +when, dep))
-			}
 		}
+	})
+	validate(t, vehicles, 'movements', 'vehicles')
 
-		t.ok(Array.isArray(v.frames))
-		for (let f of v.frames) {
-			assertValidStation(t, f.origin, true)
-			assertValidStation(t, f.destination, true)
-			t.equal(typeof f.t, 'number')
-		}
-	}
 	t.end()
 }))