Skip to content

add rspec#18

Open
CircleSoar wants to merge 5 commits intospringwq:masterfrom
CircleSoar:addtest-bycircle
Open

add rspec#18
CircleSoar wants to merge 5 commits intospringwq:masterfrom
CircleSoar:addtest-bycircle

Conversation

@CircleSoar
Copy link
Copy Markdown

add rspec

end
end

describe '#feed' do
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

feed 方法测试需要加强,需要考虑 no feeds, 1 feed 和 many feeds 的情况


describe '#create' do
# TODO
context "success" do
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

context 一般这样写 context 'where create successfully'



describe '#destroy' do
# TODO
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

无用的注释语句 需删除

@springwq
Copy link
Copy Markdown
Owner

  • rebase master 分支
  • 跑一下 rubocop
  • 完善测试代码

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants