一些约定的备忘, 待讨论

[eggjs]

一些约定的备忘, 待讨论

[atian25]

cc @eggjs/core

[shaoshuai0102]

• egg 在文档中全部以小写出现？
• plugin name 即便出现在首行，也都用小写？
• 写 plugin name 的时候，要不要都带上 egg-?

我的建议是

• 小写
• 小写
• 带

其他一些遇到的文档细节约定都在这里讨论吧

@fengmk2 @dead-horse @popomore

[fengmk2]

写 plugin name 的时候，要不要都带上 egg-?

这个不需要吧，模块名称带 egg-，但是插件名称不需要。

[atian25]

egg 全小写这个之前讨论过的，不能首字母大写和全大写

[dead-horse]

我们现在有很多插件是 userService 这样命名的。。

[shaoshuai0102]

命名这个讨论过。

• 最简单的pacakge name: @ali/egg-xx，@alipay/egg-xx，pluginName 为全小写 xx
• 比较长的用中划线package name @ali/egg-foo-bar，@alipay/egg-foo-bar，对应的 pluginName 使用小驼峰，小驼峰转换规则以 package name 的中划线为准 fooBar

对于可以中划线也可以不用的情况，不做强制约定，例如 sessiontair(@alipay/egg-sessiontair) 还是 sessionTair(@alipay/egg-session-tair)，userservice(@alipay/egg-userservice) 还是 user-service(@ali/egg-user-service)。只要遵循以上规则即可，如果选择用中划线，就要按照小驼峰命名 pluginName。

链接不贴了 issue 188

[atian25]

这个命名规范回头可以移植到 docs 下

[fengmk2]

就按 @shaoshuai0102 说的来吧。

[atian25]

注释和文档里面出现的代码, 建议都在第一行加上文件地址

// {app_root}/config/plugin.js

exports.session = {

};

[popomore]

其实这个就是约定，社区肯定会存在驼峰党和非驼峰党

[popomore]

egg 保留的 Symbol 就用全局的，之后可以整理出来

• Symbol.for('egg#eggPath')
• Symbol.for('egg#view')

[atian25]

单元测试

• before 记得调用 app.ready

let app;
before(function* () {
  app = mm.app({
    baseDir: 'security',
  });
  yield app.ready();
});

• supertest 2.x 后支持 promise 了, 可以使用:

it('test', function* (){
  const result = yield require(app.callback()).get('/');
  assert(result.text.match(/hi, egg/));
});

• 使用 power-assert 替换掉 should , 已经内置到 egg-bin

 1) Array #indexOf() should return index when the value is present:
     AssertionError: # path/to/test/mocha_node.js:10

  assert(this.ary.indexOf(zero) === two)
              |   |       |     |   |
              |   |       |     |   2
              |   -1      0     false
              [1,2,3]

  [number] two
  => 2
  [number] this.ary.indexOf(zero)
  => -1

[popomore]

那可以不用 supertest-as-promise 了？

[atian25]

不用了