Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor code/tests to apply almost rubocop rules #60

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

nthachus
Copy link

  • Change travis configuration to test on jRuby
  • Update gemspec file to manage dependencies instead of Gemfile
  • Use editor configuration file to maintain consistent coding styles
  • Add development guide into README file
  • Update git-ignore file to ignore vendor/bundle path
  • Move class LdapFluff to the root of Gem lib-path
  • Correct Rakefile to skip helpers while running unit-tests
  • Update rubocop configuration file to use almost rules
  • Move ldap_test_helper to the test root as a unit-test entrypoint
  • Reformat all source-code
  • Remove unnecessary active_support from dependencies
  • Rename method is_in_groups? to user_in_groups? by rubocop recommendation
  • Remove unnecessary instance variables by using config property
  • Refactor all code/tests to apply almost rubocop rules
  • Add optional parameter only into find_* methods to get one result
  • Use base method *_filter (e.g. group_filter) to create Net::LDAP::Filter
  • Optimize method users_for_gid to call find_group one-times only
  • Add more tests to cover almost logic cases

- Correct `Rakefile` to skip helpers while running unit-tests
- Remove unnecessary `active_support` from dependencies
- Remove unnecessary instance variables by using `config` property
- Add optional parameter `only` into `find_*` methods to get one result
- Refactor all unit-tests to apply almost `rubocop` rules
- Add more tests to cover almost logic cases
- Optimize method `users_for_gid` to call `find_group` one-times only
… correctly

- Support AD binding with an email like `username@domain`
- Fix circle-loop when biding with configured `service_user`
- Do not lowercase the result login / group names
- Add `attr_member` config to correct Posix LDAP groups finding
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.

None yet

1 participant