fix: make error messages consistent with go for interop tests

License: MIT
Signed-off-by: achingbrain <[email protected]>

Author: achingbrain

src/core/utils/errors.js

@@ -0,0 +1,22 @@
+'use strict'
+
+class NonFatalError extends Error {
+  constructor (message) {
+    super(message)
+
+    this.code = 0
+  }
+}
+
+class FatalError extends Error {
+  constructor (message) {
+    super(message)
+
+    this.code = 1
+  }
+}
+
+module.exports = {
+  NonFatalError,
+  FatalError
+}

src/core/utils/index.js

@@ -8,6 +8,7 @@ module.exports = {
   countStreamBytes: require('./count-stream-bytes'),
   createLock: require('./create-lock'),
   createNode: require('./create-node'),
+  errors: require('./errors'),
   formatCid: require('./format-cid'),
   limitStreamBytes: require('./limit-stream-bytes'),
   loadNode: require('./load-node'),

src/core/utils/traverse-to.js

@@ -12,6 +12,9 @@ const {
   FILE_SEPARATOR
 } = require('./constants')
 const createNode = require('./create-node')
+const {
+  NonFatalError
+} = require('./errors')
 
 const defaultOptions = {
   parents: false,
@@ -84,15 +87,11 @@ const traverseToMfsObject = (ipfs, path, options, callback) => {
               log(`index ${index} pathSegments.length ${pathSegments.length} pathSegment ${pathSegment} lastComponent ${lastComponent}`, options)
 
               if (lastComponent && !options.createLastComponent) {
-                return done(new Error(`Path ${path.path} does not exist`))
+                log(`Last segment of ${path} did not exist`)
+                return done(new NonFatalError('file does not exist'))
               } else if (!lastComponent && !options.parents) {
-                let message = `Cannot find ${path.path} - ${pathSegment} does not exist`
-
-                if (options.withCreateHint) {
-                  message += ': Try again with the --parents flag to create it'
-                }
-
-                return done(new Error(message))
+                log(`Cannot traverse to ${path} - ${pathSegment} did not exist`)
+                return done(new NonFatalError('file does not exist'))
               }
 
               log(`Adding empty directory '${pathSegment}' to parent ${parent.name}`)

src/http/cp.js

@@ -29,7 +29,7 @@ const mfsCp = (api) => {
           .catch(error => {
             reply({
               Message: error.message,
-              Code: 0,
+              Code: error.code || 0,
               Type: 'error'
             }).code(500).takeover()
           })

src/http/flush.js

@@ -20,7 +20,7 @@ const mfsFlush = (api) => {
           .catch(error => {
             reply({
               Message: error.message,
-              Code: 0,
+              Code: error.code || 0,
               Type: 'error'
             }).code(500).takeover()
           })

src/http/ls.js

@@ -34,7 +34,7 @@ const mfsLs = (api) => {
           .catch(error => {
             reply({
               Message: error.message,
-              Code: 0,
+              Code: error.code || 0,
               Type: 'error'
             }).code(500).takeover()
           })

src/http/mkdir.js

@@ -31,7 +31,7 @@ const mfsMkdir = (api) => {
           .catch(error => {
             reply({
               Message: error.message,
-              Code: 0,
+              Code: error.code || 0,
               Type: 'error'
             }).code(500).takeover()
           })

src/http/mv.js

@@ -29,7 +29,7 @@ const mfsMv = (api) => {
           .catch(error => {
             reply({
               Message: error.message,
-              Code: 0,
+              Code: error.code || 0,
               Type: 'error'
             }).code(500).takeover()
           })

src/http/read.js

@@ -44,7 +44,7 @@ const mfsRead = (api) => {
         stream.once('error', (error) => {
           reply({
             Message: error.message,
-            Code: 0,
+            Code: error.code || 0,
             Type: 'error'
           }).code(500).takeover()
         })

src/http/rm.js

@@ -23,7 +23,7 @@ const mfsRm = (api) => {
           .catch(error => {
             reply({
               Message: error.message,
-              Code: 0,
+              Code: error.code || 0,
               Type: 'error'
             }).code(500).takeover()
           })