From 4585d39cbc1581e3da9f6246256339feb8adf547 Mon Sep 17 00:00:00 2001
From: Pascal Precht <pascal.precht@gmail.com>
Date: Wed, 18 Jul 2018 11:28:28 +0200
Subject: [PATCH] test(files): add test to ensure meaningful error is thrown in
 `cat()`

To verify changes discussed in https://github.com/ipfs/js-ipfs-api/issues/799#issuecomment-402086401, this commit
adds a test that checks `files.cat()` throws a meaningful error when a key or ipfs path with
unsupported encoding is given.

License: MIT
Signed-off-by: Pascal Precht pascal.precht@gmail.com
---
 js/src/files/cat.js | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/js/src/files/cat.js b/js/src/files/cat.js
index 1d54c567..8927dd16 100644
--- a/js/src/files/cat.js
+++ b/js/src/files/cat.js
@@ -130,6 +130,16 @@ module.exports = (createCommon, options) => {
         })
     })
 
+    it('should error on key/path with invalid encoding', () => {
+      const invalidCid = '/ipfs/QmUzdHisA7ouM8DM5rVAKursVK7t7tKeqDYGvmaCdJw5jEeesdlfjsldkfjsldkfjsdlkfjskldfj'
+
+      return ipfs.files.cat(invalidCid)
+        .catch((err) => {
+          expect(err).to.exist()
+          expect(err.toString()).to.contain('Error: selected encoding not supported')
+        })
+    })
+
     it('should error on unknown path (promised)', () => {
       return ipfs.files.cat(fixtures.smallFile.cid + '/does-not-exist')
         .catch((err) => {