Skip to content

Conversation

@tjnet
Copy link
Contributor

@tjnet tjnet commented May 6, 2015

What's this PR do?

add basic tests to this plugin

Where should the reviewer start?

under spec/ directory

How should this be manually tested?

 $ rspec

Any background context you want to provide?

Test code allows us to make changes to code quickly. I know it works now because I've run the tests, when I make the changes I need to make, I need to get the tests working again. This saves hours. 😺

What are the relevant tickets?

#3

Questions:

  • Is this an appropriate testing?
  • If I have a mistake, forgive me and tell me how to be a perfect. I will learn it :' 😄

@tjnet tjnet closed this May 6, 2015
@tjnet tjnet deleted the 3_add_basic_test branch May 6, 2015 07:41
@tjnet tjnet restored the 3_add_basic_test branch May 6, 2015 07:42
@tjnet tjnet reopened this May 6, 2015
@arthurnn
Copy link
Member

Looking good @tjnet . Will take a more closer look and let you know..
@stevecrozz any thoughts?

@stevecrozz
Copy link
Member

@arthurnn thanks for the reminder. yes this does look good. the first few tests that just assert the right constants are returned seem a bit silly, but I don't have a better idea and they do at least exercise the right code.

The only change I'd suggest is very minor. I would add: --require spec_helper to .rspec and remove the require from the spec file. But I'm good with this either way.

@tjnet
Copy link
Contributor Author

tjnet commented May 15, 2015

@stevecrozz @arthurnn thanks for your review!
I'll fix it later. 😺

@tjnet
Copy link
Contributor Author

tjnet commented May 15, 2015

The only change I'd suggest is very minor. I would add: --require spec_helper to .rspec and remove the require from the spec file. But I'm good with this either way.

I fixed it . Thank you for your review. 😺

arthurnn pushed a commit that referenced this pull request May 19, 2015
@arthurnn arthurnn merged commit abe1ffb into errbit:master May 19, 2015
@arthurnn
Copy link
Member

merci!

@tjnet
Copy link
Contributor Author

tjnet commented May 20, 2015

thank you for your feedback and suggestions!

@tjnet tjnet deleted the 3_add_basic_test branch May 22, 2015 06:44
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.

3 participants