Fixes shuffle always on. Adds test

index.js

@@ -12,7 +12,7 @@ module.exports.getItems = (lang = 'en', shuffle = false) => {
     throw new Error('Inventory not found. Try another language input.')
   }
 
-  const inventory = shuffle ? knuthShuffle(questions) : questions
+  const inventory = shuffle === true ? knuthShuffle(questions) : questions
   return inventory.map((question, i) => Object.assign(question, {choices: choices[question.keyed], num: ++i}))
 }
 

package.json

@@ -1,7 +1,7 @@
 {
   "name": "b5-johnson-120-ipip-neo-pi-r",
   "description": "Module for returning Big Five Johnson 120 IPIP-NEO-PI-R items",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "license": "MIT",
   "author": {
     "name": "Geir Gåsodden",

test/test.js

@@ -2,7 +2,7 @@ const test = require('ava')
 const pkg = require('../package.json')
 const dependencies = pkg.dependencies || false
 const devDependencies = pkg.devDependencies || false
-const {getInfo, getItems} = require('../')
+const { getInfo, getItems } = require('../')
 
 test('basic check', t => {
   t.true(true, 'ava works ok')
@@ -37,6 +37,18 @@ test('it throws error for lang xx', t => {
   }
 })
 
+test('validation of question ids across languages', t => {
+  const languages = getInfo().languages
+  const questions = languages.map(getItems)
+  const ids = questions.map(qs => qs.map(q => q.id))
+  ids.reduce((previous, current) => {
+    if (previous !== false) {
+      t.deepEqual(previous, current, 'ids match')
+    }
+    return current
+  }, false)
+})
+
 test('it returns sorted inventory items', t =>
   t.truthy(getItems('en', false), 'sorted items ok')
 )