如何测试AngularJS服务中的私有方法是否被调用？

嗨，作为前端测试新手碰到这种问题太正常啦！咱们先看看你当前的测试代码为啥不工作，再一步步找到合适的解决方案~

你的测试代码为啥失效？ #

你现在的思路是想用$provide.value替换sendEvent，但这个方法根本不成立——因为sendEvent是EventService内部定义的私有函数，它不是通过Angular依赖注入进来的服务/值，所以Angular根本找不到你要替换的"event"依赖，自然没法把你的spy绑定上去。而且你的expect(event.sendEvent)也没有对应的变量，这也是测试报错的原因之一。

推荐的解决方案 #

方案1：测试公共API的行为（最佳实践） #

其实测试的核心应该是验证公共方法的输出/副作用，而不是纠结内部私有方法是否被调用。私有方法是实现细节，只要公共方法的行为符合预期，就算以后重构sendEvent的逻辑，测试依然能通过。

比如你的sendEvent内部调用了$http发送请求，那我们可以mock$httpBackend来验证请求是否符合预期：

describe('EventService', function() {
  var EventService, $httpBackend;

  beforeEach(module('login'));

  beforeEach(inject(function (_EventService_, _$httpBackend_) {
    EventService = _EventService_;
    $httpBackend = _$httpBackend_;
    
    // 这里根据sendEvent实际的请求地址和参数调整
    $httpBackend.expectPOST('/your-event-api-url', {
      category: 'LoginPage',
      action: 'action',
      label: null,
      value: null,
      customParams: { template: 'templateName' }
    }).respond(200);
  }));

  afterEach(function() {
    // 验证没有未处理的请求
    $httpBackend.verifyNoOutstandingExpectation();
    $httpBackend.verifyNoOutstandingRequest();
  });

  it('should send the correct login page event when sendLoginPageEvent is called', function() {
    EventService.sendLoginPageEvent('action');
    $httpBackend.flush(); // 触发mock的请求响应
    // 这里不需要额外expect，因为$httpBackend.expectPOST已经验证了请求参数
  });
});

方案2：重构代码，让私有方法可测试 #

如果你确实需要验证sendEvent是否被调用，最稳妥的方式是把它抽成一个独立的可注入依赖（比如工厂函数），这样测试时就能轻松mock它：

重构后的EventService代码

angular.module('login')
// 把sendEvent抽成独立的工厂
.factory('eventSender', ['$http', '$log', function($http, $log) {
  return function(category, action, label, value, customParams) {
    // 原来sendEvent的逻辑
  };
}])
.service('EventService', ['eventSender', function(eventSender) {
  return {
    sendLoginPageEvent: function(action, customParams) {
      if (!customParams) {
        customParams = {};
      }
      customParams.template = 'templateName';
      eventSender('LoginPage', action, null, null, customParams);
    }
  };
}]);

对应的测试代码

describe('EventService', function() {
  var EventService, eventSenderSpy;

  beforeEach(module('login', function($provide) {
    // 创建spy并替换eventSender工厂
    eventSenderSpy = jasmine.createSpy('eventSender');
    $provide.factory('eventSender', function() {
      return eventSenderSpy;
    });
  }));

  beforeEach(inject(function (_EventService_) {
    EventService = _EventService_;
  }));

  it('should call eventSender with correct parameters', function() {
    EventService.sendLoginPageEvent('action');
    expect(eventSenderSpy).toHaveBeenCalledWith(
      'LoginPage', 
      'action', 
      null, 
      null, 
      { template: 'templateName' }
    );
  });
});

方案3：hack式访问私有方法（不推荐） #

如果实在不想重构代码，也可以通过Angular内部的_invokeQueue来获取服务的构造函数，替换内部的私有方法，但这种方式非常脆弱——一旦Angular版本更新或者代码结构变化，测试就会失效，所以只推荐在紧急情况下使用：

describe('EventService', function() {
  var EventService, sendEventSpy;

  beforeEach(module('login', function($provide) {
    $provide.decorator('EventService', function($delegate, $injector) {
      // 从模块的invokeQueue里找到EventService的构造函数
      var serviceDef = angular.module('login')._invokeQueue.find(function(item) {
        return item[1] === 'service' && item[2][0] === 'EventService';
      });
      var originalConstructor = serviceDef[2][1];
      
      sendEventSpy = jasmine.createSpy('sendEvent');
      // 重新构造服务，替换内部的sendEvent为spy
      var newService = $injector.invoke(function($http, $log) {
        function sendEvent() {
          sendEventSpy.apply(null, arguments);
        }
        return {
          sendLoginPageEvent: function(action, customParams) {
            if (!customParams) {
              customParams = {};
            }
            customParams.template = 'templateName';
            sendEvent('LoginPage', action, null, null, customParams);
          }
        };
      }, null, { $http: $injector.get('$http'), $log: $injector.get('$log') });
      return newService;
    });
  }));

  beforeEach(inject(function (_EventService_) {
    EventService = _EventService_;
  }));

  it('should call sendEvent with correct parameters', function() {
    EventService.sendLoginPageEvent('action');
    expect(sendEventSpy).toHaveBeenCalledWith(
      'LoginPage', 
      'action', 
      null, 
      null, 
      { template: 'templateName' }
    );
  });
});

总结 #

• 优先选择方案1，测试公共API的行为，这是前端测试的最佳实践，能让你的测试更稳定。
• 如果必须验证私有方法的调用，方案2是最稳妥的，符合Angular的依赖注入设计思想。
• 尽量避免使用方案3，除非实在没有其他办法。

内容的提问来源于stack exchange，提问作者Michael Lester