摩卡/柴返回未定义

Question

我有一个工作节点脚本，现在正在尝试学习 mocha/chai 以添加一些单元测试。

//Module Code
var path = require('path');
var recursive = require('recursive-readdir');


function findData(folderPath) {
  recursive(folderPath, function (err, files) {
    return files;
  });
};

module.exports.findData = findData;

我的摩卡测试代码：

var expect = require('chai').expect;

describe('checkData', function () {
  var findData = require('../custom_modules/findData').findData;
  var path;

  before (function () {
    path = '/Data'
  });

  it('should have 53 files in array', function () {
    expect(findData(path)).to.have.lengthOf(53);
  })

});

但是，它总是失败，因为返回似乎是未定义的。所以我剥离了我的模块代码来测试 return true 并且有效。

所以它必须是递归模块的异步特性，所以我尝试在我的模块代码中添加回调：

var path = require('path');
var recursive = require('recursive-readdir');


function findData(folderPath, cb) {
  recursive(folderPath, function (err, files) {
    cb(null, files);
  });
};

module.exports.findData = findData;

但这也没有用。

奇怪的是，如果我运行node index.js，我会得到文件列表。

谁能向我解释这段代码如何正常工作，但是当我尝试使用 mocha/chai 进行测试时，我得到了未定义？

谢谢

已编辑：

因此，根据@Louis 在 cmets 中所说的，我为函数添加了回调。

describe('checkData', function () {
  var findData = require('../custom_modules/findData').findData;
  var path;
  var files;

  before (function () {
    path = '/Users/tjmartin/Documents/OSData/OpenNames/Data'
  });

  it('should have 53 files in array', function () {
    expect(findData(path, function(results) {
        files = results;
      })).to.have.lengthOf(53);
  })
});

但这仍然返回一个未定义的。

Answer 1

首先，我会在您的 findData 实现中记录错误结果。即使只是在开发期间，您也可以轻松查看是否报告了任何错误（您可能已经这样做了，只是想提一下）。

正如您所发现的，问题的主要原因之一是回调是异步的。因此，您不能像在原始示例中那样简单地从您的 findData 方法中返回它。

接下来，我不会像在 before 函数中那样对路径进行硬编码。而是使用本地路径，以便测试可以作为 CI 的一部分运行（如果你有的话），甚至可以在另一台机器上获取它并让它在那里工作。

before(function() {
    path = './tests/TestData';
});

在您修改后的示例中，尽管您使用的是回调，但您仍在测试返回结果。您需要更改您的测试以使用回调的结果。

it('should have 53 files in array', function(done) {
    findData(path, function(results) {
        expect(results).to.have.lengthOf(53);
        done();
    });
});