rubyforgood / inkind-admin

A project serving Community Education Partnerships - https://www.cep.ngo/ . This Rails application presents an admin interface for CEP to manage their volunteers and students. And provides a GraphQL backend for https://github.com/rubyforgood/inkind-volunteer .
MIT License
9 stars 27 forks source link

improve user models specs #22

Closed gabrielbaldao closed 3 years ago

gabrielbaldao commented 3 years ago

Checklist:

Resolves #000

Description

I trying improve the user models specs, avoid using create when could be use build, btw I have one doubt, make sense into user role, the type active have zero as value?

Type of change

Which of these options are relevant? (remove those that do not apply)

How Has This Been Tested?

This is just spec improve Please describe the tests that you ran to verify your changes and instructions to reproduce.

garettarrowood commented 3 years ago

Thank you for contributing!

I just added these workflow actions a couple days ago, and it looks like I still need to make some adjustments to the rspec workflow.

If you push the linting fixes for the standardrb check, I will run this locally to verify the units tests are passing (and work on CI again myself).

garettarrowood commented 3 years ago

Thanks! And lastly. I believe I have fixed our issue with the rspec / rspec Github action. Could you merge master into this branch once more?

garettarrowood commented 3 years ago

^ With that said. Since you've opened this, the codebase has been renamed. And the name field on the User model has been split.

I'm receptive to adding shoulda-matchers and merging your test updates. But the path of least resistance might be opening up a new PR given the architectural changes. (I've never had an open PR against a repo that changes name. I wonder what happens?)

garettarrowood commented 3 years ago

Great. Thank you!