From 80aee8bcc0f0b433837ee8d8850e9a82a30f242f Mon Sep 17 00:00:00 2001 From: junv Date: Sun, 7 Jul 2013 23:11:15 +0800 Subject: [PATCH] try to fix set display name bug --- api/services/WeixinService.js | 2 +- test/actionService.test.js | 16 ++++++++-------- test/test.js | 7 +++++++ 3 files changed, 16 insertions(+), 9 deletions(-) diff --git a/api/services/WeixinService.js b/api/services/WeixinService.js index d63c246..ea41255 100644 --- a/api/services/WeixinService.js +++ b/api/services/WeixinService.js @@ -29,7 +29,7 @@ var WeixinService = { } else if (content === 'menu' || content === 'm') { callback(null, constants.reply.menu); return; - } else if (new RegExp('^[=]{1}[^=-+]*$').test(content)) { + } else if (new RegExp('^[=]{1}[^=+-]*$').test(content)) { ActionsService.setDisplayName(message.FromUserName, content, function (err, result) { if (err) { callback(null, contants.reply.systemErr); diff --git a/test/actionService.test.js b/test/actionService.test.js index 93a8219..ad67961 100644 --- a/test/actionService.test.js +++ b/test/actionService.test.js @@ -2,12 +2,12 @@ var actionService = require('../api/services/ActionService.js'); var assert = require('assert'); describe('ActionService test',function(){ - it('should set display name success',function(){ - var origionalName = ''; - var newName = '=Test'; - actionService.setDisplayName(origionalName,newName,function(err,result){ - return result; - }); - - }); +// it('should set display name success',function(){ +// var origionalName = ''; +// var newName = '=Test'; +// actionService.setDisplayName(origionalName,newName,function(err,result){ +// return result; +// }); +// +// }); }); diff --git a/test/test.js b/test/test.js index 4656cfa..8044680 100644 --- a/test/test.js +++ b/test/test.js @@ -57,4 +57,11 @@ describe('Sohub', function () { }); + + describe('action pattern test',function(){ + it('should be set display name action',function(){ + assert.equal(new RegExp('^[=]{1}[^=+-]*$').test('=junv'),true); + }); + }); + }) \ No newline at end of file